Title: [150238] trunk/Source/WebCore
Revision
150238
Author
mifen...@rim.com
Date
2013-05-16 20:20:24 -0700 (Thu, 16 May 2013)

Log Message

[BlackBerry] Strip invoke URLs when writing to the clipboard.
https://bugs.webkit.org/show_bug.cgi?id=116226

Reviewed by Rob Buis.

PR 333516.

Invoke URLs should not be included in the paste data.

Internally Reviewed by Gen Mak.

* platform/blackberry/PasteboardBlackBerry.cpp:
(WebCore::Pasteboard::writeSelection):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (150237 => 150238)


--- trunk/Source/WebCore/ChangeLog	2013-05-17 03:18:00 UTC (rev 150237)
+++ trunk/Source/WebCore/ChangeLog	2013-05-17 03:20:24 UTC (rev 150238)
@@ -1,3 +1,19 @@
+2013-05-16  Mike Fenton  <mifen...@rim.com>
+
+        [BlackBerry] Strip invoke URLs when writing to the clipboard.
+        https://bugs.webkit.org/show_bug.cgi?id=116226
+
+        Reviewed by Rob Buis.
+
+        PR 333516.
+
+        Invoke URLs should not be included in the paste data.
+
+        Internally Reviewed by Gen Mak.
+
+        * platform/blackberry/PasteboardBlackBerry.cpp:
+        (WebCore::Pasteboard::writeSelection):
+
 2013-05-16  Eduardo Lima Mitev  <el...@igalia.com>
 
         Missing UNUSED_PARAM macro for textPosition argument in WebKitAccessibleInterfaceText

Modified: trunk/Source/WebCore/platform/blackberry/PasteboardBlackBerry.cpp (150237 => 150238)


--- trunk/Source/WebCore/platform/blackberry/PasteboardBlackBerry.cpp	2013-05-17 03:18:00 UTC (rev 150237)
+++ trunk/Source/WebCore/platform/blackberry/PasteboardBlackBerry.cpp	2013-05-17 03:20:24 UTC (rev 150238)
@@ -65,6 +65,12 @@
 {
     WTF::String text = shouldSerializeSelectedTextForClipboard == IncludeImageAltTextForClipboard ? frame->editor()->selectedTextForClipboard() : frame->editor()->selectedText();
     WTF::String html = createMarkup(selectedRange, 0, AnnotateForInterchange);
+    DEFINE_STATIC_LOCAL(AtomicString, invokeString, ("href=""
+    size_t startOfInvoke = html.find(invokeString);
+    if (startOfInvoke != notFound) {
+        size_t endOfInvoke = html.find("\"", startOfInvoke + invokeString.length()) + 1;
+        html.remove(startOfInvoke, endOfInvoke - startOfInvoke);
+    }
     WTF::String url = ""
 
     BlackBerry::Platform::Clipboard::write(text, html, url);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to