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

Side by Side Diff: Source/core/rendering/RenderFrameSet.cpp

Issue 129853005: Fix checkForRepaintDuringLayout do return false when doing repaintAfterLayout. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink@master
Patch Set: Created 6 years, 11 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
« no previous file with comments | « Source/core/rendering/RenderFlexibleBox.cpp ('k') | Source/core/rendering/RenderObject.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /** 1 /**
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 2000 Simon Hausmann <hausmann@kde.org> 3 * (C) 2000 Simon Hausmann <hausmann@kde.org>
4 * (C) 2000 Stefan Schimanski (1Stein@gmx.de) 4 * (C) 2000 Stefan Schimanski (1Stein@gmx.de)
5 * Copyright (C) 2004, 2005, 2006 Apple Computer, Inc. 5 * Copyright (C) 2004, 2005, 2006 Apple Computer, Inc.
6 * 6 *
7 * This library is free software; you can redistribute it and/or 7 * This library is free software; you can redistribute it and/or
8 * modify it under the terms of the GNU Library General Public 8 * modify it under the terms of the GNU Library General Public
9 * License as published by the Free Software Foundation; either 9 * License as published by the Free Software Foundation; either
10 * version 2 of the License, or (at your option) any later version. 10 * version 2 of the License, or (at your option) any later version.
(...skipping 424 matching lines...) Expand 10 before | Expand all | Expand 10 after
435 } 435 }
436 436
437 return result; 437 return result;
438 } 438 }
439 439
440 void RenderFrameSet::layout() 440 void RenderFrameSet::layout()
441 { 441 {
442 ASSERT(needsLayout()); 442 ASSERT(needsLayout());
443 443
444 LayoutRectRecorder recorder(*this); 444 LayoutRectRecorder recorder(*this);
445 bool doFullRepaint = !RuntimeEnabledFeatures::repaintAfterLayoutEnabled() && selfNeedsLayout() && checkForRepaintDuringLayout(); 445 bool doFullRepaint = selfNeedsLayout() && checkForRepaintDuringLayout();
446 LayoutRect oldBounds; 446 LayoutRect oldBounds;
447 RenderLayerModelObject* repaintContainer = 0; 447 RenderLayerModelObject* repaintContainer = 0;
448 if (doFullRepaint) { 448 if (doFullRepaint) {
449 repaintContainer = containerForRepaint(); 449 repaintContainer = containerForRepaint();
450 oldBounds = clippedOverflowRectForRepaint(repaintContainer); 450 oldBounds = clippedOverflowRectForRepaint(repaintContainer);
451 } 451 }
452 452
453 if (!parent()->isFrameSet() && !document().printing()) { 453 if (!parent()->isFrameSet() && !document().printing()) {
454 setWidth(view()->viewWidth()); 454 setWidth(view()->viewWidth());
455 setHeight(view()->viewHeight()); 455 setHeight(view()->viewHeight());
(...skipping 218 matching lines...) Expand 10 before | Expand all | Expand 10 after
674 return SetCursor; 674 return SetCursor;
675 } 675 }
676 if (canResizeColumn(roundedPoint)) { 676 if (canResizeColumn(roundedPoint)) {
677 cursor = columnResizeCursor(); 677 cursor = columnResizeCursor();
678 return SetCursor; 678 return SetCursor;
679 } 679 }
680 return RenderBox::getCursor(point, cursor); 680 return RenderBox::getCursor(point, cursor);
681 } 681 }
682 682
683 } // namespace WebCore 683 } // namespace WebCore
OLDNEW
« no previous file with comments | « Source/core/rendering/RenderFlexibleBox.cpp ('k') | Source/core/rendering/RenderObject.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698