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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutFlexibleBox.h

Issue 1639723003: [css-flexbox] Use correct aspect ratio for min-size: auto (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: more tests & a bugfix Created 4 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/layout/LayoutFlexibleBox.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutFlexibleBox.h b/third_party/WebKit/Source/core/layout/LayoutFlexibleBox.h
index a4948ca887e121316082056525893d4280ea4292..47aebb52027a72c1bca4fabf3f8d270e308c29c7 100644
--- a/third_party/WebKit/Source/core/layout/LayoutFlexibleBox.h
+++ b/third_party/WebKit/Source/core/layout/LayoutFlexibleBox.h
@@ -127,15 +127,19 @@ private:
LayoutUnit crossAxisScrollbarExtent() const;
LayoutUnit crossAxisScrollbarExtentForChild(const LayoutBox& child) const;
LayoutPoint flowAwareLocationForChild(const LayoutBox& child) const;
+ bool useChildAspectRatio(const LayoutBox& child) const;
+ LayoutUnit computeMainSizeFromAspectRatio(const LayoutBox& child, Length crossSizeLength) const;
void setFlowAwareLocationForChild(LayoutBox& child, const LayoutPoint&);
void adjustAlignmentForChild(LayoutBox& child, LayoutUnit);
ItemPosition alignmentForChild(const LayoutBox& child) const;
LayoutUnit mainAxisBorderAndPaddingExtentForChild(const LayoutBox& child) const;
LayoutUnit computeInnerFlexBaseSizeForChild(LayoutBox& child, ChildLayoutType = LayoutIfNeeded);
bool mainAxisLengthIsDefinite(const LayoutBox& child, const Length& flexBasis) const;
+ bool crossAxisLengthIsDefinite(const LayoutBox& child, const Length& flexBasis) const;
bool childFlexBaseSizeRequiresLayout(const LayoutBox& child) const;
bool needToStretchChildLogicalHeight(const LayoutBox& child) const;
bool childHasIntrinsicMainAxisSize(const LayoutBox& child) const;
+ bool childHasAspectRatio(const LayoutBox& child) const;
EOverflow mainAxisOverflowForChild(const LayoutBox& child) const;
EOverflow crossAxisOverflowForChild(const LayoutBox& child) const;

Powered by Google App Engine
This is Rietveld 408576698