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

Side by Side Diff: third_party/WebKit/Source/core/layout/LayoutBoxModelObject.cpp

Issue 2671853003: [SPInvalidation] Use GeometryMapper in PaintLayerClipper for paint. (Closed)
Patch Set: Merge branch 'master' into paintlayerclipper Created 3 years, 10 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 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) 2005 Allan Sandfeld Jensen (kde@carewolf.com) 4 * (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
5 * (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com) 5 * (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com)
6 * Copyright (C) 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved. 6 * Copyright (C) 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
7 * Copyright (C) 2010 Google Inc. All rights reserved. 7 * Copyright (C) 2010 Google Inc. All rights reserved.
8 * 8 *
9 * This library is free software; you can redistribute it and/or 9 * This library is free software; you can redistribute it and/or
10 * modify it under the terms of the GNU Library General Public 10 * modify it under the terms of the GNU Library General Public
(...skipping 214 matching lines...) Expand 10 before | Expand all | Expand 10 after
225 // The following disablers are valid because we need to invalidate based on 225 // The following disablers are valid because we need to invalidate based on
226 // the current status. 226 // the current status.
227 DisableCompositingQueryAsserts compositingDisabler; 227 DisableCompositingQueryAsserts compositingDisabler;
228 DisablePaintInvalidationStateAsserts paintDisabler; 228 DisablePaintInvalidationStateAsserts paintDisabler;
229 ObjectPaintInvalidator(*this) 229 ObjectPaintInvalidator(*this)
230 .invalidatePaintIncludingNonCompositingDescendants(); 230 .invalidatePaintIncludingNonCompositingDescendants();
231 } 231 }
232 232
233 FloatStateForStyleChange::setWasFloating(this, isFloating()); 233 FloatStateForStyleChange::setWasFloating(this, isFloating());
234 234
235 if (hasLayer() && diff.cssClipChanged()) 235 if (hasLayer() && diff.cssClipChanged()) {
236 layer()->clipper().clearClipRectsIncludingDescendants(); 236 layer()
237 ->clipper(PaintLayer::DoNotUseGeometryMapper)
238 .clearClipRectsIncludingDescendants();
239 }
237 240
238 LayoutObject::styleWillChange(diff, newStyle); 241 LayoutObject::styleWillChange(diff, newStyle);
239 } 242 }
240 243
241 DISABLE_CFI_PERF 244 DISABLE_CFI_PERF
242 void LayoutBoxModelObject::styleDidChange(StyleDifference diff, 245 void LayoutBoxModelObject::styleDidChange(StyleDifference diff,
243 const ComputedStyle* oldStyle) { 246 const ComputedStyle* oldStyle) {
244 bool hadTransformRelatedProperty = hasTransformRelatedProperty(); 247 bool hadTransformRelatedProperty = hasTransformRelatedProperty();
245 bool hadLayer = hasLayer(); 248 bool hadLayer = hasLayer();
246 bool layerWasSelfPainting = hadLayer && layer()->isSelfPaintingLayer(); 249 bool layerWasSelfPainting = hadLayer && layer()->isSelfPaintingLayer();
(...skipping 1100 matching lines...) Expand 10 before | Expand all | Expand 10 after
1347 if (rootElementStyle->hasBackground()) 1350 if (rootElementStyle->hasBackground())
1348 return false; 1351 return false;
1349 1352
1350 if (node() != document().firstBodyElement()) 1353 if (node() != document().firstBodyElement())
1351 return false; 1354 return false;
1352 1355
1353 return true; 1356 return true;
1354 } 1357 }
1355 1358
1356 } // namespace blink 1359 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/frame/FrameView.cpp ('k') | third_party/WebKit/Source/core/layout/LayoutTreeAsText.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698