Changeset 150238 in webkit

Timestamp:

May 16, 2013 8:20:24 PM (11 years ago)

Author:

mifenton@rim.com

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):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/blackberry/PasteboardBlackBerry.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-05-16  Mike Fenton  <mifenton@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  <elima@igalia.com>
 

trunk/Source/WebCore/platform/blackberry/PasteboardBlackBerry.cpp

@@ -66 +66 @@
     WTF::String text = shouldSerializeSelectedTextForClipboard == IncludeImageAltTextForClipboard ? frame->editor()->selectedTextForClipboard() : frame->editor()->selectedText();
     WTF::String html = createMarkup(selectedRange, 0, AnnotateForInterchange);
+    DEFINE_STATIC_LOCAL(AtomicString, invokeString, ("href=\"invoke://"));
+    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 = frame->document()->url().string();