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

Unified Diff: Source/build/scripts/templates/StyleBuilderFunctions.cpp.tmpl

Issue 1226123008: CSSValue Immediates: Replace CSSPrimitiveValue usage with const references (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@cssvalue_patch_4_attempt_2
Patch Set: Rebase Created 5 years, 5 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/build/scripts/templates/StyleBuilderFunctions.cpp.tmpl
diff --git a/Source/build/scripts/templates/StyleBuilderFunctions.cpp.tmpl b/Source/build/scripts/templates/StyleBuilderFunctions.cpp.tmpl
index e3cb394e2959d9ce5f63215fd7eaecb803618db5..619df5e4c53a063661bd1237987de7e159957f42 100644
--- a/Source/build/scripts/templates/StyleBuilderFunctions.cpp.tmpl
+++ b/Source/build/scripts/templates/StyleBuilderFunctions.cpp.tmpl
@@ -23,7 +23,7 @@ void StyleBuilderFunctions::applyInitial{{property_id}}(StyleResolverState& stat
void StyleBuilderFunctions::applyInherit{{property_id}}(StyleResolverState& state)
{%- endmacro %}
{% macro declare_value_function(property_id) %}
-void StyleBuilderFunctions::applyValue{{property_id}}(StyleResolverState& state, CSSValue value)
+void StyleBuilderFunctions::applyValue{{property_id}}(StyleResolverState& state, const CSSValue& value)
{%- endmacro %}
{% macro set_value(property) %}
{% if property.svg %}
@@ -142,7 +142,7 @@ namespace blink {
if (!value.isPrimitiveValue())
return;
- CSSPrimitiveValue primitiveValue = toCSSPrimitiveValue(value);
+ const CSSPrimitiveValue& primitiveValue = toCSSPrimitiveValue(value);
if (primitiveValue.getValueID() == {{auto_identity}})
state.style()->{{auto_setter}}();
else
@@ -350,16 +350,16 @@ static bool borderImageLengthMatchesAllSides(const BorderImageLengthBox& borderI
int length = list.length();
for (int i = 0; i < length; ++i) {
- CSSValue currValue = list.item(i);
+ const CSSValue& currValue = list.item(i);
if (!currValue.isPrimitiveValue())
continue;
Pair* pair = toCSSPrimitiveValue(currValue).getPairValue();
- if (!pair || !pair->first() || !pair->second())
+ if (!pair)
continue;
- AtomicString identifier(toCSSPrimitiveValue(*pair->first()).getStringValue());
- int value = toCSSPrimitiveValue(*pair->second()).getIntValue();
+ AtomicString identifier(pair->first().getStringValue());
+ int value = pair->second().getIntValue();
CounterDirectives& directives = map.add(identifier, CounterDirectives()).storedValue->value;
{% if action == 'Reset' %}
directives.setResetValue(value);
@@ -513,22 +513,23 @@ static bool borderImageLengthMatchesAllSides(const BorderImageLengthBox& borderI
{{declare_value_function(property_id)}}
{
String url;
- if (value.isValueList()) {
- CSSValueList& list = toCSSValueList(value);
+ CSSValue realValue = value;
+ if (realValue.isValueList()) {
+ const CSSValueList& list = toCSSValueList(realValue);
ASSERT(list.length() > 1);
if (!list.item(0).isPrimitiveValue())
return;
- CSSPrimitiveValue primitiveValue = toCSSPrimitiveValue(list.item(0));
+ const CSSPrimitiveValue& primitiveValue = toCSSPrimitiveValue(list.item(0));
if (!primitiveValue.isURI())
return;
url = primitiveValue.getStringValue();
- value = list.item(1);
+ realValue = list.item(1);
}
- if (value.isPrimitiveValue()) {
- CSSPrimitiveValue primitiveValue = toCSSPrimitiveValue(value);
+ if (realValue.isPrimitiveValue()) {
+ const CSSPrimitiveValue& primitiveValue = toCSSPrimitiveValue(realValue);
Color color;
SVGPaintType paintType = SVG_PAINTTYPE_RGBCOLOR;
if (primitiveValue.getValueID() == CSSValueNone) {
@@ -540,7 +541,7 @@ static bool borderImageLengthMatchesAllSides(const BorderImageLengthBox& borderI
color = state.style()->color();
paintType = url.isEmpty() ? SVG_PAINTTYPE_CURRENTCOLOR : SVG_PAINTTYPE_URI_CURRENTCOLOR;
} else {
- color = StyleBuilderConverter::convertColor(state, value);
+ color = StyleBuilderConverter::convertColor(state, realValue);
paintType = url.isEmpty() ? SVG_PAINTTYPE_RGBCOLOR : SVG_PAINTTYPE_URI_RGBCOLOR;
}
{{set_value(property)}}(paintType, color, url,
« no previous file with comments | « Source/build/scripts/templates/StyleBuilder.cpp.tmpl ('k') | Source/build/scripts/templates/StyleBuilderFunctions.h.tmpl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698