Title: [123828] trunk/Source/WebCore
- Revision
- 123828
- Author
- t...@chromium.org
- Date
- 2012-07-26 18:41:48 -0700 (Thu, 26 Jul 2012)
Log Message
[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):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (123827 => 123828)
--- trunk/Source/WebCore/ChangeLog 2012-07-27 01:28:51 UTC (rev 123827)
+++ trunk/Source/WebCore/ChangeLog 2012-07-27 01:41:48 UTC (rev 123828)
@@ -1,3 +1,17 @@
+2012-07-26 Tony Chang <t...@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 <sukol...@google.com>
Implement undoscope attribute.
Modified: trunk/Source/WebCore/platform/chromium/ClipboardChromium.cpp (123827 => 123828)
--- trunk/Source/WebCore/platform/chromium/ClipboardChromium.cpp 2012-07-27 01:28:51 UTC (rev 123827)
+++ trunk/Source/WebCore/platform/chromium/ClipboardChromium.cpp 2012-07-27 01:41:48 UTC (rev 123828)
@@ -263,27 +263,6 @@
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 = "" line);
- if (url.isValid())
- return url;
- }
-
- return String();
}
bool ClipboardChromium::setData(const String& type, const String& data)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes