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

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

Issue 2816873002: Update PaintLayer size during layout, not after.
Patch Set: Speculatively remove call to UpdateScrollbars() Created 3 years, 7 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) 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, 2009 Apple Inc. All rights reserved. 5 * Copyright (C) 2004, 2005, 2006, 2009 Apple Inc. All rights reserved.
6 * Copyright (C) Research In Motion Limited 2011. All rights reserved. 6 * Copyright (C) Research In Motion Limited 2011. All rights reserved.
7 * 7 *
8 * This library is free software; you can redistribute it and/or 8 * This library is free software; you can redistribute it and/or
9 * modify it under the terms of the GNU Library General Public 9 * modify it under the terms of the GNU Library General Public
10 * License as published by the Free Software Foundation; either 10 * License as published by the Free Software Foundation; either
(...skipping 268 matching lines...) Expand 10 before | Expand all | Expand 10 after
279 if (Style()->Visibility() != EVisibility::kVisible) { 279 if (Style()->Visibility() != EVisibility::kVisible) {
280 frame_or_plugin->Hide(); 280 frame_or_plugin->Hide();
281 } else { 281 } else {
282 frame_or_plugin->Show(); 282 frame_or_plugin->Show();
283 } 283 }
284 } 284 }
285 285
286 void LayoutPart::UpdateLayout() { 286 void LayoutPart::UpdateLayout() {
287 DCHECK(NeedsLayout()); 287 DCHECK(NeedsLayout());
288 LayoutAnalyzer::Scope analyzer(*this); 288 LayoutAnalyzer::Scope analyzer(*this);
289 UpdateAfterLayout();
289 ClearNeedsLayout(); 290 ClearNeedsLayout();
290 } 291 }
291 292
292 void LayoutPart::Paint(const PaintInfo& paint_info, 293 void LayoutPart::Paint(const PaintInfo& paint_info,
293 const LayoutPoint& paint_offset) const { 294 const LayoutPoint& paint_offset) const {
294 PartPainter(*this).Paint(paint_info, paint_offset); 295 PartPainter(*this).Paint(paint_info, paint_offset);
295 } 296 }
296 297
297 void LayoutPart::PaintContents(const PaintInfo& paint_info, 298 void LayoutPart::PaintContents(const PaintInfo& paint_info,
298 const LayoutPoint& paint_offset) const { 299 const LayoutPoint& paint_offset) const {
(...skipping 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
419 LayoutReplaced::InvalidatePaintOfSubtreesIfNeeded(paint_invalidation_state); 420 LayoutReplaced::InvalidatePaintOfSubtreesIfNeeded(paint_invalidation_state);
420 } 421 }
421 422
422 bool LayoutPart::IsThrottledFrameView() const { 423 bool LayoutPart::IsThrottledFrameView() const {
423 if (FrameView* frame_view = ChildFrameView()) 424 if (FrameView* frame_view = ChildFrameView())
424 return frame_view->ShouldThrottleRendering(); 425 return frame_view->ShouldThrottleRendering();
425 return false; 426 return false;
426 } 427 }
427 428
428 } // namespace blink 429 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/layout/LayoutIFrame.cpp ('k') | third_party/WebKit/Source/core/layout/LayoutReplaced.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698