26009 2009-05-25 07:07:16 -0700 REGRESSION (r43973): Problems While Working With OpenVanilla 2009-06-26 10:44:58 -0700 1 1 1 Unclassified WebKit WebKit Misc. 528+ (Nightly build) Mac (Intel) OS X 10.5 RESOLVED FIXED InRadar, Regression P1 Major --- 1 ksliou ap ap mitz oldest_to_newest 123040 0 ksliou 2009-05-25 07:07:16 -0700 When input with openarry (array module in openvanilla) in a text input form such like the search field on Google, it ignores the first input key, actually, the second key replace the first key. It doest not occur in Safari 4 Beta. 123270 1 ap 2009-05-27 05:01:30 -0700 This was hopefully fixed in <http://trac.webkit.org/projects/webkit/changeset/44164>. Please test again with the latest nightly build. 123287 2 ksliou 2009-05-27 08:01:14 -0700 It still does not work. Please check it one more time. Here is the link of the IM OpenVanilla: http://code.google.com/p/openvanilla/ My situation is, the first key input in the field would be replaced by the second key. 123288 3 ksliou 2009-05-27 08:08:00 -0700 Excuse me, it sometimes work in the new build, but not always work. 123289 4 ap 2009-05-27 08:33:06 -0700 Hmm, I think I get it now, looks like a completely unrelated issue. Please confirm that this is the problem: 1) Switch to OpenVanilla Array input method. 2) Type 'q', then '1'. Results: Instead of an ideogram, '1' is inserted. 123290 5 ap 2009-05-27 08:37:00 -0700 > looks like a completely unrelated issue. or maybe not that unrelated - in console log, I see unexpected messages from OpenVanilla about changing input server. 123291 6 ksliou 2009-05-27 08:38:40 -0700 Yes, '1' is inserted instead of an ideogram. 123298 7 ap 2009-05-27 09:51:33 -0700 the problem here is that when a character is typed, WebCore temporarily sets selection to none, so we tell input methods that the input context is nil. 123299 8 ap 2009-05-27 09:52:14 -0700 <rdar://problem/6925656> 123301 9 30711 ap 2009-05-27 10:10:43 -0700 Created attachment 30711 proposed fix 123304 10 30711 darin 2009-05-27 10:58:02 -0700 Comment on attachment 30711 proposed fix r=me 123305 11 ap 2009-05-27 11:02:19 -0700 Committed <http://trac.webkit.org/changeset/44194>. 127407 12 ap 2009-06-22 11:36:54 -0700 Please verify that OpenVanilla works with current nightly builds (newer than r44941) - we've had many changes in this area after fixing this bug. 128511 13 ksliou 2009-06-26 10:44:58 -0700 It works correctly in r45247 30711 2009-05-27 10:10:43 -0700 2009-05-27 10:58:02 -0700 proposed fix OpenVanilla.txt text/plain 1789 ap SW5kZXg6IFdlYktpdC9tYWMvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYktpdC9tYWMvQ2hh bmdlTG9nCShyZXZpc2lvbiA0NDE4OSkKKysrIFdlYktpdC9tYWMvQ2hhbmdlTG9nCSh3b3JraW5n IGNvcHkpCkBAIC0xLDMgKzEsMTUgQEAKKzIwMDktMDUtMjcgIEFsZXhleSBQcm9za3VyeWFrb3Yg IDxhcEB3ZWJraXQub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yNjAwOQor ICAgICAgICA8cmRhcjovL3Byb2JsZW0vNjkyNTY1Nj4gUkVHUkVTU0lPTiAocjQzOTczKTogUHJv YmxlbXMgV2hpbGUgV29ya2luZyBXaXRoIE9wZW5WYW5pbGxhCisKKyAgICAgICAgKiBXZWJWaWV3 L1dlYkhUTUxWaWV3Lm1tOgorICAgICAgICAoLVtXZWJIVE1MVmlldyBfdXBkYXRlU2VsZWN0aW9u Rm9ySW5wdXRNYW5hZ2VyXSk6IERvbid0IGNhbGwgdXBkYXRlV2luZG93cyBpZiB0aGUgc2VsZWN0 aW9uCisgICAgICAgIGlzIE5vbmUuIFRoaXMgcm91dGluZWx5IGhhcHBlbnMgZHVyaW5nIGVkaXRp bmcsIGFuZCBpdCBkb2Vzbid0IG1lYW4gdGhhdCB3ZSBsZWZ0IGFuIGVkaXRhYmxlCisgICAgICAg IGFyZWEgKGluIHdoaWNoIGNhc2UgdGhlIHNlbGVjdGlvbiBjaGFuZ2VzIHRvIGEgbm9uLWVkaXRh YmxlIG9uZSkuCisKIDIwMDktMDUtMjYgIFNhbSBXZWluaWcgIDxzYW1Ad2Via2l0Lm9yZz4KIAog ICAgICAgICBSZXZpZXdlZCBieSBCcmFkeSBFaWRzb24uCkluZGV4OiBXZWJLaXQvbWFjL1dlYlZp ZXcvV2ViSFRNTFZpZXcubW0KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViS2l0L21hYy9XZWJWaWV3L1dlYkhU TUxWaWV3Lm1tCShyZXZpc2lvbiA0NDE4MSkKKysrIFdlYktpdC9tYWMvV2ViVmlldy9XZWJIVE1M Vmlldy5tbQkod29ya2luZyBjb3B5KQpAQCAtNTg1OSw5ICs1ODU5LDExIEBAIHN0YXRpYyB2b2lk IGV4dHJhY3RVbmRlcmxpbmVzKE5TQXR0cmlidXQKIAogICAgIEJPT0wgZXhwb3NlSW5wdXRDb250 ZXh0ID0gaXNUZXh0SW5wdXQoY29yZUZyYW1lKSAmJiAhaXNJblBhc3N3b3JkRmllbGQoY29yZUZy YW1lKTsKICAgICBpZiAoZXhwb3NlSW5wdXRDb250ZXh0ICE9IF9wcml2YXRlLT5leHBvc2VJbnB1 dENvbnRleHQpIHsKLSAgICAgICAgLy8gTGV0IEFwcEtpdCBjYWNoZSBhIHBvdGVudGlhbGx5IGlu cHV0IGNvbnRleHQuCiAgICAgICAgIF9wcml2YXRlLT5leHBvc2VJbnB1dENvbnRleHQgPSBleHBv c2VJbnB1dENvbnRleHQ7Ci0gICAgICAgIFtOU0FwcCB1cGRhdGVXaW5kb3dzXTsKKyAgICAgICAg Ly8gTGV0IEFwcEtpdCBjYWNoZSBhIHBvdGVudGlhbGx5IGNoYW5nZWQgaW5wdXQgY29udGV4dC4K KyAgICAgICAgLy8gV2ViQ29yZSByb3V0aW5lbHkgc2V0cyB0aGUgc2VsZWN0aW9uIHRvIE5vbmUg d2hlbiBlZGl0aW5nLCBhbmQgSU1zIGJlY29tZSB1bmhhcHB5IHdoZW4gYW4gaW5wdXQgY29udGV4 dCBzdWRkZW5seSB0dXJucyBuaWwsIHNlZSBidWcgMjYwMDkuCisgICAgICAgIGlmICghY29yZUZy YW1lIHx8ICFjb3JlRnJhbWUtPnNlbGVjdGlvbigpLT5pc05vbmUoKSkKKyAgICAgICAgICAgIFtO U0FwcCB1cGRhdGVXaW5kb3dzXTsKICAgICB9CiAKICAgICBpZiAoIWNvcmVGcmFtZS0+ZWRpdG9y KCktPmhhc0NvbXBvc2l0aW9uKCkpCg==