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

Unified Diff: Source/core/frame/FrameView.cpp

Issue 235933002: Defer updateCompositingRequirementsState until the compositing update (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 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
Index: Source/core/frame/FrameView.cpp
diff --git a/Source/core/frame/FrameView.cpp b/Source/core/frame/FrameView.cpp
index e4b516cfe5339a4d857ed7f6b7ca924a3046f6ec..ea9a4d9aea8440b6faab8a993fa02c160c270201 100644
--- a/Source/core/frame/FrameView.cpp
+++ b/Source/core/frame/FrameView.cpp
@@ -623,13 +623,6 @@ void FrameView::updateCompositingLayersAfterStyleChange()
// This call will make sure the cached hasAcceleratedCompositing is updated from the pref
renderView->compositor()->cacheAcceleratedCompositingFlags();
-
- // Sometimes we will change a property (for example, z-index) that will not
- // cause a layout, but will require us to update compositing state. We only
- // need to do this if a layout is not already scheduled.
- if (!needsLayout())
- renderView->compositor()->updateCompositingRequirementsState();
-
renderView->compositor()->setNeedsCompositingUpdate(CompositingUpdateAfterStyleChange);
}
@@ -646,7 +639,6 @@ void FrameView::updateCompositingLayersAfterLayout()
// This call will make sure the cached hasAcceleratedCompositing is updated from the pref
renderView->compositor()->cacheAcceleratedCompositingFlags();
- renderView->compositor()->updateCompositingRequirementsState();
renderView->compositor()->setNeedsCompositingUpdate(CompositingUpdateAfterLayout);
}

Powered by Google App Engine
This is Rietveld 408576698