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

Unified Diff: third_party/WebKit/Source/web/WebViewImpl.cpp

Issue 2802503002: Start out deferring commits in WebViewImpl (Closed)
Patch Set: Merge branch 'master' into defercommits Created 3 years, 8 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/web/WebViewImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebViewImpl.cpp b/third_party/WebKit/Source/web/WebViewImpl.cpp
index 769863756ac14764bdeb425d5473cd465f638f14..d94fbad13531033080ee1dde04113b0a3ae5c276 100644
--- a/third_party/WebKit/Source/web/WebViewImpl.cpp
+++ b/third_party/WebKit/Source/web/WebViewImpl.cpp
@@ -3998,8 +3998,13 @@ void WebViewImpl::InitializeLayerTreeView() {
dev_tools->LayerTreeViewChanged(layer_tree_view_);
page_->GetSettings().SetAcceleratedCompositingEnabled(layer_tree_view_);
- if (layer_tree_view_)
+ if (layer_tree_view_) {
page_->LayerTreeViewInitialized(*layer_tree_view_, nullptr);
+ // We don't yet have a page loaded at this point of the initialization of
+ // WebViewImpl, so don't allow cc to commit any frames Blink might
+ // try to create in the meantime.
+ layer_tree_view_->SetDeferCommits(true);
+ }
// FIXME: only unittests, click to play, Android printing, and printing (for
// headers and footers) make this assert necessary. We should make them not
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698