Title: [150966] trunk/Source/WebCore
Revision
150966
Author
[email protected]
Date
2013-05-30 10:03:53 -0700 (Thu, 30 May 2013)

Log Message

Fix double hash lookup in RenderView::repaintSelection() loop.
<http://webkit.org/b/117029>

Reviewed by Anders Carlsson.

* rendering/RenderView.cpp:
(WebCore::RenderView::repaintSelection):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (150965 => 150966)


--- trunk/Source/WebCore/ChangeLog	2013-05-30 16:44:32 UTC (rev 150965)
+++ trunk/Source/WebCore/ChangeLog	2013-05-30 17:03:53 UTC (rev 150966)
@@ -1,3 +1,13 @@
+2013-05-30  Andreas Kling  <[email protected]>
+
+        Fix double hash lookup in RenderView::repaintSelection() loop.
+        <http://webkit.org/b/117029>
+
+        Reviewed by Anders Carlsson.
+
+        * rendering/RenderView.cpp:
+        (WebCore::RenderView::repaintSelection):
+
 2013-05-30  Peter Gal  <[email protected]>
 
         CURL: Incorrect stripping of query from file URL

Modified: trunk/Source/WebCore/rendering/RenderView.cpp (150965 => 150966)


--- trunk/Source/WebCore/rendering/RenderView.cpp	2013-05-30 16:44:32 UTC (rev 150965)
+++ trunk/Source/WebCore/rendering/RenderView.cpp	2013-05-30 17:03:53 UTC (rev 150966)
@@ -662,9 +662,8 @@
 
         // Blocks are responsible for painting line gaps and margin gaps. They must be examined as well.
         for (RenderBlock* block = o->containingBlock(); block && !block->isRenderView(); block = block->containingBlock()) {
-            if (processedBlocks.contains(block))
+            if (!processedBlocks.add(block).isNewEntry)
                 break;
-            processedBlocks.add(block);
             RenderSelectionInfo(block, true).repaint();
         }
     }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to