CSSValue Immediates: Replace CSSPrimitiveValue usage with const references

Source/core/css/FontFace.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 224 matching lines @@
         exceptionState->throwDOMException(SyntaxError, message);
     else
         setError(DOMException::create(SyntaxError, message));
 }
 
 bool FontFace::setPropertyFromStyle(const StylePropertySet& properties, CSSPropertyID propertyID)
 {
     return setPropertyValue(properties.getPropertyCSSValue(propertyID), propertyID);
 }
 
-bool FontFace::setPropertyValue(NullableCSSValue value, CSSPropertyID propertyID)
+bool FontFace::setPropertyValue(const NullableCSSValue& value, CSSPropertyID propertyID)
 {
     switch (propertyID) {
     case CSSPropertyFontStyle:
         m_style = value;
         break;
     case CSSPropertyFontWeight:
         m_weight = value;
         break;
     case CSSPropertyFontStretch:
         m_stretch = value;
@@ skipping 15 matching lines @@
     }
     return true;
 }
 
 bool FontFace::setFamilyValue(CSSValueList* familyList)
 {
     // The font-family descriptor has to have exactly one family name.
     if (familyList->length() != 1)
         return false;
 
-    CSSPrimitiveValue familyValue = toCSSPrimitiveValue(familyList->item(0));
+    const CSSPrimitiveValue& familyValue = toCSSPrimitiveValue(familyList->item(0));
     AtomicString family;
     if (familyValue.isCustomIdent()) {
         family = AtomicString(familyValue.getStringValue());
     } else if (familyValue.isValueID()) {
         // We need to use the raw text for all the generic family types, since @font-face is a way of actually
         // defining what font to use for those types.
         switch (familyValue.getValueID()) {
         case CSSValueSerif:
             family =  FontFamilyNames::webkit_serif;
             break;
@@ skipping 212 matching lines @@
         unsigned numRanges = rangeList->length();
         for (unsigned i = 0; i < numRanges; i++) {
             CSSUnicodeRangeValue& range = toCSSUnicodeRangeValue(rangeList->item(i));
             ranges.append(CSSFontFace::UnicodeRange(range.from(), range.to()));
         }
     }
 
     return adoptPtrWillBeNoop(new CSSFontFace(fontFace, ranges));
 }
 
-void FontFace::initCSSFontFace(Document* document, CSSValue src)
+void FontFace::initCSSFontFace(Document* document, const CSSValue& src)
 {
     m_cssFontFace = createCSSFontFace(this, m_unicodeRange);
     if (m_error)
         return;
 
     // Each item in the src property's list is a single CSSFontFaceSource. Put them all into a CSSFontFace.
     ASSERT(src.isValueList());
     CSSValueList& srcList = toCSSValueList(src);
     int srcLength = srcList.length();
 
@@ skipping 56 matching lines @@
 {
     return m_cssFontFace->hadBlankText();
 }
 
 bool FontFace::hasPendingActivity() const
 {
     return m_status == Loading && executionContext() && !executionContext()->activeDOMObjectsAreStopped();
 }
 
 } // namespace blink