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

Side by Side Diff: third_party/WebKit/Source/core/layout/LayoutDeprecatedFlexibleBox.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 * This file is part of the layout object implementation for KHTML. 2 * This file is part of the layout object implementation for KHTML.
3 * 3 *
4 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 4 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
5 * (C) 1999 Antti Koivisto (koivisto@kde.org) 5 * (C) 1999 Antti Koivisto (koivisto@kde.org)
6 * Copyright (C) 2003 Apple Computer, Inc. 6 * Copyright (C) 2003 Apple Computer, Inc.
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 385 matching lines...) Expand 10 before | Expand all | Expand 10 after
396 UpdateLogicalHeight(); 396 UpdateLogicalHeight();
397 397
398 if (previous_size.Height() != Size().Height()) 398 if (previous_size.Height() != Size().Height())
399 relayout_children = true; 399 relayout_children = true;
400 400
401 LayoutPositionedObjects(relayout_children || IsDocumentElement()); 401 LayoutPositionedObjects(relayout_children || IsDocumentElement());
402 402
403 ComputeOverflow(old_client_after_edge); 403 ComputeOverflow(old_client_after_edge);
404 } 404 }
405 405
406 UpdateLayerTransformAfterLayout();
407 UpdateAfterLayout(); 406 UpdateAfterLayout();
408 407
409 ClearNeedsLayout(); 408 ClearNeedsLayout();
410 } 409 }
411 410
412 // The first walk over our kids is to find out if we have any flexible children. 411 // The first walk over our kids is to find out if we have any flexible children.
413 static void GatherFlexChildrenInfo(FlexBoxIterator& iterator, 412 static void GatherFlexChildrenInfo(FlexBoxIterator& iterator,
414 bool relayout_children, 413 bool relayout_children,
415 unsigned& highest_flex_group, 414 unsigned& highest_flex_group,
416 unsigned& lowest_flex_group, 415 unsigned& lowest_flex_group,
(...skipping 860 matching lines...) Expand 10 before | Expand all | Expand 10 after
1277 if (min_height.IsFixed() || min_height.IsAuto()) { 1276 if (min_height.IsFixed() || min_height.IsAuto()) {
1278 LayoutUnit min_height(child->Style()->MinHeight().Value()); 1277 LayoutUnit min_height(child->Style()->MinHeight().Value());
1279 LayoutUnit height = ContentHeightForChild(child); 1278 LayoutUnit height = ContentHeightForChild(child);
1280 LayoutUnit allowed_shrinkage = (min_height - height).ClampPositiveToZero(); 1279 LayoutUnit allowed_shrinkage = (min_height - height).ClampPositiveToZero();
1281 return allowed_shrinkage; 1280 return allowed_shrinkage;
1282 } 1281 }
1283 return LayoutUnit(); 1282 return LayoutUnit();
1284 } 1283 }
1285 1284
1286 } // namespace blink 1285 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698