Title: [163459] trunk/Source/WebCore
Revision
163459
Author
[email protected]
Date
2014-02-05 10:51:22 -0800 (Wed, 05 Feb 2014)

Log Message

Remove leftover seamless iframe logic from containerForRepaint().
<https://webkit.org/b/128235>

The parent-flow-thread-in-different-document case is no longer
relevant after <iframe seamless> was removed.

Reviewed by David Hyatt.

* rendering/RenderObject.cpp:
(WebCore::RenderObject::containerForRepaint):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (163458 => 163459)


--- trunk/Source/WebCore/ChangeLog	2014-02-05 18:39:36 UTC (rev 163458)
+++ trunk/Source/WebCore/ChangeLog	2014-02-05 18:51:22 UTC (rev 163459)
@@ -1,3 +1,16 @@
+2014-02-05  Andreas Kling  <[email protected]>
+
+        Remove leftover seamless iframe logic from containerForRepaint().
+        <https://webkit.org/b/128235>
+
+        The parent-flow-thread-in-different-document case is no longer
+        relevant after <iframe seamless> was removed.
+
+        Reviewed by David Hyatt.
+
+        * rendering/RenderObject.cpp:
+        (WebCore::RenderObject::containerForRepaint):
+
 2014-02-05  Hans Muller  <[email protected]>
 
         [CSS Shapes] Dynamically created element with image valued shape-outside doesn't update automatically

Modified: trunk/Source/WebCore/rendering/RenderObject.cpp (163458 => 163459)


--- trunk/Source/WebCore/rendering/RenderObject.cpp	2014-02-05 18:39:36 UTC (rev 163458)
+++ trunk/Source/WebCore/rendering/RenderObject.cpp	2014-02-05 18:51:22 UTC (rev 163459)
@@ -1225,10 +1225,6 @@
         // then the repaint container is not the flow thread.
         if (hasFixedPosInNamedFlowContainingBlock(this))
             return repaintContainer;
-        // The ancestor document will do the reparenting when the repaint propagates further up.
-        // We're just a seamless child document, and we don't need to do the hacking.
-        if (parentRenderFlowThread && &parentRenderFlowThread->document() != &document())
-            return repaintContainer;
         // If we have already found a repaint container then we will repaint into that container only if it is part of the same
         // flow thread. Otherwise we will need to catch the repaint call and send it to the flow thread.
         RenderFlowThread* repaintContainerFlowThread = repaintContainer ? repaintContainer->flowThreadContainingBlock() : 0;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to