Title: [201143] branches/safari-602.1.32-branch/Source/WebKit2
Revision
201143
Author
[email protected]
Date
2016-05-19 01:40:32 -0700 (Thu, 19 May 2016)

Log Message

Merge r200961. rdar://problem/25984480

Modified Paths

Diff

Modified: branches/safari-602.1.32-branch/Source/WebKit2/ChangeLog (201142 => 201143)


--- branches/safari-602.1.32-branch/Source/WebKit2/ChangeLog	2016-05-19 08:40:29 UTC (rev 201142)
+++ branches/safari-602.1.32-branch/Source/WebKit2/ChangeLog	2016-05-19 08:40:32 UTC (rev 201143)
@@ -1,5 +1,24 @@
 2016-05-19  Babak Shafiei  <[email protected]>
 
+        Merge r200961. rdar://problem/25984480
+
+    2016-05-16  Brian Burg  <[email protected]>
+
+            Crash simulating keystrokes at WebKit::WebAutomationSession::platformSimulateKeyStroke
+            https://bugs.webkit.org/show_bug.cgi?id=157737
+            <rdar://problem/26292946>
+
+            Reviewed by Timothy Hatcher.
+
+            * UIProcess/Cocoa/WebAutomationSessionCocoa.mm:
+            (WebKit::WebAutomationSession::platformSimulateKeyStroke):
+            AppKit expects characters passed to [NSEvent keyEventWithType:...]
+            to be non-nil even if there are no characters. Initialize characters
+            to the empty string @"". It will be overwritten if the keystroke should
+            produce any unicode characters.
+
+2016-05-19  Babak Shafiei  <[email protected]>
+
         Merge r200959. rdar://problem/26298247
 
     2016-05-16  Brian Burg  <[email protected]>

Modified: branches/safari-602.1.32-branch/Source/WebKit2/UIProcess/Cocoa/WebAutomationSessionCocoa.mm (201142 => 201143)


--- branches/safari-602.1.32-branch/Source/WebKit2/UIProcess/Cocoa/WebAutomationSessionCocoa.mm	2016-05-19 08:40:29 UTC (rev 201142)
+++ branches/safari-602.1.32-branch/Source/WebKit2/UIProcess/Cocoa/WebAutomationSessionCocoa.mm	2016-05-19 08:40:32 UTC (rev 201143)
@@ -180,7 +180,7 @@
     // The likely keyCode for the virtual key as defined in <HIToolbox/Events.h>.
     int keyCode = 0;
     // Typical characters produced by the virtual key, if any.
-    NSString *characters;
+    NSString *characters = @"";
 
     // FIXME: this function and the Automation protocol enum should probably adopt key names
     // from W3C UIEvents standard. For more details: https://w3c.github.io/uievents-code/
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to