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

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

Issue 196533012: Make LayoutState always be RAII (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix FlexBox LayoutState scoping Created 6 years, 9 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 | Annotate | Revision Log
OLDNEW
1 /** 1 /**
2 * Copyright (C) 1997 Martin Jones (mjones@kde.org) 2 * Copyright (C) 1997 Martin Jones (mjones@kde.org)
3 * (C) 1997 Torben Weis (weis@kde.org) 3 * (C) 1997 Torben Weis (weis@kde.org)
4 * (C) 1998 Waldo Bastian (bastian@kde.org) 4 * (C) 1998 Waldo Bastian (bastian@kde.org)
5 * (C) 1999 Lars Knoll (knoll@kde.org) 5 * (C) 1999 Lars Knoll (knoll@kde.org)
6 * (C) 1999 Antti Koivisto (koivisto@kde.org) 6 * (C) 1999 Antti Koivisto (koivisto@kde.org)
7 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All r ights reserved. 7 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All r ights 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 178 matching lines...) Expand 10 before | Expand all | Expand 10 after
189 if (RuntimeEnabledFeatures::repaintAfterLayoutEnabled()) { 189 if (RuntimeEnabledFeatures::repaintAfterLayoutEnabled()) {
190 // FIXME: Is this needed with repaint After Layout? 190 // FIXME: Is this needed with repaint After Layout?
191 child->setShouldDoFullRepaintAfterLayout(true); 191 child->setShouldDoFullRepaintAfterLayout(true);
192 } else { 192 } else {
193 child->repaint(); 193 child->repaint();
194 } 194 }
195 } 195 }
196 } 196 }
197 } 197 }
198 198
199 statePusher.pop();
200 // RenderTableSection::layoutRows will set our logical height and width late r, so it calls updateLayerTransform(). 199 // RenderTableSection::layoutRows will set our logical height and width late r, so it calls updateLayerTransform().
201 clearNeedsLayout(); 200 clearNeedsLayout();
202 } 201 }
203 202
204 LayoutRect RenderTableRow::clippedOverflowRectForRepaint(const RenderLayerModelO bject* repaintContainer) const 203 LayoutRect RenderTableRow::clippedOverflowRectForRepaint(const RenderLayerModelO bject* repaintContainer) const
205 { 204 {
206 ASSERT(parent()); 205 ASSERT(parent());
207 206
208 if (repaintContainer == this) 207 if (repaintContainer == this)
209 return RenderBox::clippedOverflowRectForRepaint(repaintContainer); 208 return RenderBox::clippedOverflowRectForRepaint(repaintContainer);
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
282 281
283 RenderTableRow* RenderTableRow::createAnonymousWithParentRenderer(const RenderOb ject* parent) 282 RenderTableRow* RenderTableRow::createAnonymousWithParentRenderer(const RenderOb ject* parent)
284 { 283 {
285 RenderTableRow* newRow = RenderTableRow::createAnonymous(&parent->document() ); 284 RenderTableRow* newRow = RenderTableRow::createAnonymous(&parent->document() );
286 RefPtr<RenderStyle> newStyle = RenderStyle::createAnonymousStyleWithDisplay( parent->style(), TABLE_ROW); 285 RefPtr<RenderStyle> newStyle = RenderStyle::createAnonymousStyleWithDisplay( parent->style(), TABLE_ROW);
287 newRow->setStyle(newStyle.release()); 286 newRow->setStyle(newStyle.release());
288 return newRow; 287 return newRow;
289 } 288 }
290 289
291 } // namespace WebCore 290 } // namespace WebCore
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698