Bug 19347 - After the password is input , Japanese can't be input.
Summary: After the password is input , Japanese can't be input.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Major
Assignee: Alexey Proskuryakov
URL: https://edit.yahoo.co.jp/config/eval_...
Keywords: InRadar
: 20392 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-05-31 21:45 PDT by Sato
Modified: 2008-08-19 00:49 PDT (History)
4 users (show)

See Also:


Attachments
proposed fix (6.49 KB, patch)
2008-08-18 09:35 PDT, Alexey Proskuryakov
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sato 2008-05-31 21:45:19 PDT
I'm using WebKit-SVN-r34279/Mac OS 10.5.3(intel).
After is input in password form (<input type="password">), Japanese cannot be input to other forms.

Example, "Create new account" page in Yahoo! Japan,
After the password is input, the answer to a secret question cannot be input in 
Japanese.
Comment 1 Mark Rowe (bdash) 2008-05-31 23:36:32 PDT
Does this work correctly in Safari 3.1.1?
Comment 2 Mark Rowe (bdash) 2008-05-31 23:36:43 PDT
<rdar://problem/5977562>
Comment 3 Sato 2008-06-01 07:33:47 PDT
(In reply to comment #1)
> Does this work correctly in Safari 3.1.1?
> 

When having tried now,The problem occurred in Safari 3.1.1 (5525.20). 
This problem seems to occur only by Leopard.The problem did not occur in Tiger.
Comment 4 mitz 2008-06-01 09:43:38 PDT
I think this is a duplicate of bug 16946.
Comment 5 Alexey Proskuryakov 2008-07-21 02:30:49 PDT
I believe that the root cause for both bugs is that we're manipulating properties on wrong documents. In this bug, TSMGetActiveDocument() returns 0 when switching from a password field to a normal one, so we fail to remove kTSMDocumentEnabledInputSourcesPropertyTag. This is caused by a hack in -[WebHTMLView inputContext], and is fixed by removing it.

It's not obvious to me if bug 16946 is exactly the same, as there may be many reasons for an active document to be not what we expect. Ideally, we should find a less fragile solution.
Comment 6 Mark Rowe (bdash) 2008-08-15 04:07:45 PDT
*** Bug 20392 has been marked as a duplicate of this bug. ***
Comment 7 Alexey Proskuryakov 2008-08-18 09:35:54 PDT
Created attachment 22853 [details]
proposed fix
Comment 8 mitz 2008-08-18 09:43:53 PDT
Comment on attachment 22853 [details]
proposed fix

Cool! r=me

+    Frame* coreFrame = core([self _frame]);

You can use the local variable frame from the line above.

Can you now remove
@interface NSResponder (AppKitDetails)
from WebHTMLView.mm?
Comment 9 Alexey Proskuryakov 2008-08-19 00:49:05 PDT
Committed revision 35831. I wonder how this affects bug 16946.

(In reply to comment #8)
> Can you now remove
> @interface NSResponder (AppKitDetails)
> from WebHTMLView.mm?

I can't - the method is used in -[WebHTMLView mouseDown:]