Move StyleSheet::addSubresourceStyleURLs() to CSSStyleSheet.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 24 Oct 2011 01:49:50 +0000 (01:49 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 24 Oct 2011 01:49:50 +0000 (01:49 +0000)
https://bugs.webkit.org/show_bug.cgi?id=70690

Patch by Andreas Kling <kling@webkit.org> on 2011-10-23
Reviewed by Antti Koivisto.

This method isn't part of the StyleSheet DOM interface, nor is it used by
XSLStyleSheet, so move it down to CSSStyleSheet and make it non-virtual.
Also remove some nearby unneeded casts from CSSStyleSheet* to StyleSheet*.

* css/CSSStyleSheet.h:
* css/StyleSheet.h:
* html/HTMLLinkElement.cpp:
(WebCore::HTMLLinkElement::isLoading):
(WebCore::HTMLLinkElement::addSubresourceAttributeURLs):
* html/HTMLStyleElement.cpp:
(WebCore::HTMLStyleElement::addSubresourceAttributeURLs):
(WebCore::HTMLStyleElement::disabled):
(WebCore::HTMLStyleElement::setDisabled):

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

Source/WebCore/ChangeLog
Source/WebCore/css/CSSStyleSheet.h
Source/WebCore/css/StyleSheet.h
Source/WebCore/html/HTMLLinkElement.cpp
Source/WebCore/html/HTMLStyleElement.cpp

index b2367cf..006bc25 100755 (executable)
@@ -1,5 +1,26 @@
 2011-10-23  Andreas Kling  <kling@webkit.org>
 
+        Move StyleSheet::addSubresourceStyleURLs() to CSSStyleSheet.
+        https://bugs.webkit.org/show_bug.cgi?id=70690
+
+        Reviewed by Antti Koivisto.
+
+        This method isn't part of the StyleSheet DOM interface, nor is it used by
+        XSLStyleSheet, so move it down to CSSStyleSheet and make it non-virtual.
+        Also remove some nearby unneeded casts from CSSStyleSheet* to StyleSheet*.
+
+        * css/CSSStyleSheet.h:
+        * css/StyleSheet.h:
+        * html/HTMLLinkElement.cpp:
+        (WebCore::HTMLLinkElement::isLoading):
+        (WebCore::HTMLLinkElement::addSubresourceAttributeURLs):
+        * html/HTMLStyleElement.cpp:
+        (WebCore::HTMLStyleElement::addSubresourceAttributeURLs):
+        (WebCore::HTMLStyleElement::disabled):
+        (WebCore::HTMLStyleElement::setDisabled):
+
+2011-10-23  Andreas Kling  <kling@webkit.org>
+
         Remove style sheet constructors that take a style sheet parent.
         https://bugs.webkit.org/show_bug.cgi?id=70692
 
index b9e5090..36785ae 100644 (file)
@@ -95,7 +95,7 @@ public:
     bool loadCompleted() const { return m_loadCompleted; }
 
     virtual KURL completeURL(const String& url) const;
-    virtual void addSubresourceStyleURLs(ListHashSet<KURL>&);
+    void addSubresourceStyleURLs(ListHashSet<KURL>&);
 
     void setStrictParsing(bool b) { m_strictParsing = b; }
     bool useStrictParsing() const { return m_strictParsing; }
index e88abb2..40908d0 100644 (file)
@@ -61,8 +61,6 @@ public:
     virtual void styleSheetChanged() { }
 
     virtual KURL completeURL(const String& url) const;
-    virtual void addSubresourceStyleURLs(ListHashSet<KURL>&) { }
-
     virtual bool parseString(const String&, bool strict = true) = 0;
 
 protected:
index 73d2301..273f895 100644 (file)
@@ -339,7 +339,7 @@ bool HTMLLinkElement::isLoading() const
         return true;
     if (!m_sheet)
         return false;
-    return static_cast<CSSStyleSheet *>(m_sheet.get())->isLoading();
+    return m_sheet->isLoading();
 }
 
 void HTMLLinkElement::linkLoaded()
@@ -408,7 +408,7 @@ void HTMLLinkElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
     addSubresourceURL(urls, href());
     
     // Walk the URLs linked by the linked-to stylesheet.
-    if (StyleSheet* styleSheet = const_cast<HTMLLinkElement*>(this)->sheet())
+    if (CSSStyleSheet* styleSheet = const_cast<HTMLLinkElement*>(this)->sheet())
         styleSheet->addSubresourceStyleURLs(urls);
 }
 
index 648d0bd..2581661 100644 (file)
@@ -98,22 +98,21 @@ void HTMLStyleElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) cons
 {    
     HTMLElement::addSubresourceAttributeURLs(urls);
 
-    if (StyleSheet* styleSheet = const_cast<HTMLStyleElement*>(this)->sheet())
+    if (CSSStyleSheet* styleSheet = const_cast<HTMLStyleElement*>(this)->sheet())
         styleSheet->addSubresourceStyleURLs(urls);
 }
 
 bool HTMLStyleElement::disabled() const
 {
-    StyleSheet* styleSheet = sheet();
-    if (!styleSheet)
+    if (!m_sheet)
         return false;
 
-    return styleSheet->disabled();
+    return m_sheet->disabled();
 }
 
 void HTMLStyleElement::setDisabled(bool setDisabled)
 {
-    if (StyleSheet* styleSheet = sheet())
+    if (CSSStyleSheet* styleSheet = sheet())
         styleSheet->setDisabled(setDisabled);
 }