Rename Foo::addSubresourceStyleURLs and Node::addSubresourceAttributeURLs
authorantti@apple.com <antti@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 30 May 2012 12:09:54 +0000 (12:09 +0000)
committerantti@apple.com <antti@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 30 May 2012 12:09:54 +0000 (12:09 +0000)
https://bugs.webkit.org/show_bug.cgi?id=87739

Rubber-stamped by Andreas Kling.

Rename them to collectSubresourceURLs and collectSubresourceURLsFromAttributes to match
the naming conventions.

Move collectSubresourceURLsFromAttributes from Node to Element as only elements have attributes.

* css/CSSFontFaceSrcValue.cpp:
(WebCore::CSSFontFaceSrcValue::collectSubresourceURLs):
* css/CSSFontFaceSrcValue.h:
(CSSFontFaceSrcValue):
* css/CSSPrimitiveValue.cpp:
(WebCore::CSSPrimitiveValue::collectSubresourceURLs):
* css/CSSPrimitiveValue.h:
(CSSPrimitiveValue):
* css/CSSReflectValue.cpp:
(WebCore::CSSReflectValue::collectSubresourceURLs):
* css/CSSReflectValue.h:
(CSSReflectValue):
* css/CSSValue.cpp:
(WebCore::CSSValue::collectSubresourceURLs):
* css/CSSValue.h:
(CSSValue):
* css/CSSValueList.cpp:
(WebCore::CSSValueList::collectSubresourceURLs):
* css/CSSValueList.h:
(CSSValueList):
* css/StylePropertySet.cpp:
(WebCore::StylePropertySet::collectSubresourceURLs):
* css/StylePropertySet.h:
(StylePropertySet):
* css/StyleSheetContents.cpp:
(WebCore::StyleSheetContents::collectSubresourceURLs):
* css/StyleSheetContents.h:
(StyleSheetContents):
* dom/Element.h:
(WebCore::Element::collectSubresourceURLsFromAttributes):
(Element):
* dom/Node.cpp:
(WebCore::Node::getSubresourceURLs):
* dom/Node.h:
(WebCore):
* dom/ProcessingInstruction.cpp:
(WebCore::ProcessingInstruction::collectSubresourceURLsFromAttributes):
* dom/ProcessingInstruction.h:
(ProcessingInstruction):
* dom/StyledElement.cpp:
(WebCore::StyledElement::collectSubresourceURLsFromAttributes):
* dom/StyledElement.h:
(StyledElement):
* html/HTMLBodyElement.cpp:
(WebCore::HTMLBodyElement::collectSubresourceURLsFromAttributes):
* html/HTMLBodyElement.h:
(HTMLBodyElement):
* html/HTMLEmbedElement.cpp:
(WebCore::HTMLEmbedElement::collectSubresourceURLsFromAttributes):
* html/HTMLEmbedElement.h:
(HTMLEmbedElement):
* html/HTMLImageElement.cpp:
(WebCore::HTMLImageElement::collectSubresourceURLsFromAttributes):
* html/HTMLImageElement.h:
(HTMLImageElement):
* html/HTMLInputElement.cpp:
(WebCore::HTMLInputElement::collectSubresourceURLsFromAttributes):
* html/HTMLInputElement.h:
(HTMLInputElement):
* html/HTMLLinkElement.cpp:
(WebCore::HTMLLinkElement::collectSubresourceURLsFromAttributes):
* html/HTMLLinkElement.h:
(HTMLLinkElement):
* html/HTMLObjectElement.cpp:
(WebCore::HTMLObjectElement::collectSubresourceURLsFromAttributes):
* html/HTMLObjectElement.h:
(HTMLObjectElement):
* html/HTMLParamElement.cpp:
(WebCore::HTMLParamElement::collectSubresourceURLsFromAttributes):
* html/HTMLParamElement.h:
(HTMLParamElement):
* html/HTMLScriptElement.cpp:
(WebCore::HTMLScriptElement::collectSubresourceURLsFromAttributes):
* html/HTMLScriptElement.h:
(HTMLScriptElement):
* html/HTMLStyleElement.cpp:
(WebCore::HTMLStyleElement::collectSubresourceURLsFromAttributes):
* html/HTMLStyleElement.h:
(HTMLStyleElement):
* html/HTMLTableCellElement.cpp:
(WebCore::HTMLTableCellElement::collectSubresourceURLsFromAttributes):
* html/HTMLTableCellElement.h:
(HTMLTableCellElement):
* html/HTMLTableElement.cpp:
(WebCore::HTMLTableElement::collectSubresourceURLsFromAttributes):
* html/HTMLTableElement.h:
(HTMLTableElement):
* svg/SVGCursorElement.cpp:
(WebCore::SVGCursorElement::collectSubresourceURLsFromAttributes):
* svg/SVGCursorElement.h:
(SVGCursorElement):
* svg/SVGFEImageElement.cpp:
(WebCore::SVGFEImageElement::collectSubresourceURLsFromAttributes):
* svg/SVGFEImageElement.h:
(SVGFEImageElement):
* svg/SVGImageElement.cpp:
(WebCore::SVGImageElement::collectSubresourceURLsFromAttributes):
* svg/SVGImageElement.h:
(SVGImageElement):
* svg/SVGScriptElement.cpp:
(WebCore::SVGScriptElement::collectSubresourceURLsFromAttributes):
* svg/SVGScriptElement.h:
(SVGScriptElement):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@118913 268f45cc-cd09-0410-ab3c-d52691b4dbfc

52 files changed:
Source/WebCore/ChangeLog
Source/WebCore/css/CSSFontFaceSrcValue.cpp
Source/WebCore/css/CSSFontFaceSrcValue.h
Source/WebCore/css/CSSPrimitiveValue.cpp
Source/WebCore/css/CSSPrimitiveValue.h
Source/WebCore/css/CSSReflectValue.cpp
Source/WebCore/css/CSSReflectValue.h
Source/WebCore/css/CSSValue.cpp
Source/WebCore/css/CSSValue.h
Source/WebCore/css/CSSValueList.cpp
Source/WebCore/css/CSSValueList.h
Source/WebCore/css/StylePropertySet.cpp
Source/WebCore/css/StylePropertySet.h
Source/WebCore/css/StyleSheetContents.cpp
Source/WebCore/css/StyleSheetContents.h
Source/WebCore/dom/Element.h
Source/WebCore/dom/Node.cpp
Source/WebCore/dom/Node.h
Source/WebCore/dom/ProcessingInstruction.cpp
Source/WebCore/dom/ProcessingInstruction.h
Source/WebCore/dom/StyledElement.cpp
Source/WebCore/dom/StyledElement.h
Source/WebCore/html/HTMLBodyElement.cpp
Source/WebCore/html/HTMLBodyElement.h
Source/WebCore/html/HTMLEmbedElement.cpp
Source/WebCore/html/HTMLEmbedElement.h
Source/WebCore/html/HTMLImageElement.cpp
Source/WebCore/html/HTMLImageElement.h
Source/WebCore/html/HTMLInputElement.cpp
Source/WebCore/html/HTMLInputElement.h
Source/WebCore/html/HTMLLinkElement.cpp
Source/WebCore/html/HTMLLinkElement.h
Source/WebCore/html/HTMLObjectElement.cpp
Source/WebCore/html/HTMLObjectElement.h
Source/WebCore/html/HTMLParamElement.cpp
Source/WebCore/html/HTMLParamElement.h
Source/WebCore/html/HTMLScriptElement.cpp
Source/WebCore/html/HTMLScriptElement.h
Source/WebCore/html/HTMLStyleElement.cpp
Source/WebCore/html/HTMLStyleElement.h
Source/WebCore/html/HTMLTableCellElement.cpp
Source/WebCore/html/HTMLTableCellElement.h
Source/WebCore/html/HTMLTableElement.cpp
Source/WebCore/html/HTMLTableElement.h
Source/WebCore/svg/SVGCursorElement.cpp
Source/WebCore/svg/SVGCursorElement.h
Source/WebCore/svg/SVGFEImageElement.cpp
Source/WebCore/svg/SVGFEImageElement.h
Source/WebCore/svg/SVGImageElement.cpp
Source/WebCore/svg/SVGImageElement.h
Source/WebCore/svg/SVGScriptElement.cpp
Source/WebCore/svg/SVGScriptElement.h

index 0d4a7b8..950ade2 100644 (file)
@@ -1,3 +1,119 @@
+2012-05-29  Antti Koivisto  <antti@apple.com>
+
+        Rename Foo::addSubresourceStyleURLs and Node::addSubresourceAttributeURLs
+        https://bugs.webkit.org/show_bug.cgi?id=87739
+
+        Rubber-stamped by Andreas Kling.
+
+        Rename them to collectSubresourceURLs and collectSubresourceURLsFromAttributes to match
+        the naming conventions.
+        
+        Move collectSubresourceURLsFromAttributes from Node to Element as only elements have attributes.
+
+        * css/CSSFontFaceSrcValue.cpp:
+        (WebCore::CSSFontFaceSrcValue::collectSubresourceURLs):
+        * css/CSSFontFaceSrcValue.h:
+        (CSSFontFaceSrcValue):
+        * css/CSSPrimitiveValue.cpp:
+        (WebCore::CSSPrimitiveValue::collectSubresourceURLs):
+        * css/CSSPrimitiveValue.h:
+        (CSSPrimitiveValue):
+        * css/CSSReflectValue.cpp:
+        (WebCore::CSSReflectValue::collectSubresourceURLs):
+        * css/CSSReflectValue.h:
+        (CSSReflectValue):
+        * css/CSSValue.cpp:
+        (WebCore::CSSValue::collectSubresourceURLs):
+        * css/CSSValue.h:
+        (CSSValue):
+        * css/CSSValueList.cpp:
+        (WebCore::CSSValueList::collectSubresourceURLs):
+        * css/CSSValueList.h:
+        (CSSValueList):
+        * css/StylePropertySet.cpp:
+        (WebCore::StylePropertySet::collectSubresourceURLs):
+        * css/StylePropertySet.h:
+        (StylePropertySet):
+        * css/StyleSheetContents.cpp:
+        (WebCore::StyleSheetContents::collectSubresourceURLs):
+        * css/StyleSheetContents.h:
+        (StyleSheetContents):
+        * dom/Element.h:
+        (WebCore::Element::collectSubresourceURLsFromAttributes):
+        (Element):
+        * dom/Node.cpp:
+        (WebCore::Node::getSubresourceURLs):
+        * dom/Node.h:
+        (WebCore):
+        * dom/ProcessingInstruction.cpp:
+        (WebCore::ProcessingInstruction::collectSubresourceURLsFromAttributes):
+        * dom/ProcessingInstruction.h:
+        (ProcessingInstruction):
+        * dom/StyledElement.cpp:
+        (WebCore::StyledElement::collectSubresourceURLsFromAttributes):
+        * dom/StyledElement.h:
+        (StyledElement):
+        * html/HTMLBodyElement.cpp:
+        (WebCore::HTMLBodyElement::collectSubresourceURLsFromAttributes):
+        * html/HTMLBodyElement.h:
+        (HTMLBodyElement):
+        * html/HTMLEmbedElement.cpp:
+        (WebCore::HTMLEmbedElement::collectSubresourceURLsFromAttributes):
+        * html/HTMLEmbedElement.h:
+        (HTMLEmbedElement):
+        * html/HTMLImageElement.cpp:
+        (WebCore::HTMLImageElement::collectSubresourceURLsFromAttributes):
+        * html/HTMLImageElement.h:
+        (HTMLImageElement):
+        * html/HTMLInputElement.cpp:
+        (WebCore::HTMLInputElement::collectSubresourceURLsFromAttributes):
+        * html/HTMLInputElement.h:
+        (HTMLInputElement):
+        * html/HTMLLinkElement.cpp:
+        (WebCore::HTMLLinkElement::collectSubresourceURLsFromAttributes):
+        * html/HTMLLinkElement.h:
+        (HTMLLinkElement):
+        * html/HTMLObjectElement.cpp:
+        (WebCore::HTMLObjectElement::collectSubresourceURLsFromAttributes):
+        * html/HTMLObjectElement.h:
+        (HTMLObjectElement):
+        * html/HTMLParamElement.cpp:
+        (WebCore::HTMLParamElement::collectSubresourceURLsFromAttributes):
+        * html/HTMLParamElement.h:
+        (HTMLParamElement):
+        * html/HTMLScriptElement.cpp:
+        (WebCore::HTMLScriptElement::collectSubresourceURLsFromAttributes):
+        * html/HTMLScriptElement.h:
+        (HTMLScriptElement):
+        * html/HTMLStyleElement.cpp:
+        (WebCore::HTMLStyleElement::collectSubresourceURLsFromAttributes):
+        * html/HTMLStyleElement.h:
+        (HTMLStyleElement):
+        * html/HTMLTableCellElement.cpp:
+        (WebCore::HTMLTableCellElement::collectSubresourceURLsFromAttributes):
+        * html/HTMLTableCellElement.h:
+        (HTMLTableCellElement):
+        * html/HTMLTableElement.cpp:
+        (WebCore::HTMLTableElement::collectSubresourceURLsFromAttributes):
+        * html/HTMLTableElement.h:
+        (HTMLTableElement):
+        * svg/SVGCursorElement.cpp:
+        (WebCore::SVGCursorElement::collectSubresourceURLsFromAttributes):
+        * svg/SVGCursorElement.h:
+        (SVGCursorElement):
+        * svg/SVGFEImageElement.cpp:
+        (WebCore::SVGFEImageElement::collectSubresourceURLsFromAttributes):
+        * svg/SVGFEImageElement.h:
+        (SVGFEImageElement):
+        * svg/SVGImageElement.cpp:
+        (WebCore::SVGImageElement::collectSubresourceURLsFromAttributes):
+        * svg/SVGImageElement.h:
+        (SVGImageElement):
+        * svg/SVGScriptElement.cpp:
+        (WebCore::SVGScriptElement::collectSubresourceURLsFromAttributes):
+        * svg/SVGScriptElement.h:
+        (SVGScriptElement):
+
 2012-05-30  Eugene Klyuchnikov  <eustas.bug@gmail.com>
 
         Web Inspector: Fix JS compiler warning
index 79ae0e7..40143f8 100644 (file)
@@ -73,7 +73,7 @@ String CSSFontFaceSrcValue::customCssText() const
     return result;
 }
 
-void CSSFontFaceSrcValue::addSubresourceStyleURLs(ListHashSet<KURL>& urls, const StyleSheetContents* styleSheet) const
+void CSSFontFaceSrcValue::collectSubresourceURLs(ListHashSet<KURL>& urls, const StyleSheetContents* styleSheet) const
 {
     if (!isLocal())
         addSubresourceURL(urls, styleSheet->completeURL(m_resource));
index 94607fe..0632f14 100644 (file)
@@ -65,7 +65,7 @@ public:
 
     String customCssText() const;
 
-    void addSubresourceStyleURLs(ListHashSet<KURL>&, const StyleSheetContents*) const;
+    void collectSubresourceURLs(ListHashSet<KURL>&, const StyleSheetContents*) const;
 
     CachedFont* cachedFont(Document*);
 
index 592494a..4640f2c 100644 (file)
@@ -1064,7 +1064,7 @@ String CSSPrimitiveValue::customCssText() const
     return text;
 }
 
-void CSSPrimitiveValue::addSubresourceStyleURLs(ListHashSet<KURL>& urls, const StyleSheetContents* styleSheet) const
+void CSSPrimitiveValue::collectSubresourceURLs(ListHashSet<KURL>& urls, const StyleSheetContents* styleSheet) const
 {
     if (m_primitiveUnitType == CSS_URI)
         addSubresourceURL(urls, styleSheet->completeURL(m_value.string));
index ccd187b..c7bf011 100644 (file)
@@ -283,7 +283,7 @@ public:
 
     bool isQuirkValue() { return m_isQuirkValue; }
 
-    void addSubresourceStyleURLs(ListHashSet<KURL>&, const StyleSheetContents*) const;
+    void collectSubresourceURLs(ListHashSet<KURL>&, const StyleSheetContents*) const;
 
     Length viewportPercentageLength();
     
index b36c3ad..50de108 100644 (file)
@@ -59,10 +59,10 @@ String CSSReflectValue::customCssText() const
     return result;
 }
 
-void CSSReflectValue::addSubresourceStyleURLs(ListHashSet<KURL>& urls, const StyleSheetContents* styleSheet) const
+void CSSReflectValue::collectSubresourceURLs(ListHashSet<KURL>& urls, const StyleSheetContents* styleSheet) const
 {
     if (m_mask)
-        m_mask->addSubresourceStyleURLs(urls, styleSheet);
+        m_mask->collectSubresourceURLs(urls, styleSheet);
 }
 
 } // namespace WebCore
index 5cfb20e..2720ba8 100644 (file)
@@ -49,7 +49,7 @@ public:
 
     String customCssText() const;
 
-    void addSubresourceStyleURLs(ListHashSet<KURL>&, const StyleSheetContents*) const;
+    void collectSubresourceURLs(ListHashSet<KURL>&, const StyleSheetContents*) const;
 
 private:
     CSSReflectValue(CSSReflectionDirection direction, PassRefPtr<CSSPrimitiveValue> offset, PassRefPtr<CSSValue> mask)
index f568cc5..4df8c56 100644 (file)
@@ -99,19 +99,19 @@ CSSValue::Type CSSValue::cssValueType() const
     return CSS_CUSTOM;
 }
 
-void CSSValue::addSubresourceStyleURLs(ListHashSet<KURL>& urls, const StyleSheetContents* styleSheet) const
+void CSSValue::collectSubresourceURLs(ListHashSet<KURL>& urls, const StyleSheetContents* styleSheet) const
 {
     // This should get called for internal instances only.
     ASSERT(!isCSSOMSafe());
 
     if (isPrimitiveValue())
-        static_cast<const CSSPrimitiveValue*>(this)->addSubresourceStyleURLs(urls, styleSheet);
+        static_cast<const CSSPrimitiveValue*>(this)->collectSubresourceURLs(urls, styleSheet);
     else if (isValueList())
-        static_cast<const CSSValueList*>(this)->addSubresourceStyleURLs(urls, styleSheet);
+        static_cast<const CSSValueList*>(this)->collectSubresourceURLs(urls, styleSheet);
     else if (classType() == FontFaceSrcClass)
-        static_cast<const CSSFontFaceSrcValue*>(this)->addSubresourceStyleURLs(urls, styleSheet);
+        static_cast<const CSSFontFaceSrcValue*>(this)->collectSubresourceURLs(urls, styleSheet);
     else if (classType() == ReflectClass)
-        static_cast<const CSSReflectValue*>(this)->addSubresourceStyleURLs(urls, styleSheet);
+        static_cast<const CSSReflectValue*>(this)->collectSubresourceURLs(urls, styleSheet);
 }
 
 String CSSValue::cssText() const
index fe56428..f05c843 100644 (file)
@@ -107,7 +107,7 @@ public:
 
     PassRefPtr<CSSValue> cloneForCSSOM() const;
 
-    void addSubresourceStyleURLs(ListHashSet<KURL>&, const StyleSheetContents*) const;
+    void collectSubresourceURLs(ListHashSet<KURL>&, const StyleSheetContents*) const;
 
 protected:
 
index 3ab7f94..1cc42c4 100644 (file)
@@ -136,11 +136,11 @@ String CSSValueList::customCssText() const
     return result.toString();
 }
 
-void CSSValueList::addSubresourceStyleURLs(ListHashSet<KURL>& urls, const StyleSheetContents* styleSheet) const
+void CSSValueList::collectSubresourceURLs(ListHashSet<KURL>& urls, const StyleSheetContents* styleSheet) const
 {
     size_t size = m_values.size();
     for (size_t i = 0; i < size; ++i)
-        m_values[i]->addSubresourceStyleURLs(urls, styleSheet);
+        m_values[i]->collectSubresourceURLs(urls, styleSheet);
 }
 
 CSSValueList::CSSValueList(const CSSValueList& cloneFrom)
index a4c59b4..cad6335 100644 (file)
@@ -60,7 +60,7 @@ public:
 
     String customCssText() const;
 
-    void addSubresourceStyleURLs(ListHashSet<KURL>&, const StyleSheetContents*) const;
+    void collectSubresourceURLs(ListHashSet<KURL>&, const StyleSheetContents*) const;
     
     PassRefPtr<CSSValueList> cloneForCSSOM() const;
 
index 786aec5..ebc81d2 100644 (file)
@@ -875,11 +875,11 @@ void StylePropertySet::merge(const StylePropertySet* other, bool argOverridesOnC
     }
 }
 
-void StylePropertySet::addSubresourceStyleURLs(ListHashSet<KURL>& urls, StyleSheetContents* contextStyleSheet) const
+void StylePropertySet::collectSubresourceURLs(ListHashSet<KURL>& urls, StyleSheetContents* contextStyleSheet) const
 {
     unsigned size = propertyCount();
     for (unsigned i = 0; i < size; ++i)
-        propertyAt(i).value()->addSubresourceStyleURLs(urls, contextStyleSheet);
+        propertyAt(i).value()->collectSubresourceURLs(urls, contextStyleSheet);
 }
 
 // This is the list of properties we want to copy in the copyBlockProperties() function.
index 31f6fb4..b4c01fc 100644 (file)
@@ -90,7 +90,7 @@ public:
     void setCSSParserMode(CSSParserMode);
     CSSParserMode cssParserMode() const { return static_cast<CSSParserMode>(m_cssParserMode); }
 
-    void addSubresourceStyleURLs(ListHashSet<KURL>&, StyleSheetContents* contextStyleSheet) const;
+    void collectSubresourceURLs(ListHashSet<KURL>&, StyleSheetContents* contextStyleSheet) const;
 
     PassRefPtr<StylePropertySet> copy() const;
 
index 970c828..bee210c 100644 (file)
@@ -384,7 +384,7 @@ KURL StyleSheetContents::completeURL(const String& url) const
     return CSSParser::completeURL(m_parserContext, url);
 }
 
-void StyleSheetContents::addSubresourceStyleURLs(ListHashSet<KURL>& urls)
+void StyleSheetContents::collectSubresourceURLs(ListHashSet<KURL>& urls)
 {
     Deque<StyleSheetContents*> styleSheetQueue;
     styleSheetQueue.append(this);
@@ -402,9 +402,9 @@ void StyleSheetContents::addSubresourceStyleURLs(ListHashSet<KURL>& urls)
         for (unsigned i = 0; i < styleSheet->m_childRules.size(); ++i) {
             StyleRuleBase* rule = styleSheet->m_childRules[i].get();
             if (rule->isStyleRule())
-                static_cast<StyleRule*>(rule)->properties()->addSubresourceStyleURLs(urls, this);
+                static_cast<StyleRule*>(rule)->properties()->collectSubresourceURLs(urls, this);
             else if (rule->isFontFaceRule())
-                static_cast<StyleRuleFontFace*>(rule)->properties()->addSubresourceStyleURLs(urls, this);
+                static_cast<StyleRuleFontFace*>(rule)->properties()->collectSubresourceURLs(urls, this);
         }
     }
 }
index 7df34b7..0325dff 100644 (file)
@@ -80,7 +80,7 @@ public:
     bool loadCompleted() const { return m_loadCompleted; }
 
     KURL completeURL(const String& url) const;
-    void addSubresourceStyleURLs(ListHashSet<KURL>&);
+    void collectSubresourceURLs(ListHashSet<KURL>&);
 
     void setIsUserStyleSheet(bool b) { m_isUserStyleSheet = b; }
     bool isUserStyleSheet() const { return m_isUserStyleSheet; }
index fa39a18..363152b 100644 (file)
@@ -418,6 +418,8 @@ public:
     IntSize savedLayerScrollOffset() const;
     void setSavedLayerScrollOffset(const IntSize&);
 
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const { }
+
 protected:
     Element(const QualifiedName& tagName, Document* document, ConstructionType type)
         : ContainerNode(document, type)
index 79450c5..d40ee0d 100644 (file)
@@ -2423,7 +2423,9 @@ bool NodeListsNodeData::isEmpty() const
 
 void Node::getSubresourceURLs(ListHashSet<KURL>& urls) const
 {
-    addSubresourceAttributeURLs(urls);
+    if (!isElementNode())
+        return;
+    static_cast<const Element*>(this)->collectSubresourceURLsFromAttributes(urls);
 }
 
 Node* Node::enclosingLinkEventParentOrSelf()
index d8c62ee..09ab113 100644 (file)
@@ -727,7 +727,6 @@ protected:
 
     virtual void didMoveToNewDocument(Document* oldDocument);
     
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const { }
     void setTabIndexExplicitly(short);
     void clearTabIndexExplicitly();
     
@@ -822,7 +821,7 @@ protected:
 #endif
 };
 
-// Used in Node::addSubresourceAttributeURLs() and in addSubresourceStyleURLs()
+// Used in Node::collectSubresourceURLsFromAttributes() and in collectSubresourceURLs()
 inline void addSubresourceURL(ListHashSet<KURL>& urls, const KURL& url)
 {
     if (!url.isNull())
index 0943643..62ce023 100644 (file)
@@ -280,7 +280,7 @@ int ProcessingInstruction::maxCharacterOffset() const
     return static_cast<int>(m_data.length());
 }
 
-void ProcessingInstruction::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void ProcessingInstruction::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
     if (!sheet())
         return;
index e331e21..459909e 100644 (file)
@@ -76,7 +76,7 @@ private:
     bool isLoading() const;
     virtual bool sheetLoaded();
 
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 
     void parseStyleSheet(const String& sheet);
 
index a2bed3f..8ad9305 100644 (file)
@@ -239,10 +239,10 @@ bool StyledElement::removeInlineStyleProperty(CSSPropertyID propertyID)
     return changes;
 }
 
-void StyledElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void StyledElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
     if (StylePropertySet* inlineStyle = attributeData() ? attributeData()->inlineStyle() : 0)
-        inlineStyle->addSubresourceStyleURLs(urls, document()->elementSheet()->contents());
+        inlineStyle->collectSubresourceURLs(urls, document()->elementSheet()->contents());
 }
 
 static inline bool attributeNameSort(const pair<AtomicStringImpl*, AtomicString>& p1, const pair<AtomicStringImpl*, AtomicString>& p2)
index 4736a98..f845051 100644 (file)
@@ -74,7 +74,7 @@ protected:
     void addPropertyToAttributeStyle(StylePropertySet*, CSSPropertyID, double value, CSSPrimitiveValue::UnitTypes);
     void addPropertyToAttributeStyle(StylePropertySet*, CSSPropertyID, const String& value);
 
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 
     // classAttributeChanged() exists to share code between
     // parseAttribute (called via setAttribute()) and
index 846e331..b98e7b6 100644 (file)
@@ -317,9 +317,9 @@ int HTMLBodyElement::scrollWidth()
     return view ? adjustForZoom(view->contentsWidth(), document) : 0;    
 }
 
-void HTMLBodyElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void HTMLBodyElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
-    HTMLElement::addSubresourceAttributeURLs(urls);
+    HTMLElement::collectSubresourceURLsFromAttributes(urls);
 
     addSubresourceURL(urls, document()->completeURL(getAttribute(backgroundAttr)));
 }
index 22ac19c..7a40c24 100644 (file)
@@ -90,7 +90,7 @@ private:
     virtual int scrollHeight();
     virtual int scrollWidth();
     
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 };
 
 } //namespace
index b9c6e41..256bcd7 100644 (file)
@@ -216,9 +216,9 @@ const QualifiedName& HTMLEmbedElement::imageSourceAttributeName() const
     return srcAttr;
 }
 
-void HTMLEmbedElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void HTMLEmbedElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
-    HTMLPlugInImageElement::addSubresourceAttributeURLs(urls);
+    HTMLPlugInImageElement::collectSubresourceURLsFromAttributes(urls);
 
     addSubresourceURL(urls, document()->completeURL(getAttribute(srcAttr)));
 }
index ba828a1..05259fc 100644 (file)
@@ -47,7 +47,7 @@ private:
 
     virtual void updateWidget(PluginCreationOption);
 
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 
     void parametersForPlugin(Vector<String>& paramNames, Vector<String>& paramValues);
 
index 012d9ec..2476686 100644 (file)
@@ -325,9 +325,9 @@ bool HTMLImageElement::complete() const
     return m_imageLoader.imageComplete();
 }
 
-void HTMLImageElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void HTMLImageElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
-    HTMLElement::addSubresourceAttributeURLs(urls);
+    HTMLElement::collectSubresourceURLsFromAttributes(urls);
 
     addSubresourceURL(urls, src());
     // FIXME: What about when the usemap attribute begins with "#"?
index 9c6ff5a..bda8810 100644 (file)
@@ -97,7 +97,7 @@ private:
 
     virtual bool draggable() const;
 
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 
     virtual InsertionNotificationRequest insertedInto(ContainerNode*) OVERRIDE;
     virtual void removedFrom(ContainerNode*) OVERRIDE;
index 3950cd0..43de81f 100644 (file)
@@ -1378,9 +1378,9 @@ void HTMLInputElement::didMoveToNewDocument(Document* oldDocument)
     HTMLTextFormControlElement::didMoveToNewDocument(oldDocument);
 }
 
-void HTMLInputElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void HTMLInputElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
-    HTMLTextFormControlElement::addSubresourceAttributeURLs(urls);
+    HTMLTextFormControlElement::collectSubresourceURLsFromAttributes(urls);
 
     addSubresourceURL(urls, src());
 }
index 56a8641..d63a83c 100644 (file)
@@ -316,7 +316,7 @@ private:
 
     virtual void documentDidResumeFromPageCache();
 
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 
     bool needsSuspensionCallback();
     void registerForSuspensionCallbackIfNeeded();
index f378de5..50075c0 100644 (file)
@@ -418,9 +418,9 @@ String HTMLLinkElement::type() const
     return getAttribute(typeAttr);
 }
 
-void HTMLLinkElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void HTMLLinkElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
-    HTMLElement::addSubresourceAttributeURLs(urls);
+    HTMLElement::collectSubresourceURLsFromAttributes(urls);
 
     // Favicons are handled by a special case in LegacyWebArchive::create()
     if (m_relAttribute.m_iconType != InvalidIcon)
@@ -434,7 +434,7 @@ void HTMLLinkElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
     
     // Walk the URLs linked by the linked-to stylesheet.
     if (CSSStyleSheet* styleSheet = const_cast<HTMLLinkElement*>(this)->sheet())
-        styleSheet->contents()->addSubresourceStyleURLs(urls);
+        styleSheet->contents()->collectSubresourceURLs(urls);
 }
 
 void HTMLLinkElement::addPendingSheet(PendingSheetType type)
index bb69e12..f72cfd2 100644 (file)
@@ -94,7 +94,7 @@ private:
     virtual bool isURLAttribute(const Attribute&) const OVERRIDE;
 
 private:
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 
     virtual void finishParsingChildren();
     
index 024e825..5096755 100644 (file)
@@ -467,9 +467,9 @@ bool HTMLObjectElement::containsJavaApplet() const
     return false;
 }
 
-void HTMLObjectElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void HTMLObjectElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
-    HTMLPlugInImageElement::addSubresourceAttributeURLs(urls);
+    HTMLPlugInImageElement::collectSubresourceURLsFromAttributes(urls);
 
     addSubresourceURL(urls, document()->completeURL(getAttribute(dataAttr)));
 
index c44da02..3b1a953 100644 (file)
@@ -83,7 +83,7 @@ private:
 
     virtual RenderWidget* renderWidgetForJSBindings();
 
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 
     virtual void updateWidget(PluginCreationOption);
     void updateDocNamedItem();
index 9f609ce..7190123 100644 (file)
@@ -66,9 +66,9 @@ bool HTMLParamElement::isURLAttribute(const Attribute& attribute) const
     return HTMLElement::isURLAttribute(attribute);
 }
 
-void HTMLParamElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void HTMLParamElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
-    HTMLElement::addSubresourceAttributeURLs(urls);
+    HTMLElement::collectSubresourceURLsFromAttributes(urls);
 
     if (!isURLParameter(name()))
         return;
index f868e13..a8c7778 100644 (file)
@@ -41,7 +41,7 @@ private:
 
     virtual bool isURLAttribute(const Attribute&) const OVERRIDE;
 
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 };
 
 } // namespace WebCore
index 4f2c956..13f8561 100644 (file)
@@ -113,9 +113,9 @@ KURL HTMLScriptElement::src() const
     return document()->completeURL(sourceAttributeValue());
 }
 
-void HTMLScriptElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void HTMLScriptElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
-    HTMLElement::addSubresourceAttributeURLs(urls);
+    HTMLElement::collectSubresourceURLsFromAttributes(urls);
 
     addSubresourceURL(urls, src());
 }
index c2ca0db..f905fbf 100644 (file)
@@ -50,7 +50,7 @@ private:
 
     virtual bool isURLAttribute(const Attribute&) const OVERRIDE;
 
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 
     virtual String sourceAttributeValue() const;
     virtual String charsetAttributeValue() const;
index 67ddceb..d6e74e8 100644 (file)
@@ -246,12 +246,12 @@ void HTMLStyleElement::notifyLoadedSheetAndAllCriticalSubresources(bool errorOcc
     m_firedLoad = true;
 }
 
-void HTMLStyleElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void HTMLStyleElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {    
-    HTMLElement::addSubresourceAttributeURLs(urls);
+    HTMLElement::collectSubresourceURLsFromAttributes(urls);
 
     if (CSSStyleSheet* styleSheet = const_cast<HTMLStyleElement*>(this)->sheet())
-        styleSheet->contents()->addSubresourceStyleURLs(urls);
+        styleSheet->contents()->collectSubresourceURLs(urls);
 }
 
 bool HTMLStyleElement::disabled() const
index 7e31254..3222b53 100644 (file)
@@ -71,7 +71,7 @@ private:
     virtual void notifyLoadedSheetAndAllCriticalSubresources(bool errorOccurred);
     virtual void startLoadingDynamicSheet() { StyleElement::startLoadingDynamicSheet(document()); }
 
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 
     virtual const AtomicString& media() const;
     virtual const AtomicString& type() const;
index c75ecc8..8d2ee73 100644 (file)
@@ -156,9 +156,9 @@ String HTMLTableCellElement::scope() const
     return getAttribute(scopeAttr);
 }
 
-void HTMLTableCellElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void HTMLTableCellElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
-    HTMLTablePartElement::addSubresourceAttributeURLs(urls);
+    HTMLTablePartElement::collectSubresourceURLsFromAttributes(urls);
 
     addSubresourceURL(urls, document()->completeURL(getAttribute(backgroundAttr)));
 }
index 5213e1c..8cba939 100644 (file)
@@ -61,7 +61,7 @@ private:
 
     virtual bool isURLAttribute(const Attribute&) const OVERRIDE;
 
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 };
 
 HTMLTableCellElement* toHTMLTableCellElement(Node* node);
index a6fa17d..af61e28 100644 (file)
@@ -582,9 +582,9 @@ String HTMLTableElement::summary() const
     return getAttribute(summaryAttr);
 }
 
-void HTMLTableElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void HTMLTableElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
-    HTMLElement::addSubresourceAttributeURLs(urls);
+    HTMLElement::collectSubresourceURLsFromAttributes(urls);
 
     addSubresourceURL(urls, document()->completeURL(getAttribute(backgroundAttr)));
 }
index d492b54..4e4c02f 100644 (file)
@@ -79,7 +79,7 @@ private:
     // Used to obtain either a solid or outset border decl and to deal with the frame and rules attributes.
     virtual StylePropertySet* additionalAttributeStyle() OVERRIDE;
 
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 
     enum TableRules { UnsetRules, NoneRules, GroupsRules, RowsRules, ColsRules, AllRules };
     enum CellBorders { NoBorders, SolidBorders, InsetBorders, SolidBordersColsOnly, SolidBordersRowsOnly };
index f58816b..ca01ec9 100644 (file)
@@ -134,9 +134,9 @@ void SVGCursorElement::svgAttributeChanged(const QualifiedName& attrName)
         (*it)->setNeedsStyleRecalc();
 }
 
-void SVGCursorElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void SVGCursorElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
-    SVGElement::addSubresourceAttributeURLs(urls);
+    SVGElement::collectSubresourceURLsFromAttributes(urls);
 
     addSubresourceURL(urls, document()->completeURL(href()));
 }
index 68757ab..e81c9ff 100644 (file)
@@ -54,7 +54,7 @@ private:
     virtual void parseAttribute(const Attribute&) OVERRIDE;
     virtual void svgAttributeChanged(const QualifiedName&);
 
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 
     BEGIN_DECLARE_ANIMATED_PROPERTIES(SVGCursorElement)
         DECLARE_ANIMATED_LENGTH(X, x)
index 75d0b43..a2ee710 100644 (file)
@@ -210,9 +210,9 @@ PassRefPtr<FilterEffect> SVGFEImageElement::build(SVGFilterBuilder*, Filter* fil
     return FEImage::createWithIRIReference(filter, document(), href(), preserveAspectRatio());
 }
 
-void SVGFEImageElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void SVGFEImageElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
-    SVGFilterPrimitiveStandardAttributes::addSubresourceAttributeURLs(urls);
+    SVGFilterPrimitiveStandardAttributes::collectSubresourceURLsFromAttributes(urls);
 
     addSubresourceURL(urls, document()->completeURL(href()));
 }
index de60642..740acc0 100644 (file)
@@ -53,7 +53,7 @@ private:
     virtual void svgAttributeChanged(const QualifiedName&);
     virtual void notifyFinished(CachedResource*);
 
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
     virtual PassRefPtr<FilterEffect> build(SVGFilterBuilder*, Filter*);
 
     void clearResourceReferences();
index 5558d10..000fd08 100644 (file)
@@ -227,9 +227,9 @@ const QualifiedName& SVGImageElement::imageSourceAttributeName() const
     return XLinkNames::hrefAttr;
 }
 
-void SVGImageElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void SVGImageElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
-    SVGStyledTransformableElement::addSubresourceAttributeURLs(urls);
+    SVGStyledTransformableElement::collectSubresourceURLsFromAttributes(urls);
 
     addSubresourceURL(urls, document()->completeURL(href()));
 }
index f2f8d9b..f28a9f4 100644 (file)
@@ -60,7 +60,7 @@ private:
     virtual RenderObject* createRenderer(RenderArena*, RenderStyle*);
  
     virtual const QualifiedName& imageSourceAttributeName() const;       
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 
     virtual bool haveLoadedRequiredResources();
 
index ff9834c..53fd4b8 100644 (file)
@@ -153,9 +153,9 @@ void SVGScriptElement::setType(const String& type)
     m_type = type;
 }
 
-void SVGScriptElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
+void SVGScriptElement::collectSubresourceURLsFromAttributes(ListHashSet<KURL>& urls) const
 {
-    SVGElement::addSubresourceAttributeURLs(urls);
+    SVGElement::collectSubresourceURLsFromAttributes(urls);
 
     addSubresourceURL(urls, document()->completeURL(href()));
 }
index 503598f..06af648 100644 (file)
@@ -53,7 +53,7 @@ private:
     virtual bool isURLAttribute(const Attribute&) const OVERRIDE;
     virtual void finishParsingChildren();
 
-    virtual void addSubresourceAttributeURLs(ListHashSet<KURL>&) const;
+    virtual void collectSubresourceURLsFromAttributes(ListHashSet<KURL>&) const;
 
     virtual bool haveLoadedRequiredResources() { return SVGExternalResourcesRequired::haveLoadedRequiredResources(); }