<?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>51173</bug_id>
          
          <creation_ts>2010-12-16 02:51:52 -0800</creation_ts>
          <short_desc>[GTK] Buttons rendering was broken in r74129</short_desc>
          <delta_ts>2010-12-17 18:37:52 -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>WebKitGTK</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</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="Carlos Garcia Campos">cgarcia</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>commit-queue</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>323088</commentid>
    <comment_count>0</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2010-12-16 02:51:52 -0800</bug_when>
    <thetext>-    // FIXME: The depressed value should probably apply for other theme parts too.
-    // It must be used for range thumbs, because otherwise when the thumb is pressed,
-    // the rendering is incorrect.
-    if (type == MOZ_GTK_SCALE_THUMB_HORIZONTAL || type == MOZ_GTK_SCALE_THUMB_VERTICAL)
-        widgetState.depressed = isPressed(renderObject);
-    else
-        widgetState.depressed = false;
-

We still need to set the depressed state for other widget types</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>323089</commentid>
    <comment_count>1</comment_count>
      <attachid>76747</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2010-12-16 02:53:59 -0800</bug_when>
    <thetext>Created attachment 76747
Patch to fix the problem</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>323209</commentid>
    <comment_count>2</comment_count>
      <attachid>76747</attachid>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2010-12-16 09:20:22 -0800</bug_when>
    <thetext>Comment on attachment 76747
Patch to fix the problem

Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>323251</commentid>
    <comment_count>3</comment_count>
      <attachid>76747</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-12-16 10:23:53 -0800</bug_when>
    <thetext>Comment on attachment 76747
Patch to fix the problem

Rejecting attachment 76747 from commit-queue.

Failed to run &quot;[&apos;./WebKitTools/Scripts/webkit-patch&apos;, &apos;--status-host=queues.webkit.org&apos;, &apos;--bot-id=cr-jail-3&apos;, &apos;apply-attachment&apos;, &apos;--non-interactive&apos;, 76747]&quot; exit_code: 2
Last 500 characters of output:
ailed to merge in the changes.
Patch failed at 0001 2010-12-16  Yury Semikhatsky  &lt;yurys@chromium.org&gt;

When you have resolved this problem run &quot;git rebase --continue&quot;.
If you would prefer to skip this patch, instead run &quot;git rebase --skip&quot;.
To restore the original branch and stop rebasing run &quot;git rebase --abort&quot;.

rebase refs/remotes/origin/master: command returned error: 1

Died at WebKitTools/Scripts/update-webkit line 132.

Failed to run &quot;[&apos;WebKitTools/Scripts/update-webkit&apos;]&quot; exit_code: 2

Full output: http://queues.webkit.org/results/7101076</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>323294</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-12-16 11:04:13 -0800</bug_when>
    <thetext>The commit-queue encountered the following flaky tests while processing attachment 76747:

fast/dom/onerror-img.html bug 51019
animations/suspend-resume-animation.html bug 48161 (author: cmarrin@apple.com)
The commit-queue is continuing to process your patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>324071</commentid>
    <comment_count>5</comment_count>
      <attachid>76747</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-12-17 18:37:47 -0800</bug_when>
    <thetext>Comment on attachment 76747
Patch to fix the problem

Clearing flags on attachment: 76747

Committed r74311: &lt;http://trac.webkit.org/changeset/74311&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>324072</commentid>
    <comment_count>6</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-12-17 18:37:52 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>76747</attachid>
            <date>2010-12-16 02:53:59 -0800</date>
            <delta_ts>2010-12-17 18:37:47 -0800</delta_ts>
            <desc>Patch to fix the problem</desc>
            <filename>fix-buttons.diff</filename>
            <type>text/plain</type>
            <size>1250</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg
YWI5ZGYyZS4uMjQxZDY1ZCAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi
Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMyBAQAorMjAxMC0xMi0xNiAgQ2FybG9zIEdhcmNp
YSBDYW1wb3MgIDxjZ2FyY2lhQGlnYWxpYS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9C
T0RZIChPT1BTISkuCisKKyAgICAgICAgW0dUS10gQnV0dG9ucyByZW5kZXJpbmcgd2FzIGJyb2tl
biBpbiByNzQxMjkKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp
P2lkPTUxMTczCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ndGsvUmVuZGVyVGhlbWVHdGsuY3BwOgor
ICAgICAgICAoV2ViQ29yZTo6UmVuZGVyVGhlbWVHdGs6OnBhaW50UmVuZGVyT2JqZWN0KToKKwog
MjAxMC0xMi0xNSAgQnJpYW4gV2VpbnN0ZWluICA8YndlaW5zdGVpbkBhcHBsZS5jb20+CiAKICAg
ICAgICAgUmV2aWV3ZWQgYnkgSm9uIEhvbmV5Y3V0dC4KZGlmZiAtLWdpdCBhL1dlYkNvcmUvcGxh
dGZvcm0vZ3RrL1JlbmRlclRoZW1lR3RrLmNwcCBiL1dlYkNvcmUvcGxhdGZvcm0vZ3RrL1JlbmRl
clRoZW1lR3RrLmNwcAppbmRleCBmNzMyYzE2Li41YWVlOTViIDEwMDY0NAotLS0gYS9XZWJDb3Jl
L3BsYXRmb3JtL2d0ay9SZW5kZXJUaGVtZUd0ay5jcHAKKysrIGIvV2ViQ29yZS9wbGF0Zm9ybS9n
dGsvUmVuZGVyVGhlbWVHdGsuY3BwCkBAIC0yOTgsNiArMjk4LDcgQEAgYm9vbCBSZW5kZXJUaGVt
ZUd0azo6cGFpbnRSZW5kZXJPYmplY3QoR3RrVGhlbWVXaWRnZXRUeXBlIHR5cGUsIFJlbmRlck9i
amVjdCogcmUKICAgICB3aWRnZXRTdGF0ZS5kaXNhYmxlZCA9ICFpc0VuYWJsZWQocmVuZGVyT2Jq
ZWN0KSB8fCBpc1JlYWRPbmx5Q29udHJvbChyZW5kZXJPYmplY3QpOwogICAgIHdpZGdldFN0YXRl
LmlzRGVmYXVsdCA9IGZhbHNlOwogICAgIHdpZGdldFN0YXRlLmNhbkRlZmF1bHQgPSBmYWxzZTsK
KyAgICB3aWRnZXRTdGF0ZS5kZXByZXNzZWQgPSBmYWxzZTsKIAogICAgIFdpZGdldFJlbmRlcmlu
Z0NvbnRleHQgd2lkZ2V0Q29udGV4dChjb250ZXh0LCByZWN0KTsKICAgICByZXR1cm4gIXdpZGdl
dENvbnRleHQucGFpbnRNb3ppbGxhV2lkZ2V0KHR5cGUsICZ3aWRnZXRTdGF0ZSwgZmxhZ3MsIGd0
a1RleHREaXJlY3Rpb24ocmVuZGVyT2JqZWN0LT5zdHlsZSgpLT5kaXJlY3Rpb24oKSkpOwo=
</data>

          </attachment>
      

    </bug>

</bugzilla>