Removed deprecated method CSSValueList::itemWithBoundsCheck()

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

 /*
  * (C) 1999-2003 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2011 Research In Motion Limited. All rights reserved.
  * Copyright (C) 2013 Intel Corporation. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
@@ skipping 681 matching lines @@
         bool useSingleWordShorthand = false;
         bool foundPositionXCSSProperty = false;
         bool foundPositionYCSSProperty = false;
 
         for (unsigned propertyIndex = 0; propertyIndex < size; propertyIndex++) {
             const CSSValue* value = nullptr;
             CSSPropertyID property = shorthand.properties()[propertyIndex];
 
             // 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.
                 value = values[propertyIndex];
             }
             // No point proceeding if there's not a value to look at.
             if (!value)
                 continue;
 
             // Special case for background-repeat.
@@ skipping 165 matching lines @@
             builder.append(", ");
 
         const CSSValue* xValue = repeatXList ? repeatXList->item(i % repeatXList->length()) : repeatX;
         const CSSValue* yValue = repeatYList ? repeatYList->item(i % repeatYList->length()) : repeatY;
         appendBackgroundRepeatValue(builder, *xValue, *yValue);
     }
     return builder.toString();
 }
 
 } // namespace blink