<?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>24036</bug_id>
          
          <creation_ts>2009-02-19 11:51:07 -0800</creation_ts>
          <short_desc>Keyboard events need disambiguation on the Linux platform too.</short_desc>
          <delta_ts>2009-02-20 07:34:58 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Platform</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Other</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Enhancement</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Avi Drissman">avi</reporter>
          <assigned_to name="Avi Drissman">avi</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>110261</commentid>
    <comment_count>0</comment_count>
    <who name="Avi Drissman">avi</who>
    <bug_when>2009-02-19 11:51:07 -0800</bug_when>
    <thetext>Linux (in all its flavors) needs the same disambiguation of events as the Mac does (see plat/gtk/KeyEventGtk.cpp, plat/qt/PlatformKeyboardEventQt.cpp, and plat/wx/KeyboardEventWc.cpp).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>110263</commentid>
    <comment_count>1</comment_count>
      <attachid>27805</attachid>
    <who name="Avi Drissman">avi</who>
    <bug_when>2009-02-19 11:51:59 -0800</bug_when>
    <thetext>Created attachment 27805
Diff to provide disambiguation for all non-Windows platforms in Chromium, not just the Mac.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>110265</commentid>
    <comment_count>2</comment_count>
    <who name="Avi Drissman">avi</who>
    <bug_when>2009-02-19 11:52:27 -0800</bug_when>
    <thetext>See http://codereview.chromium.org/21514 .</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>110301</commentid>
    <comment_count>3</comment_count>
      <attachid>27805</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-02-19 14:27:33 -0800</bug_when>
    <thetext>Comment on attachment 27805
Diff to provide disambiguation for all non-Windows platforms in Chromium, not just the Mac.

Looks fine to me.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>110393</commentid>
    <comment_count>4</comment_count>
    <who name="Darin Fisher (:fishd, Google)">fishd</who>
    <bug_when>2009-02-20 07:34:58 -0800</bug_when>
    <thetext>Landed as http://trac.webkit.org/changeset/41106</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>27805</attachid>
            <date>2009-02-19 11:51:59 -0800</date>
            <delta_ts>2009-02-19 14:27:33 -0800</delta_ts>
            <desc>Diff to provide disambiguation for all non-Windows platforms in Chromium, not just the Mac.</desc>
            <filename>key.diff</filename>
            <type>text/plain</type>
            <size>1913</size>
            <attacher name="Avi Drissman">avi</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA0MTA3NykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTIgQEAKKzIwMDktMDItMTkgIEF2aSBEcmlzc21hbiAgPGF2aUBjaHJvbWl1bS5v
cmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgS2V5
Ym9hcmQgZXZlbnRzIG5lZWQgZGlzYW1iaWd1YXRpb24gb24gdGhlIExpbnV4IHBsYXRmb3JtIHRv
by4KKworICAgICAgICAqIHBsYXRmb3JtL2Nocm9taXVtL1BsYXRmb3JtS2V5Ym9hcmRFdmVudENo
cm9taXVtLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OlBsYXRmb3JtS2V5Ym9hcmRFdmVudDo6ZGlz
YW1iaWd1YXRlS2V5RG93bkV2ZW50KToKKwogMjAwOS0wMi0xOSAgR3VzdGF2byBOb3JvbmhhIFNp
bHZhICA8Z25zQGdub21lLm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBBbGV4ZXkgUHJvc2t1
cnlha292LgpJbmRleDogV2ViQ29yZS9wbGF0Zm9ybS9jaHJvbWl1bS9QbGF0Zm9ybUtleWJvYXJk
RXZlbnRDaHJvbWl1bS5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9wbGF0Zm9ybS9jaHJvbWl1
bS9QbGF0Zm9ybUtleWJvYXJkRXZlbnRDaHJvbWl1bS5jcHAJKHJldmlzaW9uIDQxMDc3KQorKysg
V2ViQ29yZS9wbGF0Zm9ybS9jaHJvbWl1bS9QbGF0Zm9ybUtleWJvYXJkRXZlbnRDaHJvbWl1bS5j
cHAJKHdvcmtpbmcgY29weSkKQEAgLTQyLDcgKzQyLDcgQEAgdm9pZCBQbGF0Zm9ybUtleWJvYXJk
RXZlbnQ6OmRpc2FtYmlndWF0ZQogI2lmIFBMQVRGT1JNKFdJTl9PUykKICAgICAvLyBObyBLZXlE
b3duIGV2ZW50cyBvbiBXaW5kb3dzIHRvIGRpc2FtYmlndWF0ZS4KICAgICBBU1NFUlRfTk9UX1JF
QUNIRUQoKTsKLSNlbGlmIFBMQVRGT1JNKERBUldJTikKKyNlbHNlCiAgICAgLy8gQ2FuIG9ubHkg
Y2hhbmdlIHR5cGUgZnJvbSBLZXlEb3duIHRvIFJhd0tleURvd24gb3IgQ2hhciwgYXMgd2UgbGFj
ayBpbmZvcm1hdGlvbiBmb3Igb3RoZXIgY29udmVyc2lvbnMuCiAgICAgQVNTRVJUKG1fdHlwZSA9
PSBLZXlEb3duKTsKICAgICBBU1NFUlQodHlwZSA9PSBSYXdLZXlEb3duIHx8IHR5cGUgPT0gQ2hh
cik7CkBAIC01Niw2ICs1Niw3IEBAIHZvaWQgUGxhdGZvcm1LZXlib2FyZEV2ZW50OjpkaXNhbWJp
Z3VhdGUKICAgICB9IGVsc2UgewogICAgICAgICBtX2tleUlkZW50aWZpZXIgPSBTdHJpbmcoKTsK
ICAgICAgICAgbV93aW5kb3dzVmlydHVhbEtleUNvZGUgPSAwOworI2lmIFBMQVRGT1JNKERBUldJ
TikKICAgICAgICAgaWYgKG1fdGV4dC5sZW5ndGgoKSA9PSAxICYmIChtX3RleHRbMFVdID49IDB4
RjcwMCAmJiBtX3RleHRbMFVdIDw9IDB4RjdGRikpIHsKICAgICAgICAgICAgIC8vIEFjY29yZGlu
ZyB0byBOU0V2ZW50cy5oLCBPcGVuU3RlcCByZXNlcnZlcyB0aGUgcmFuZ2UgMHhGNzAwLTB4RjhG
RiBmb3IgZnVuY3Rpb24ga2V5cy4gSG93ZXZlciwgc29tZSBhY3R1YWwgcHJpdmF0ZSB1c2UgY2hh
cmFjdGVycwogICAgICAgICAgICAgLy8gaGFwcGVuIHRvIGJlIGluIHRoaXMgcmFuZ2UsIGUuZy4g
dGhlIEFwcGxlIGxvZ28gKE9wdGlvbitTaGlmdCtLKS4KQEAgLTYzLDYgKzY0LDcgQEAgdm9pZCBQ
bGF0Zm9ybUtleWJvYXJkRXZlbnQ6OmRpc2FtYmlndWF0ZQogICAgICAgICAgICAgbV90ZXh0ID0g
U3RyaW5nKCk7CiAgICAgICAgICAgICBtX3VubW9kaWZpZWRUZXh0ID0gU3RyaW5nKCk7CiAgICAg
ICAgIH0KKyNlbmRpZgogICAgIH0KICNlbmRpZgogfQo=
</data>
<flag name="review"
          id="13520"
          type_id="1"
          status="+"
          setter="eric"
    />
          </attachment>
      

    </bug>

</bugzilla>