Title: [210770] trunk/Source/WebCore
- Revision
- 210770
- Author
- [email protected]
- Date
- 2017-01-14 11:59:26 -0800 (Sat, 14 Jan 2017)
Log Message
Small code cleanup after r210760
https://bugs.webkit.org/show_bug.cgi?id=167047
Reviewed by Sam Weinig.
No change in functionality.
* rendering/RenderLayer.cpp:
(WebCore::RenderLayer::updateLayerPosition):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (210769 => 210770)
--- trunk/Source/WebCore/ChangeLog 2017-01-14 19:35:23 UTC (rev 210769)
+++ trunk/Source/WebCore/ChangeLog 2017-01-14 19:59:26 UTC (rev 210770)
@@ -1,5 +1,17 @@
2017-01-14 Zalan Bujtas <[email protected]>
+ Small code cleanup after r210760
+ https://bugs.webkit.org/show_bug.cgi?id=167047
+
+ Reviewed by Sam Weinig.
+
+ No change in functionality.
+
+ * rendering/RenderLayer.cpp:
+ (WebCore::RenderLayer::updateLayerPosition):
+
+2017-01-14 Zalan Bujtas <[email protected]>
+
Renderers should have a simple way to access Settings.
https://bugs.webkit.org/show_bug.cgi?id=167048
Modified: trunk/Source/WebCore/rendering/RenderLayer.cpp (210769 => 210770)
--- trunk/Source/WebCore/rendering/RenderLayer.cpp 2017-01-14 19:35:23 UTC (rev 210769)
+++ trunk/Source/WebCore/rendering/RenderLayer.cpp 2017-01-14 19:59:26 UTC (rev 210770)
@@ -1348,8 +1348,8 @@
box->applyTopLeftLocationOffset(localPoint);
}
- RenderElement* ancestor;
- if (!renderer().isOutOfFlowPositioned() && (ancestor = renderer().parent())) {
+ if (!renderer().isOutOfFlowPositioned()) {
+ auto* ancestor = renderer().parent();
// We must adjust our position by walking up the render tree looking for the
// nearest enclosing object with a layer.
while (ancestor && !ancestor->hasLayer()) {
@@ -1360,9 +1360,9 @@
}
ancestor = ancestor->parent();
}
- if (ancestor && is<RenderBox>(*ancestor) && is<RenderTableRow>(*ancestor)) {
+ if (is<RenderTableRow>(ancestor)) {
// Put ourselves into the row coordinate space.
- localPoint -= downcast<RenderBox>(*ancestor).topLeftLocationOffset();
+ localPoint -= downcast<RenderTableRow>(*ancestor).topLeftLocationOffset();
}
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
