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

Unified Diff: third_party/WebKit/Source/core/css/resolver/StyleBuilderConverter.h

Issue 2399633003: reflow comments in core/css/resolver (Closed)
Patch Set: Created 4 years, 2 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/css/resolver/StyleBuilderConverter.h
diff --git a/third_party/WebKit/Source/core/css/resolver/StyleBuilderConverter.h b/third_party/WebKit/Source/core/css/resolver/StyleBuilderConverter.h
index 77b8d41de168230eaf675d16cdf568f466a075a7..ba1d289375506c1efc9a7ef29955cb0e27821b7a 100644
--- a/third_party/WebKit/Source/core/css/resolver/StyleBuilderConverter.h
+++ b/third_party/WebKit/Source/core/css/resolver/StyleBuilderConverter.h
@@ -223,10 +223,11 @@ T StyleBuilderConverter::convertLineWidth(StyleResolverState& state,
return 0;
}
const CSSPrimitiveValue& primitiveValue = toCSSPrimitiveValue(value);
- // FIXME: We are moving to use the full page zoom implementation to handle high-dpi.
- // In that case specyfing a border-width of less than 1px would result in a border that is one device pixel thick.
- // With this change that would instead be rounded up to 2 device pixels.
- // Consider clamping it to device pixels or zoom adjusted CSS pixels instead of raw CSS pixels.
+ // FIXME: We are moving to use the full page zoom implementation to handle
+ // high-dpi. In that case specyfing a border-width of less than 1px would
+ // result in a border that is one device pixel thick. With this change that
+ // would instead be rounded up to 2 device pixels. Consider clamping it to
+ // device pixels or zoom adjusted CSS pixels instead of raw CSS pixels.
// Reference crbug.com/485650 and crbug.com/382483
double result =
primitiveValue.computeLength<double>(state.cssToLengthConversionData());

Powered by Google App Engine
This is Rietveld 408576698