<?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>18364</bug_id>
          
          <creation_ts>2008-04-08 07:19:11 -0700</creation_ts>
          <short_desc>[GTK] Buttons activated with the keyboard should be painted as clicked</short_desc>
          <delta_ts>2010-10-11 15:44:59 -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>WebKitGTK</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</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>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Marco Barisione">marco.barisione</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>abarth</cc>
    
    <cc>eric</cc>
    
    <cc>jmalonzo</cc>
    
    <cc>mrobinson</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>76606</commentid>
    <comment_count>0</comment_count>
    <who name="Marco Barisione">marco.barisione</who>
    <bug_when>2008-04-08 07:19:11 -0700</bug_when>
    <thetext>When activating a button with the keyboard it should be painted as pressed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>95868</commentid>
    <comment_count>1</comment_count>
    <who name="Jan Alonzo">jmalonzo</who>
    <bug_when>2008-10-18 04:19:39 -0700</bug_when>
    <thetext>marco, pressed or focused? can you please confirm if this doesn&apos;t occur in ToT yet? 

cheersm</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>292007</commentid>
    <comment_count>2</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2010-10-08 18:31:20 -0700</bug_when>
    <thetext>I can confirm that tabbing to a button and pressing space does not cause the button to be shown as pressed. Compare the behavior that happens in Chromium or Firefox.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>292155</commentid>
    <comment_count>3</comment_count>
      <attachid>70360</attachid>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2010-10-09 12:24:57 -0700</bug_when>
    <thetext>Created attachment 70360
Patch for this issue</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>292645</commentid>
    <comment_count>4</comment_count>
      <attachid>70360</attachid>
    <who name="Gustavo Noronha (kov)">gustavo</who>
    <bug_when>2010-10-11 14:14:00 -0700</bug_when>
    <thetext>Comment on attachment 70360
Patch for this issue

Cunning.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>292650</commentid>
    <comment_count>5</comment_count>
      <attachid>70360</attachid>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2010-10-11 14:21:00 -0700</bug_when>
    <thetext>Comment on attachment 70360
Patch for this issue

Clearing flags on attachment: 70360

Committed r69526: &lt;http://trac.webkit.org/changeset/69526&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>292651</commentid>
    <comment_count>6</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2010-10-11 14:21:04 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>292721</commentid>
    <comment_count>7</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2010-10-11 15:44:59 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/69526 might have broken GTK Linux 64-bit Debug
The following tests are not passing:
editing/selection/move-past-trailing-space.html
fast/css/shadow-multiple.html
fast/text/shadow-no-blur.html</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>70360</attachid>
            <date>2010-10-09 12:24:57 -0700</date>
            <delta_ts>2010-10-11 14:21:00 -0700</delta_ts>
            <desc>Patch for this issue</desc>
            <filename>bug-18364-20101009122456.patch</filename>
            <type>text/plain</type>
            <size>2474</size>
            <attacher name="Martin Robinson">mrobinson</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg
OWQzMTA2NDhiYThiMGY0OGQyN2FjNDcyZjExNWRmZjU2NmExODRlMS4uYjFjOTdkMGQxZjllY2U0
MWNlZmNhYmZmMWViMWFlNThjMGUzNDc3NSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cK
KysrIGIvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsNSArMSwyNCBAQAogMjAxMC0xMC0wOSAgTWFy
dGluIFJvYmluc29uICA8bXJvYmluc29uQGlnYWxpYS5jb20+CiAKKyAgICAgICAgUmV2aWV3ZWQg
YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgW0dUS10gQnV0dG9ucyBhY3RpdmF0ZWQgd2l0
aCB0aGUga2V5Ym9hcmQgc2hvdWxkIGJlIHBhaW50ZWQgYXMgY2xpY2tlZAorICAgICAgICBodHRw
czovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTgzNjQKKworICAgICAgICBXaGVu
IGEgYnV0dG9uIGlzIHByZXNzZWQsIGZvcmNlIGl0IGludG8gdGhlIGhvdmVyZWQgc3RhdGUgYXMg
ZmFyIGFzIHRoZQorICAgICAgICBNb3ppbGxhIHRoZW1lIGRyYXdpbmcgY29kZSBpcyBjb25jZXJu
ZWQuIFRoaXMgYnVnIGFsc28gc2VlbXMgdG8gYmUgYW4KKyAgICAgICAgaXNzdWUgZm9yIEZpcmVm
b3guIFdoZW4gd2Ugc3dpdGNoIGF3YXkgZnJvbSB0aGUgTW96aWxsYSB0aGVtZSBkcmF3aW5nCisg
ICAgICAgIGNvZGUgdGhpcyB3b3JrLWFyb3VuZCBzaG91bGQgbm8gbG9uZ2VyIGJlIG5lY2Vzc2Fy
eS4KKworICAgICAgICBObyBuZXcgdGVzdHMuIEkgd2FzIG5vdCBhYmxlIHRvIGdlbmVyYXRlIGEg
cGl4ZWwgdGVzdCBmb3IgdGhpcyBpc3N1ZS4gSXQKKyAgICAgICAgc2VlbXMgdGhhdCBvdXIgRFJU
IGRvZXMgbm90IGhhdmUgdGhlIG5lY2Vzc2FyeSBpbmZyYXN0cnVjdHVyZSB0byBnZW5lcmF0ZSBh
CisgICAgICAgIHBpeGVsIGR1bXAgd2hpbGUgYSBidXR0b24gaXMgaW4gdGhlIHByZXNzZWQgc3Rh
dGUuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ndGsvUmVuZGVyVGhlbWVHdGsuY3BwOgorICAgICAg
ICAoV2ViQ29yZTo6UmVuZGVyVGhlbWVHdGs6OnBhaW50UmVuZGVyT2JqZWN0KToKKworMjAxMC0x
MC0wOSAgTWFydGluIFJvYmluc29uICA8bXJvYmluc29uQGlnYWxpYS5jb20+CisKICAgICAgICAg
UmV2aWV3ZWQgYnkgWGFuIExvcGV6LgogCiAgICAgICAgIFtHdGtdIE5vbi1yZWN0YW5ndWxhciBi
dXR0b25zIGhhdmUgYSBzcXVhcmUgYmFja2dyb3VuZCBiZWhpbmQgdGhlbQpkaWZmIC0tZ2l0IGEv
V2ViQ29yZS9wbGF0Zm9ybS9ndGsvUmVuZGVyVGhlbWVHdGsuY3BwIGIvV2ViQ29yZS9wbGF0Zm9y
bS9ndGsvUmVuZGVyVGhlbWVHdGsuY3BwCmluZGV4IGU2NDdlZWU5Y2YxY2I0Mjg0ZjdiMmIyYWJi
ODk0YWNjOTA2YTNhZTAuLmFmOTNlYjZkMGM3NTZjYjhiYTRjN2UyZDU1NjRjNTY0MmUzNmNmY2Yg
MTAwNjQ0Ci0tLSBhL1dlYkNvcmUvcGxhdGZvcm0vZ3RrL1JlbmRlclRoZW1lR3RrLmNwcAorKysg
Yi9XZWJDb3JlL3BsYXRmb3JtL2d0ay9SZW5kZXJUaGVtZUd0ay5jcHAKQEAgLTM2MSw3ICszNjEs
MTMgQEAgYm9vbCBSZW5kZXJUaGVtZUd0azo6cGFpbnRSZW5kZXJPYmplY3QoR3RrVGhlbWVXaWRn
ZXRUeXBlIHR5cGUsIFJlbmRlck9iamVjdCogcmUKICAgICBHdGtXaWRnZXRTdGF0ZSB3aWRnZXRT
dGF0ZTsKICAgICB3aWRnZXRTdGF0ZS5hY3RpdmUgPSBpc1ByZXNzZWQocmVuZGVyT2JqZWN0KTsK
ICAgICB3aWRnZXRTdGF0ZS5mb2N1c2VkID0gaXNGb2N1c2VkKHJlbmRlck9iamVjdCk7Ci0gICAg
d2lkZ2V0U3RhdGUuaW5Ib3ZlciA9IGlzSG92ZXJlZChyZW5kZXJPYmplY3QpOworCisgICAgLy8g
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE4MzY0CisgICAgLy8gVGhl
IE1vemlsbGEgdGhlbWUgZHJhd2luZyBjb2RlLCBvbmx5IHBhaW50cyBhIGJ1dHRvbiBhcyBwcmVz
c2VkIHdoZW4gaXQncyBwcmVzc2VkIAorICAgIC8vIHdoaWxlIGhvdmVyZWQuIFVudGlsIHdlIG1v
dmUgYXdheSBmcm9tIHRoZSBNb3ppbGEgY29kZSwgd29yay1hcm91bmQgdGhlIGlzc3VlIGJ5Cisg
ICAgLy8gZm9yY2luZyBhIHByZXNzZWQgYnV0dG9uIGludG8gdGhlIGhvdmVyZWQgc3RhdGUuIFRo
aXMgZW5zdXJlcyB0aGF0IGJ1dHRvbnMgYWN0aXZhdGVkCisgICAgLy8gdmlhIHRoZSBrZXlib2Fy
ZCBoYXZlIHRoZSBwcm9wZXIgcmVuZGVyaW5nLgorICAgIHdpZGdldFN0YXRlLmluSG92ZXIgPSBp
c0hvdmVyZWQocmVuZGVyT2JqZWN0KSB8fCAodHlwZSA9PSBNT1pfR1RLX0JVVFRPTiAmJiBpc1By
ZXNzZWQocmVuZGVyT2JqZWN0KSk7CiAKICAgICAvLyBGSVhNRTogRGlzYWJsZWQgZG9lcyBub3Qg
YWx3YXlzIGdpdmUgdGhlIGNvcnJlY3QgYXBwZWFyYW5jZSBmb3IgUmVhZE9ubHkKICAgICB3aWRn
ZXRTdGF0ZS5kaXNhYmxlZCA9ICFpc0VuYWJsZWQocmVuZGVyT2JqZWN0KSB8fCBpc1JlYWRPbmx5
Q29udHJvbChyZW5kZXJPYmplY3QpOwo=
</data>

          </attachment>
      

    </bug>

</bugzilla>