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

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

Issue 2541763002: Invalidate previousLocationInBacking in LayoutObject::clearPreviousVisualRects() (Closed)
Patch Set: Created 4 years 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) 2000 Dirk Mueller (mueller@kde.org) 4 * (C) 2000 Dirk Mueller (mueller@kde.org)
5 * (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com) 5 * (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com)
6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2011 Apple Inc. 6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2011 Apple Inc.
7 * All rights reserved. 7 * All rights reserved.
8 * Copyright (C) 2009 Google Inc. All rights reserved. 8 * Copyright (C) 2009 Google Inc. All rights reserved.
9 * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. 9 * Copyright (C) 2009 Torch Mobile Inc. All rights reserved.
10 * (http://www.torchmobile.com/) 10 * (http://www.torchmobile.com/)
(...skipping 1240 matching lines...) Expand 10 before | Expand all | Expand 10 after
1251 1251
1252 void LayoutObject::adjustPreviousPaintInvalidationForScrollIfNeeded( 1252 void LayoutObject::adjustPreviousPaintInvalidationForScrollIfNeeded(
1253 const DoubleSize& scrollDelta) { 1253 const DoubleSize& scrollDelta) {
1254 if (containerForPaintInvalidation().usesCompositedScrolling()) 1254 if (containerForPaintInvalidation().usesCompositedScrolling())
1255 return; 1255 return;
1256 m_previousVisualRect.move(LayoutSize(scrollDelta)); 1256 m_previousVisualRect.move(LayoutSize(scrollDelta));
1257 } 1257 }
1258 1258
1259 void LayoutObject::clearPreviousVisualRects() { 1259 void LayoutObject::clearPreviousVisualRects() {
1260 setPreviousVisualRect(LayoutRect()); 1260 setPreviousVisualRect(LayoutRect());
1261 // After clearing ("invalidating" the visual rects, mark this object as 1261 ObjectPaintInvalidator(*this).setPreviousLocationInBacking(LayoutPoint());
1262 // Ensure check paint invalidation of subtree that would be triggered by
1263 // location change if we had valid previous location.
1264 setMayNeedPaintInvalidationSubtree();
1265 // After clearing ("invalidating") the visual rects, mark this object as
1262 // needing to re-compute them. 1266 // needing to re-compute them.
1263 setShouldDoFullPaintInvalidation(); 1267 setShouldDoFullPaintInvalidation();
1264 } 1268 }
1265 1269
1266 LayoutRect LayoutObject::absoluteVisualRect() const { 1270 LayoutRect LayoutObject::absoluteVisualRect() const {
1267 LayoutRect rect = localVisualRect(); 1271 LayoutRect rect = localVisualRect();
1268 mapToVisualRectInAncestorSpace(view(), rect); 1272 mapToVisualRectInAncestorSpace(view(), rect);
1269 return rect; 1273 return rect;
1270 } 1274 }
1271 1275
(...skipping 2275 matching lines...) Expand 10 before | Expand all | Expand 10 after
3547 const blink::LayoutObject* root = object1; 3551 const blink::LayoutObject* root = object1;
3548 while (root->parent()) 3552 while (root->parent())
3549 root = root->parent(); 3553 root = root->parent();
3550 root->showLayoutTreeAndMark(object1, "*", object2, "-", 0); 3554 root->showLayoutTreeAndMark(object1, "*", object2, "-", 0);
3551 } else { 3555 } else {
3552 WTFLogAlways("%s", "Cannot showLayoutTree. Root is (nil)"); 3556 WTFLogAlways("%s", "Cannot showLayoutTree. Root is (nil)");
3553 } 3557 }
3554 } 3558 }
3555 3559
3556 #endif 3560 #endif
OLDNEW
« no previous file with comments | « third_party/WebKit/LayoutTests/paint/invalidation/compositing/become-composited-zero-offset-expected.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698