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

Side by Side Diff: third_party/WebKit/Source/core/dom/Document.cpp

Issue 2272773002: Use intersection observer to control frame throttling (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update LeakExpectations Created 4 years, 1 month 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * (C) 2001 Dirk Mueller (mueller@kde.org) 4 * (C) 2001 Dirk Mueller (mueller@kde.org)
5 * (C) 2006 Alexey Proskuryakov (ap@webkit.org) 5 * (C) 2006 Alexey Proskuryakov (ap@webkit.org)
6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All 6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All
7 * rights reserved. 7 * rights reserved.
8 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. 8 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
9 * (http://www.torchmobile.com/) 9 * (http://www.torchmobile.com/)
10 * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved. 10 * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved.
(...skipping 5195 matching lines...) Expand 10 before | Expand all | Expand 10 after
5206 if (!isRenderingReady()) 5206 if (!isRenderingReady())
5207 return; 5207 return;
5208 if (LocalFrame* frame = this->frame()) { 5208 if (LocalFrame* frame = this->frame()) {
5209 // Avoid pumping frames for the initially empty document. 5209 // Avoid pumping frames for the initially empty document.
5210 if (!frame->loader().stateMachine()->committedFirstRealDocumentLoad()) 5210 if (!frame->loader().stateMachine()->committedFirstRealDocumentLoad())
5211 return; 5211 return;
5212 // The compositor will "defer commits" for the main frame until we 5212 // The compositor will "defer commits" for the main frame until we
5213 // explicitly request them. 5213 // explicitly request them.
5214 if (frame->isMainFrame()) 5214 if (frame->isMainFrame())
5215 frame->page()->chromeClient().beginLifecycleUpdates(); 5215 frame->page()->chromeClient().beginLifecycleUpdates();
5216 if (frame->view())
5217 frame->view()->setupRenderThrottling();
esprehn 2016/11/05 03:01:11 Why do you need to call this here if you're also c
5216 } 5218 }
5217 } 5219 }
5218 5220
5219 Vector<IconURL> Document::iconURLs(int iconTypesMask) { 5221 Vector<IconURL> Document::iconURLs(int iconTypesMask) {
5220 IconURL firstFavicon; 5222 IconURL firstFavicon;
5221 IconURL firstTouchIcon; 5223 IconURL firstTouchIcon;
5222 IconURL firstTouchPrecomposedIcon; 5224 IconURL firstTouchPrecomposedIcon;
5223 Vector<IconURL> secondaryIcons; 5225 Vector<IconURL> secondaryIcons;
5224 5226
5225 // Start from the last child node so that icons seen later take precedence as 5227 // Start from the last child node so that icons seen later take precedence as
(...skipping 1236 matching lines...) Expand 10 before | Expand all | Expand 10 after
6462 } 6464 }
6463 6465
6464 void showLiveDocumentInstances() { 6466 void showLiveDocumentInstances() {
6465 WeakDocumentSet& set = liveDocumentSet(); 6467 WeakDocumentSet& set = liveDocumentSet();
6466 fprintf(stderr, "There are %u documents currently alive:\n", set.size()); 6468 fprintf(stderr, "There are %u documents currently alive:\n", set.size());
6467 for (Document* document : set) 6469 for (Document* document : set)
6468 fprintf(stderr, "- Document %p URL: %s\n", document, 6470 fprintf(stderr, "- Document %p URL: %s\n", document,
6469 document->url().getString().utf8().data()); 6471 document->url().getString().utf8().data());
6470 } 6472 }
6471 #endif 6473 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698