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

Unified Diff: Source/core/rendering/RenderObject.cpp

Issue 233063004: Suppress layer creation for descendants of GPU-rasterized layers (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Address abarth's comments 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/rendering/RenderObject.cpp
diff --git a/Source/core/rendering/RenderObject.cpp b/Source/core/rendering/RenderObject.cpp
index 2ef143e69c780f09279da570d31648723a065ea4..fe241b5114a125e289a68530a5ff813e5c6e1a20 100644
--- a/Source/core/rendering/RenderObject.cpp
+++ b/Source/core/rendering/RenderObject.cpp
@@ -1848,6 +1848,8 @@ void RenderObject::handleDynamicFloatPositionChange()
StyleDifference RenderObject::adjustStyleDifference(StyleDifference diff, unsigned contextSensitiveProperties) const
{
+ // FIXME: The calls to hasDirectReasonsForCompositing are using state that may not be up to date.
+ DisableCompositingQueryAsserts disabler;
abarth-chromium 2014/04/17 17:29:51 This should be asking whether we have any style-de
ajuma 2014/04/17 19:24:44 Yes, switching to styleDeterminedReasons() does ca
// If transform changed, and the layer does not paint into its own separate backing, then we need to do a layout.
// FIXME: The comment above is what the code does, but it is technically not following spec. This means we will
// not to layout for 3d transforms, but we should be invoking a simplified relayout. Is it possible we are avoiding

Powered by Google App Engine
This is Rietveld 408576698