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

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

Issue 2458823002: Don't establish LayoutState for LayoutTableRow objects. (Closed)
Patch Set: Created 4 years, 1 month 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, 2010 Apple Inc. 6 * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc.
7 * All rights reserved. 7 * All rights reserved.
8 * Copyright (C) 2013 Adobe Systems Incorporated. All rights reserved. 8 * Copyright (C) 2013 Adobe Systems Incorporated. All rights reserved.
9 * 9 *
10 * This library is free software; you can redistribute it and/or 10 * This library is free software; you can redistribute it and/or
(...skipping 447 matching lines...) Expand 10 before | Expand all | Expand 10 after
458 void LayoutBox::layout() { 458 void LayoutBox::layout() {
459 ASSERT(needsLayout()); 459 ASSERT(needsLayout());
460 LayoutAnalyzer::Scope analyzer(*this); 460 LayoutAnalyzer::Scope analyzer(*this);
461 461
462 LayoutObject* child = slowFirstChild(); 462 LayoutObject* child = slowFirstChild();
463 if (!child) { 463 if (!child) {
464 clearNeedsLayout(); 464 clearNeedsLayout();
465 return; 465 return;
466 } 466 }
467 467
468 LayoutState state(*this, locationOffset()); 468 LayoutState state(*this);
469 while (child) { 469 while (child) {
470 child->layoutIfNeeded(); 470 child->layoutIfNeeded();
471 ASSERT(!child->needsLayout()); 471 ASSERT(!child->needsLayout());
472 child = child->nextSibling(); 472 child = child->nextSibling();
473 } 473 }
474 invalidateBackgroundObscurationStatus(); 474 invalidateBackgroundObscurationStatus();
475 clearNeedsLayout(); 475 clearNeedsLayout();
476 } 476 }
477 477
478 // More IE extensions. clientWidth and clientHeight represent the interior of 478 // More IE extensions. clientWidth and clientHeight represent the interior of
(...skipping 5116 matching lines...) Expand 10 before | Expand all | Expand 10 after
5595 LayoutRect rect = frameRect(); 5595 LayoutRect rect = frameRect();
5596 5596
5597 LayoutBlock* block = containingBlock(); 5597 LayoutBlock* block = containingBlock();
5598 if (block) 5598 if (block)
5599 block->adjustChildDebugRect(rect); 5599 block->adjustChildDebugRect(rect);
5600 5600
5601 return rect; 5601 return rect;
5602 } 5602 }
5603 5603
5604 } // namespace blink 5604 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698