Make calls to AtomicString(const String&) explicit in svg/

Source/core/svg/SVGAltGlyphDefElement.cpp

 /*
  * Copyright (C) 2011 Leo Yang <leoyang@webkit.org>
  *
  * 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.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 21 matching lines @@
     : SVGElement(SVGNames::altGlyphDefTag, document)
 {
     ScriptWrappable::init(this);
 }
 
 PassRefPtr<SVGAltGlyphDefElement> SVGAltGlyphDefElement::create(Document& document)
 {
     return adoptRef(new SVGAltGlyphDefElement(document));
 }
 
-bool SVGAltGlyphDefElement::hasValidGlyphElements(Vector<String>& glyphNames) const
+bool SVGAltGlyphDefElement::hasValidGlyphElements(Vector<AtomicString>& glyphNames) const
 {
     // Spec: http://www.w3.org/TR/SVG/text.html#AltGlyphDefElement
     // An 'altGlyphDef' can contain either of the following:
     //
     // 1. In the simplest case, an 'altGlyphDef' contains one or more 'glyphRef' elements.
     // Each 'glyphRef' element references a single glyph within a particular font.
     // If all of the referenced glyphs are available, then these glyphs are rendered
     // instead of the character(s) inside of the referencing 'altGlyph' element.
     // If any of the referenced glyphs are unavailable, then the character(s) that are
     // inside of the 'altGlyph' element are rendered as if there were not an 'altGlyph'
@@ skipping 31 matching lines @@
     // For b), the content model is 2), so we will use <glyphRef> elements contained in
     // "i1" if they are valid and "g1" and "g2" are skipped.
 
     // These 2 variables are used to determine content model.
     bool fountFirstGlyphRef = false;
     bool foundFirstAltGlyphItem = false;
 
     for (Node* child = firstChild(); child; child = child->nextSibling()) {
         if (!foundFirstAltGlyphItem && child->hasTagName(SVGNames::glyphRefTag)) {
             fountFirstGlyphRef = true;
-            String referredGlyphName;
+            AtomicString referredGlyphName;
 
             if (toSVGGlyphRefElement(child)->hasValidGlyphElement(referredGlyphName))
                 glyphNames.append(referredGlyphName);
             else {
                 // As the spec says "If any of the referenced glyphs are unavailable,
                 // then the character(s) that are inside of the 'altGlyph' element are
                 // rendered as if there were not an 'altGlyph' element surrounding
                 // those characters.".
                 glyphNames.clear();
                 return false;
             }
         } else if (!fountFirstGlyphRef && child->hasTagName(SVGNames::altGlyphItemTag)) {
             foundFirstAltGlyphItem = true;
-            Vector<String> referredGlyphNames;
+            Vector<AtomicString> referredGlyphNames;
 
             // As the spec says "The first 'altGlyphItem' in which all referenced glyphs
             // are available is chosen."
             if (static_cast<SVGAltGlyphItemElement*>(child)->hasValidGlyphElements(glyphNames) && !glyphNames.isEmpty())
                 return true;
         }
     }
     return !glyphNames.isEmpty();
 }
 
 }
 
 #endif