[Webkit-unassigned] [Bug 13471] REGRESSION (r21045): Secure keyboard entry mode remains in effect after leaving a password field by submitting

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=13471


mitz at webkit.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #14157|review+                     |
               Flag|                            |
  Attachment #14157|0                           |1
        is obsolete|                            |




------- Comment #4 from mitz at webkit.org  2007-04-24 08:41 PDT -------
(From update of attachment 14157)
(In reply to comment #3)
> (From update of attachment 14157 [edit])
> Why not spell it "submit" everywhere?

Why not say "you made a typo"? ;-)

> What's the key reason this needs to be called? Is it really the change in view
> that's the issue? Doesn't the input element lose focus?

In many senses, it never does (so I think that even prior to r21045, if you
happened to be in a secure text field when the new page was loaded, you'd
remain in Roman-only mode). The Frame doesn't keep track of focus, it just
serves requests from nodes to switch secure mode on or off. Presumably, those
nodes are in the Frame's Document. So now I'm thinking it may be more
appropriate to patch setDocument() instead of setView(). It would still just
reset to false unconditionally, since it can't (and doesn't need to) ask the
new document's focused node if it requires secure keyboard entry. However, that
could be changed in the future (if password fields can go into the page cache).


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.