Removed deprecated method CSSValueList::itemWithBoundsCheck()

third_party/WebKit/Source/core/css/StylePropertySerializer.cpp

Index: third_party/WebKit/Source/core/css/StylePropertySerializer.cpp
diff --git a/third_party/WebKit/Source/core/css/StylePropertySerializer.cpp b/third_party/WebKit/Source/core/css/StylePropertySerializer.cpp
index c7ffb42f74de93c23f30f1f47303fb596e93a064..e6db89f59e80766aff368cc6307ff931d8ecf711 100644
--- a/third_party/WebKit/Source/core/css/StylePropertySerializer.cpp
+++ b/third_party/WebKit/Source/core/css/StylePropertySerializer.cpp
@@ -699,8 +699,10 @@ String StylePropertySerializer::getLayeredShorthandValue(const StylePropertyShor
 
             // Get a CSSValue for this property and layer.
             if (values[propertyIndex]->isBaseValueList()) {
+                const CSSValueList* propertyValues = toCSSValueList(values[propertyIndex]);
                 // Might return 0 if there is not an item for this layer for this property.

[Timothy Loh, 2016/06/06 03:11:06]

comment needs to eb updated

[sashab, 2016/06/06 03:36:51]

Yeah I saw this comment but not sure what to update it to... Changed to 'There
may not be an item for this layer for this property'.

-                value = toCSSValueList(values[propertyIndex])->itemWithBoundsCheck(layer);
+                if (layer < propertyValues->length())
+                    value = propertyValues->item(layer);
             } else if (layer == 0 || (layer != numLayers - 1 && property == CSSPropertyBackgroundColor)) {
                 // Singletons except background color belong in the 0th layer.
                 // Background color belongs in the last layer.