Changeset 41360 in webkit

Timestamp:

Mar 2, 2009 8:55:35 AM (15 years ago)

Author:

treat@webkit.org

Message:

2009-03-02 Benjamin C Meyer <​benjamin.meyer@torchmobile.com>

Reviewed by George Staikos.

​https://bugs.webkit.org/show_bug.cgi?id=21230
On X11 match the behavior of Firefox and also copy the url to the
clipboard selection when the action Copy Link Location is executed.

• Api/qwebpage.cpp: (QWebPage::triggerAction):

Location:

trunk/WebKit/qt

Files:

• Api/qwebpage.cpp (modified) (1 diff)
• ChangeLog (modified) (1 diff)

trunk/WebKit/qt/Api/qwebpage.cpp

@@ -1484 +1484 @@
             break;
         }
-        case CopyLinkToClipboard:
+        case CopyLinkToClipboard: {
+#if defined(Q_WS_X11)
+            bool oldSelectionMode = Pasteboard::generalPasteboard()->isSelectionMode();
+            Pasteboard::generalPasteboard()->setSelectionMode(true);
             editor->copyURL(d->hitTestResult.linkUrl(), d->hitTestResult.linkText());
-            break;
+            Pasteboard::generalPasteboard()->setSelectionMode(oldSelectionMode);
+#endif
+            editor->copyURL(d->hitTestResult.linkUrl(), d->hitTestResult.linkText());
+            break;
+        }
         case OpenImageInNewWindow:
             openNewWindow(d->hitTestResult.imageUrl(), frame);

trunk/WebKit/qt/ChangeLog

@@ +1 @@
+2009-03-02  Benjamin C Meyer  <benjamin.meyer@torchmobile.com>
+
+        Reviewed by George Staikos.
+
+        https://bugs.webkit.org/show_bug.cgi?id=21230
+        On X11 match the behavior of Firefox and also copy the url to the
+        clipboard selection when the action Copy Link Location is executed.
+
+        * Api/qwebpage.cpp:
+        (QWebPage::triggerAction):
+
 2009-02-25  Kavindra Palaraja  <kavindra.palaraja@nokia.com>