Title: [87983] trunk/Source/WebKit/qt
Revision
87983
Author
caio.olive...@openbossa.org
Date
2011-06-02 18:44:54 -0700 (Thu, 02 Jun 2011)

Log Message

2011-06-02  Caio Marcelo de Oliveira Filho  <caio.olive...@openbossa.org>

        Reviewed by Andreas Kling.

        [Qt] Fix enum order in qwebpage.h public API header
        https://bugs.webkit.org/show_bug.cgi?id=61959

        Since qwebpage.h is a public header, we can't change the values of
        the enumerations. This patch moves the new enumeration to the end
        of the list. As a bonus, adds a missing entry in
        editorCommandWebActions table.

        * Api/qwebpage.cpp:
        * Api/qwebpage.h:

Modified Paths

Diff

Modified: trunk/Source/WebKit/qt/Api/qwebpage.cpp (87982 => 87983)


--- trunk/Source/WebKit/qt/Api/qwebpage.cpp	2011-06-03 01:44:52 UTC (rev 87982)
+++ trunk/Source/WebKit/qt/Api/qwebpage.cpp	2011-06-03 01:44:54 UTC (rev 87983)
@@ -173,7 +173,6 @@
     0, // OpenImageInNewWindow,
     0, // DownloadImageToDisk,
     0, // CopyImageToClipboard,
-    0, // CopyImageUrlToClipboard,
 
     0, // Back,
     0, // Forward,
@@ -244,6 +243,10 @@
     "AlignLeft", // AlignLeft,
     "AlignRight", // AlignRight,
 
+    0, // StopScheduledPageRefresh,
+
+    0, // CopyImageUrlToClipboard,
+
     0 // WebActionCount
 };
 

Modified: trunk/Source/WebKit/qt/Api/qwebpage.h (87982 => 87983)


--- trunk/Source/WebKit/qt/Api/qwebpage.h	2011-06-03 01:44:52 UTC (rev 87982)
+++ trunk/Source/WebKit/qt/Api/qwebpage.h	2011-06-03 01:44:54 UTC (rev 87983)
@@ -105,7 +105,6 @@
         OpenImageInNewWindow,
         DownloadImageToDisk,
         CopyImageToClipboard,
-        CopyImageUrlToClipboard,
 
         Back,
         Forward,
@@ -179,6 +178,8 @@
 
         StopScheduledPageRefresh,
 
+        CopyImageUrlToClipboard,
+
         WebActionCount
     };
 

Modified: trunk/Source/WebKit/qt/ChangeLog (87982 => 87983)


--- trunk/Source/WebKit/qt/ChangeLog	2011-06-03 01:44:52 UTC (rev 87982)
+++ trunk/Source/WebKit/qt/ChangeLog	2011-06-03 01:44:54 UTC (rev 87983)
@@ -1,3 +1,18 @@
+2011-06-02  Caio Marcelo de Oliveira Filho  <caio.olive...@openbossa.org>
+
+        Reviewed by Andreas Kling.
+
+        [Qt] Fix enum order in qwebpage.h public API header
+        https://bugs.webkit.org/show_bug.cgi?id=61959
+
+        Since qwebpage.h is a public header, we can't change the values of
+        the enumerations. This patch moves the new enumeration to the end
+        of the list. As a bonus, adds a missing entry in
+        editorCommandWebActions table.
+
+        * Api/qwebpage.cpp:
+        * Api/qwebpage.h:
+
 2011-06-02  Andreas Kling  <kl...@webkit.org>
 
         Rubber-stamped by Simon Hausmann.
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to