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

Unified Diff: Source/core/css/CSSGridTemplateAreasValue.cpp

Issue 166623002: [CSS Grid Layout] Introduce an explicit type for resolved grid positions (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 10 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: Source/core/css/CSSGridTemplateAreasValue.cpp
diff --git a/Source/core/css/CSSGridTemplateAreasValue.cpp b/Source/core/css/CSSGridTemplateAreasValue.cpp
index d27c5b6955fd4ce4d20af5196973efe18059dc9d..73d50e352720ed60623dae6aa6fd049a7da66dc4 100644
--- a/Source/core/css/CSSGridTemplateAreasValue.cpp
+++ b/Source/core/css/CSSGridTemplateAreasValue.cpp
@@ -52,14 +52,14 @@ static String stringForPosition(const NamedGridAreaMap& gridAreaMap, size_t row,
NamedGridAreaMap::const_iterator end = gridAreaMap.end();
for (NamedGridAreaMap::const_iterator it = gridAreaMap.begin(); it != end; ++it) {
const GridCoordinate& coordinate = it->value;
- if (row >= coordinate.rows.initialPositionIndex && row <= coordinate.rows.finalPositionIndex)
+ if (row >= coordinate.rows.initialPositionIndex.integerPosition() && row <= coordinate.rows.finalPositionIndex.integerPosition())
candidates.append(it->key);
}
end = gridAreaMap.end();
for (NamedGridAreaMap::const_iterator it = gridAreaMap.begin(); it != end; ++it) {
const GridCoordinate& coordinate = it->value;
- if (column >= coordinate.columns.initialPositionIndex && column <= coordinate.columns.finalPositionIndex && candidates.contains(it->key))
+ if (column >= coordinate.columns.initialPositionIndex.integerPosition() && column <= coordinate.columns.finalPositionIndex.integerPosition() && candidates.contains(it->key))
return it->key;
}
« no previous file with comments | « no previous file | Source/core/css/parser/BisonCSSParser-in.cpp » ('j') | Source/core/rendering/style/GridResolvedPosition.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698