Title: [191574] releases/WebKitGTK/webkit-2.10/Source/WebKit2
Revision
191574
Author
[email protected]
Date
2015-10-26 01:54:29 -0700 (Mon, 26 Oct 2015)

Log Message

Merge r191503 - [GTK] Minor web editor fixes
https://bugs.webkit.org/show_bug.cgi?id=150501

Reviewed by Carlos Garcia Campos.

* UIProcess/gtk/KeyBindingTranslator.cpp: Remove
"MoveParagraphForward/Backward" as these commands do not exist in
WebKit (anymore?).
* WebProcess/WebCoreSupport/gtk/WebEditorClientGtk.cpp:
(WebKit::WebEditorClient::executePendingEditorCommands):
Editor::Command can take a String, no need to convert to a C char.

Modified Paths

Diff

Modified: releases/WebKitGTK/webkit-2.10/Source/WebKit2/ChangeLog (191573 => 191574)


--- releases/WebKitGTK/webkit-2.10/Source/WebKit2/ChangeLog	2015-10-26 08:53:19 UTC (rev 191573)
+++ releases/WebKitGTK/webkit-2.10/Source/WebKit2/ChangeLog	2015-10-26 08:54:29 UTC (rev 191574)
@@ -1,3 +1,17 @@
+2015-10-23  Claudio Saavedra  <[email protected]>
+
+        [GTK] Minor web editor fixes
+        https://bugs.webkit.org/show_bug.cgi?id=150501
+
+        Reviewed by Carlos Garcia Campos.
+
+        * UIProcess/gtk/KeyBindingTranslator.cpp: Remove
+        "MoveParagraphForward/Backward" as these commands do not exist in
+        WebKit (anymore?).
+        * WebProcess/WebCoreSupport/gtk/WebEditorClientGtk.cpp:
+        (WebKit::WebEditorClient::executePendingEditorCommands):
+        Editor::Command can take a String, no need to convert to a C char.
+
 2015-10-22  Carlos Garcia Campos  <[email protected]>
 
         ERROR: Unhandled web process message 'StorageAreaMap:DispatchStorageEvent'

Modified: releases/WebKitGTK/webkit-2.10/Source/WebKit2/UIProcess/gtk/KeyBindingTranslator.cpp (191573 => 191574)


--- releases/WebKitGTK/webkit-2.10/Source/WebKit2/UIProcess/gtk/KeyBindingTranslator.cpp	2015-10-26 08:53:19 UTC (rev 191573)
+++ releases/WebKitGTK/webkit-2.10/Source/WebKit2/UIProcess/gtk/KeyBindingTranslator.cpp	2015-10-26 08:54:29 UTC (rev 191574)
@@ -130,7 +130,7 @@
       "MoveUpAndModifySelection",                       "MoveDownAndModifySelection"                }, // Up/down line
     { "MoveToBeginningOfLine",                          "MoveToEndOfLine",
       "MoveToBeginningOfLineAndModifySelection",        "MoveToEndOfLineAndModifySelection"         }, // Up/down line ends
-    { "MoveParagraphBackward",                          "MoveParagraphForward",
+    { 0,                                                0,
       "MoveParagraphBackwardAndModifySelection",        "MoveParagraphForwardAndModifySelection"    }, // Up/down paragraphs
     { "MoveToBeginningOfParagraph",                     "MoveToEndOfParagraph",
       "MoveToBeginningOfParagraphAndModifySelection",   "MoveToEndOfParagraphAndModifySelection"    }, // Up/down paragraph ends.

Modified: releases/WebKitGTK/webkit-2.10/Source/WebKit2/WebProcess/WebCoreSupport/gtk/WebEditorClientGtk.cpp (191573 => 191574)


--- releases/WebKitGTK/webkit-2.10/Source/WebKit2/WebProcess/WebCoreSupport/gtk/WebEditorClientGtk.cpp	2015-10-26 08:53:19 UTC (rev 191573)
+++ releases/WebKitGTK/webkit-2.10/Source/WebKit2/WebProcess/WebCoreSupport/gtk/WebEditorClientGtk.cpp	2015-10-26 08:54:29 UTC (rev 191574)
@@ -39,7 +39,7 @@
 {
     Vector<Editor::Command> commands;
     for (auto& commandString : pendingEditorCommands) {
-        Editor::Command command = frame->editor().command(commandString.utf8().data());
+        Editor::Command command = frame->editor().command(commandString);
         if (command.isTextInsertion() && !allowTextInsertion)
             return false;
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to