Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1616)

Unified Diff: Source/core/plugins/PluginOcclusionSupport.cpp

Issue 603193005: Move the Widget hierarchy to the Oilpan heap. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Rebased upto r183571 Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/plugins/PluginOcclusionSupport.cpp
diff --git a/Source/core/plugins/PluginOcclusionSupport.cpp b/Source/core/plugins/PluginOcclusionSupport.cpp
index 6f47f80ee8f82d7f695f7eef5da072704e6815af..2e7c0b8b86aded2b00de57c990c61921c1dcc2f8 100644
--- a/Source/core/plugins/PluginOcclusionSupport.cpp
+++ b/Source/core/plugins/PluginOcclusionSupport.cpp
@@ -164,13 +164,13 @@ void getPluginOcclusions(Element* element, Widget* parentWidget, const IntRect&
FrameView* parentFrameView = toFrameView(parentWidget);
// Occlusions by iframes.
- const HashSet<RefPtr<Widget> >* children = parentFrameView->children();
- for (HashSet<RefPtr<Widget> >::const_iterator it = children->begin(); it != children->end(); ++it) {
+ const WillBeHeapHashSet<RefPtrWillBeMember<Widget> >* children = parentFrameView->children();
haraken 2014/10/11 17:33:03 ChildrenWidgetSet
sof 2014/10/12 08:16:22 Done.
+ for (WillBeHeapHashSet<RefPtrWillBeMember<Widget> >::const_iterator it = children->begin(); it != children->end(); ++it) {
// We only care about FrameView's because iframes show up as FrameViews.
if (!(*it)->isFrameView())
continue;
- const FrameView* frameView = toFrameView((*it).get());
+ const FrameView* frameView = toFrameView(it->get());
// Check to make sure we can get both the element and the RenderObject
// for this FrameView, if we can't just move on to the next object.
// FIXME: Plugin occlusion by remote frames is probably broken.

Powered by Google App Engine
This is Rietveld 408576698