Title: [161088] trunk/Source/WebCore
- Revision
- 161088
- Author
- [email protected]
- Date
- 2013-12-26 09:58:56 -0800 (Thu, 26 Dec 2013)
Log Message
Move more inlines from RenderObject to RenderElement.
<https://webkit.org/b/126134>
Lift some inline functions that use style() from RenderObject over
to RenderElement, making them branchless.
Reviewed by Anders Carlsson.
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (161087 => 161088)
--- trunk/Source/WebCore/ChangeLog 2013-12-26 15:04:22 UTC (rev 161087)
+++ trunk/Source/WebCore/ChangeLog 2013-12-26 17:58:56 UTC (rev 161088)
@@ -1,3 +1,13 @@
+2013-12-22 Andreas Kling <[email protected]>
+
+ Move more inlines from RenderObject to RenderElement.
+ <https://webkit.org/b/126134>
+
+ Lift some inline functions that use style() from RenderObject over
+ to RenderElement, making them branchless.
+
+ Reviewed by Anders Carlsson.
+
2013-12-26 ChangSeok Oh <[email protected]>
Unreviewed build fix after r159526.
Modified: trunk/Source/WebCore/rendering/RenderElement.h (161087 => 161088)
--- trunk/Source/WebCore/rendering/RenderElement.h 2013-12-26 15:04:22 UTC (rev 161087)
+++ trunk/Source/WebCore/rendering/RenderElement.h 2013-12-26 17:58:56 UTC (rev 161088)
@@ -120,8 +120,12 @@
bool isTransparent() const { return style().opacity() < 1.0f; }
float opacity() const { return style().opacity(); }
+ bool visibleToHitTesting() const { return style().visibility() == VISIBLE && style().pointerEvents() != PE_NONE; }
+
bool hasBackground() const { return style().hasBackground(); }
bool hasMask() const { return style().hasMask(); }
+ bool hasClip() const { return isOutOfFlowPositioned() && style().hasClip(); }
+ bool hasClipOrOverflowClip() const { return hasClip() || hasOverflowClip(); }
bool hasClipPath() const { return style().clipPath(); }
bool hasHiddenBackface() const { return style().backfaceVisibility() == BackfaceVisibilityHidden; }
Modified: trunk/Source/WebCore/rendering/RenderLayerBacking.cpp (161087 => 161088)
--- trunk/Source/WebCore/rendering/RenderLayerBacking.cpp 2013-12-26 15:04:22 UTC (rev 161087)
+++ trunk/Source/WebCore/rendering/RenderLayerBacking.cpp 2013-12-26 17:58:56 UTC (rev 161088)
@@ -1656,7 +1656,7 @@
}
}
-static bool supportsDirectBoxDecorationsComposition(const RenderObject& renderer)
+static bool supportsDirectBoxDecorationsComposition(const RenderLayerModelObject& renderer)
{
if (!GraphicsLayer::supportsBackgroundColorContent())
return false;
Modified: trunk/Source/WebCore/rendering/RenderObject.h (161087 => 161088)
--- trunk/Source/WebCore/rendering/RenderObject.h 2013-12-26 15:04:22 UTC (rev 161087)
+++ trunk/Source/WebCore/rendering/RenderObject.h 2013-12-26 17:58:56 UTC (rev 161088)
@@ -565,9 +565,7 @@
bool isSelectionBorder() const;
- bool hasClip() const { return isOutOfFlowPositioned() && style().hasClip(); }
bool hasOverflowClip() const { return m_bitfields.hasOverflowClip(); }
- bool hasClipOrOverflowClip() const { return hasClip() || hasOverflowClip(); }
bool hasTransform() const { return m_bitfields.hasTransform(); }
@@ -871,8 +869,6 @@
AnimationController& animation() const;
- bool visibleToHitTesting() const { return style().visibility() == VISIBLE && style().pointerEvents() != PE_NONE; }
-
// Map points and quads through elements, potentially via 3d transforms. You should never need to call these directly; use
// localToAbsolute/absoluteToLocal methods instead.
virtual void mapLocalToContainer(const RenderLayerModelObject* repaintContainer, TransformState&, MapCoordinatesFlags = ApplyContainerFlip, bool* wasFixed = 0) const;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
