Title: [163557] trunk/Source/WebKit/win
Revision
163557
Author
[email protected]
Date
2014-02-06 12:38:48 -0800 (Thu, 06 Feb 2014)

Log Message

[Win] Correct IME Regression after r141479
https://bugs.webkit.org/show_bug.cgi?id=128323
<rdar://problem/15952986>

Reviewed by Tim Horton.

* WebView.cpp:
(WebView::updateSelectionForIME): Refactoring flipped the meaning of the test used
to select the resetIME case. Corrected for behavior of new cancel method.

Modified Paths

Diff

Modified: trunk/Source/WebKit/win/ChangeLog (163556 => 163557)


--- trunk/Source/WebKit/win/ChangeLog	2014-02-06 20:22:43 UTC (rev 163556)
+++ trunk/Source/WebKit/win/ChangeLog	2014-02-06 20:38:48 UTC (rev 163557)
@@ -1,3 +1,15 @@
+2014-02-06  Brent Fulgham  <[email protected]>
+
+        [Win] Correct IME Regression after r141479
+        https://bugs.webkit.org/show_bug.cgi?id=128323
+        <rdar://problem/15952986>
+
+        Reviewed by Tim Horton.
+
+        * WebView.cpp:
+        (WebView::updateSelectionForIME): Refactoring flipped the meaning of the test used
+        to select the resetIME case. Corrected for behavior of new cancel method.
+
 2014-02-04  Andreas Kling  <[email protected]>
 
         Remove <iframe seamless> support.

Modified: trunk/Source/WebKit/win/WebView.cpp (163556 => 163557)


--- trunk/Source/WebKit/win/WebView.cpp	2014-02-06 20:22:43 UTC (rev 163556)
+++ trunk/Source/WebKit/win/WebView.cpp	2014-02-06 20:38:48 UTC (rev 163557)
@@ -5563,7 +5563,7 @@
 void WebView::updateSelectionForIME()
 {
     Frame& targetFrame = m_page->focusController().focusedOrMainFrame();
-    if (!targetFrame.editor().cancelCompositionIfSelectionIsInvalid())
+    if (targetFrame.editor().cancelCompositionIfSelectionIsInvalid())
         resetIME(&targetFrame);
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to