Title: [161259] trunk/Source/WebCore
Revision
161259
Author
[email protected]
Date
2014-01-03 03:44:48 -0800 (Fri, 03 Jan 2014)

Log Message

Remove unused Document::openSearchDescriptionURL().
<https://webkit.org/b/126419>

Reviewed by Antti Koivisto.

* dom/Document.h:
* dom/Document.cpp:

    Scrub leftovers from the defunct Chromium port.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (161258 => 161259)


--- trunk/Source/WebCore/ChangeLog	2014-01-03 09:06:22 UTC (rev 161258)
+++ trunk/Source/WebCore/ChangeLog	2014-01-03 11:44:48 UTC (rev 161259)
@@ -1,3 +1,15 @@
+2014-01-03  Andreas Kling  <[email protected]>
+
+        Remove unused Document::openSearchDescriptionURL().
+        <https://webkit.org/b/126419>
+
+        Reviewed by Antti Koivisto.
+
+        * dom/Document.h:
+        * dom/Document.cpp:
+
+            Scrub leftovers from the defunct Chromium port.
+
 2014-01-03  Jinwoo Song  <[email protected]>
 
         VibrationPattern should allocate an single vector instance for single integer input

Modified: trunk/Source/WebCore/dom/Document.cpp (161258 => 161259)


--- trunk/Source/WebCore/dom/Document.cpp	2014-01-03 09:06:22 UTC (rev 161258)
+++ trunk/Source/WebCore/dom/Document.cpp	2014-01-03 11:44:48 UTC (rev 161259)
@@ -4247,37 +4247,6 @@
     return command(this, commandName).value();
 }
 
-URL Document::openSearchDescriptionURL()
-{
-    static const char* const openSearchMIMEType = "application/opensearchdescription+xml";
-    static const char* const openSearchRelation = "search";
-
-    // FIXME: Why do only top-level frames have openSearchDescriptionURLs?
-    if (!frame() || frame()->tree().parent())
-        return URL();
-
-    // FIXME: Why do we need to wait for FrameStateComplete?
-    if (frame()->loader().state() != FrameStateComplete)
-        return URL();
-
-    if (!head())
-        return URL();
-
-    RefPtr<HTMLCollection> children = head()->children();
-    for (unsigned i = 0; Node* child = children->item(i); i++) {
-        if (!child->hasTagName(linkTag))
-            continue;
-        HTMLLinkElement* linkElement = toHTMLLinkElement(child);
-        if (!equalIgnoringCase(linkElement->type(), openSearchMIMEType) || !equalIgnoringCase(linkElement->rel(), openSearchRelation))
-            continue;
-        if (linkElement->href().isEmpty())
-            continue;
-        return linkElement->href();
-    }
-
-    return URL();
-}
-
 void Document::pushCurrentScript(PassRefPtr<HTMLScriptElement> newCurrentScript)
 {
     ASSERT(newCurrentScript);

Modified: trunk/Source/WebCore/dom/Document.h (161258 => 161259)


--- trunk/Source/WebCore/dom/Document.h	2014-01-03 09:06:22 UTC (rev 161258)
+++ trunk/Source/WebCore/dom/Document.h	2014-01-03 11:44:48 UTC (rev 161259)
@@ -897,8 +897,6 @@
     bool queryCommandSupported(const String& command);
     String queryCommandValue(const String& command);
 
-    URL openSearchDescriptionURL();
-
     // designMode support
     enum InheritedBool { off = false, on = true, inherit };    
     void setDesignMode(InheritedBool value);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to