[chromium] Remove some unreachable code in ClipboardChromium.cpp
authortony@chromium.org <tony@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 27 Jul 2012 01:41:48 +0000 (01:41 +0000)
committertony@chromium.org <tony@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 27 Jul 2012 01:41:48 +0000 (01:41 +0000)
https://bugs.webkit.org/show_bug.cgi?id=92427

Reviewed by Adam Barth.

convertURIListToURL does the same thing (implementation in ClipboardUtilitiesChromium.*).

No new tests, just deleting unused code.

* platform/chromium/ClipboardChromium.cpp:
(WebCore::ClipboardChromium::getData):

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

Source/WebCore/ChangeLog
Source/WebCore/platform/chromium/ClipboardChromium.cpp

index b2cbda9..b90e68c 100644 (file)
@@ -1,3 +1,17 @@
+2012-07-26  Tony Chang  <tony@chromium.org>
+
+        [chromium] Remove some unreachable code in ClipboardChromium.cpp
+        https://bugs.webkit.org/show_bug.cgi?id=92427
+
+        Reviewed by Adam Barth.
+
+        convertURIListToURL does the same thing (implementation in ClipboardUtilitiesChromium.*).
+
+        No new tests, just deleting unused code.
+
+        * platform/chromium/ClipboardChromium.cpp:
+        (WebCore::ClipboardChromium::getData):
+
 2012-07-26  Sukolsak Sakshuwong  <sukolsak@google.com>
 
         Implement undoscope attribute.
index 598436d..b065237 100644 (file)
@@ -263,27 +263,6 @@ String ClipboardChromium::getData(const String& type) const
     if (!convertToURL)
         return data;
     return convertURIListToURL(data);
-
-    Vector<String> uriList;
-    // Line separator is \r\n per RFC 2483 - howver, for compatiblity
-    // reasons we also allow just \n here.
-    data.split('\n', uriList);
-    // Process the input and return the first valid RUL. In case no URLs can
-    // be found, return an empty string. This is in line with the HTML5
-    // spec (see "The DragEvent and DataTransfer interfaces").
-    for (size_t i = 0; i < uriList.size(); ++i) {
-        String& line = uriList[i];
-        line = line.stripWhiteSpace();
-        if (line.isEmpty())
-            continue;
-        if (line[0] == '#')
-            continue;
-        KURL url = KURL(ParsedURLString, line);
-        if (url.isValid())
-            return url;
-    }
-
-    return String();
 }
 
 bool ClipboardChromium::setData(const String& type, const String& data)