<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>16946</bug_id>
          
          <creation_ts>2008-01-20 00:53:03 -0800</creation_ts>
          <short_desc>Non-ASCII keyboard layouts are sometimes available in password fields and vice versa</short_desc>
          <delta_ts>2009-06-26 12:47:08 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit Misc.</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>WORKSFORME</resolution>
          
          
          <bug_file_loc>http://www.youtube.com/</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>HasReduction, InRadar</keywords>
          <priority>P1</priority>
          <bug_severity>Major</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>mitz</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ap</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>67775</commentid>
    <comment_count>0</comment_count>
    <who name="">mitz</who>
    <bug_when>2008-01-20 00:53:03 -0800</bug_when>
    <thetext>Secure text entry mode is off in the password field on YouTube, but on in the search field.

Steps to reproduce:
0) In System Preferences &gt; International enable a non-Latin keyboard layout, such as Hebrew.
1) Go to http://www.youtube.com/
2) If you are logged in to your YouTube account, log out and go back to step 1)
3) Press Tab twice to focus the password field
4) Open the Input (&quot;flag&quot;) menu and see if Hebrew is enabled
5) Press Tab again to focus the search field
6) Open the Input menu again and see if Hebrew is enabled

Results:
Hebrew is enabled when the password field is focused but not when the search field is focused. It should be the other way around.

Regression:
Safari 3.0.4 behaves correctly. I suspect &lt;http://trac.webkit.org/projects/webkit/changeset/29581&gt;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>67776</commentid>
    <comment_count>1</comment_count>
    <who name="">mitz</who>
    <bug_when>2008-01-20 00:54:13 -0800</bug_when>
    <thetext>&lt;rdar://problem/5696689&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>67777</commentid>
    <comment_count>2</comment_count>
      <attachid>18563</attachid>
    <who name="">mitz</who>
    <bug_when>2008-01-20 00:59:21 -0800</bug_when>
    <thetext>Created attachment 18563
Test case</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>67778</commentid>
    <comment_count>3</comment_count>
    <who name="">mitz</who>
    <bug_when>2008-01-20 01:22:33 -0800</bug_when>
    <thetext>Using bisect-builds on the test case I get:
Works: r27021  Fails: r27031
which makes no sense.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>67789</commentid>
    <comment_count>4</comment_count>
    <who name="">mitz</who>
    <bug_when>2008-01-20 14:05:50 -0800</bug_when>
    <thetext>I am now able to reproduce the bug occasionally with even earlier revisions, so I am not sure that this ever worked correctly on Leopard.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>67791</commentid>
    <comment_count>5</comment_count>
      <attachid>18568</attachid>
    <who name="">mitz</who>
    <bug_when>2008-01-20 14:24:04 -0800</bug_when>
    <thetext>Created attachment 18568
Test case

Added the ability to see what you type into the password field.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>67792</commentid>
    <comment_count>6</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2008-01-20 14:38:52 -0800</bug_when>
    <thetext>I could reproduce on 10.5.1 with shipping Safari/WebKit.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>67793</commentid>
    <comment_count>7</comment_count>
    <who name="">mitz</who>
    <bug_when>2008-01-20 14:41:15 -0800</bug_when>
    <thetext>Using the debugger I can see that WebKit is making the right calls, so I suspect the problem is that TSMSetDocumentProperty() and TSMRemoveDocumentProperty() do not always behave as expected. With Alexey&apos;s help we have discovered that there are also differences in reproducibility (and ability to type non-ASCII text into the password field) depending on whether you use the Input menu or keyboard shortcuts to switch between input methods.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>86541</commentid>
    <comment_count>8</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2008-07-21 02:34:25 -0700</bug_when>
    <thetext>I think that we&apos;re making the right calls, but with wrong arguments, as TSMGetActiveDocument() gives us unexpected results. The bug is somewhat confusing, as there are different behaviors depending on how you switch between input methods.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>88777</commentid>
    <comment_count>9</comment_count>
    <who name="">mitz</who>
    <bug_when>2008-08-18 17:34:27 -0700</bug_when>
    <thetext>Alexey, does your patch for bug 19347 fix this one too?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>88801</commentid>
    <comment_count>10</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2008-08-19 01:04:30 -0700</bug_when>
    <thetext>AFAICT, the original steps to reproduce work correctly now.

However, there is still at least one case when secure input is not enabled, maybe we need a new bug for it:
1. Focus a password field.
2. Switch to Finder.
3. Switch back to Safari by clicking in its window.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>128541</commentid>
    <comment_count>11</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2009-06-26 12:47:08 -0700</bug_when>
    <thetext>Looks like this is all fixed in ToT as of r44941.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="0"
              isprivate="0"
          >
            <attachid>18563</attachid>
            <date>2008-01-20 00:59:21 -0800</date>
            <delta_ts>2008-01-20 14:24:04 -0800</delta_ts>
            <desc>Test case</desc>
            <filename>19468 reduction.html</filename>
            <type>text/html</type>
            <size>189</size>
            <attacher>mitz</attacher>
            
              <data encoding="base64">PHA+CiAgICBUYWIgaW50byB0aGlzIGZpZWxkOiA8aW5wdXQgdHlwZT0icGFzc3dvcmQiPiAoc2Vj
dXJlIGtleWJvYXJkIGVudHJ5IG1vZGUgc2hvdWxkIGJlIG9uKQo8L3A+CjxwPgogICAgYW5kIG5v
dyBpbnRvIHRoaXMgZmllbGQ6IDxpbnB1dD4gKHNlY3VyZSBrZXlib2FyZCBlbnRyeSBtb2RlIHNo
b3VsZCBiZSBvZmYpCjwvcD4K
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>18568</attachid>
            <date>2008-01-20 14:24:04 -0800</date>
            <delta_ts>2008-01-20 14:24:04 -0800</delta_ts>
            <desc>Test case</desc>
            <filename>16946 reduction.html</filename>
            <type>text/html</type>
            <size>279</size>
            <attacher>mitz</attacher>
            
              <data encoding="base64">PHA+CiAgICBUYWIgaW50byB0aGlzIGZpZWxkOiA8aW5wdXQgdHlwZT0icGFzc3dvcmQiIG9uaW5w
dXQ9InR5cGVkVGV4dC5pbm5lclRleHQgPSB2YWx1ZTsiPiAoc2VjdXJlIGtleWJvYXJkIGVudHJ5
IG1vZGUgc2hvdWxkIGJlIG9uOyB2YWx1ZTogJmxkcXVvOzxzcGFuIGlkPSJ0eXBlZFRleHQiPjwv
c3Bhbj4mcmRxdW87KQo8L3A+CjxwPgogICAgYW5kIG5vdyBpbnRvIHRoaXMgZmllbGQ6IDxpbnB1
dD4gKHNlY3VyZSBrZXlib2FyZCBlbnRyeSBtb2RlIHNob3VsZCBiZSBvZmYpCjwvcD4K
</data>

          </attachment>
      

    </bug>

</bugzilla>