<?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>174610</bug_id>
          
          <creation_ts>2017-07-17 21:58:49 -0700</creation_ts>
          <short_desc>Update layout test fast/forms/content-with-margins-inside-button.html to work when button has a default color</short_desc>
          <delta_ts>2017-08-02 04:57:17 -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>Other</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>P5</priority>
          <bug_severity>Minor</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Michael Catanzaro">mcatanzaro</reporter>
          <assigned_to name="Ms2ger (he/him; ⌚ UTC+1/+2)">Ms2ger</assigned_to>
          <cc>bugs-noreply</cc>
    
    <cc>commit-queue</cc>
    
    <cc>darin</cc>
    
    <cc>Ms2ger</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1329883</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2017-07-17 21:58:49 -0700</bug_when>
    <thetext>Layout test fast/forms/content-with-margins-inside-button.html is failing for GTK since, most likely, r217674 (&quot;[GTK] Bump GTK+ and its dependencies in jhbuild&quot;). However it cannot be simply rebaselined since it is a reftest. The text color inside the button no longer matches. Updating expectations accordingly.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1333692</commentid>
    <comment_count>1</comment_count>
      <attachid>316838</attachid>
    <who name="Ms2ger (he/him; ⌚ UTC+1/+2)">Ms2ger</who>
    <bug_when>2017-08-01 03:03:35 -0700</bug_when>
    <thetext>Created attachment 316838
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1333761</commentid>
    <comment_count>2</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2017-08-01 08:49:47 -0700</bug_when>
    <thetext>I don’t wish to be pedantic, but I believe that a bug like this one should not have the [GTK] prefix. We try to use those prefixes for changes that are truly specific to a particular port, changes that can’t possibly affect other ports, as opposed to patches that are motivated by a problem that shows only on one port, but are changes to cross-platform code.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1333820</commentid>
    <comment_count>3</comment_count>
      <attachid>316838</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-08-01 09:42:21 -0700</bug_when>
    <thetext>Comment on attachment 316838
Patch

Clearing flags on attachment: 316838

Committed r220104: &lt;http://trac.webkit.org/changeset/220104&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1333821</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-08-01 09:42:23 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1334195</commentid>
    <comment_count>5</comment_count>
    <who name="Ms2ger (he/him; ⌚ UTC+1/+2)">Ms2ger</who>
    <bug_when>2017-08-02 04:57:17 -0700</bug_when>
    <thetext>(In reply to Darin Adler from comment #2)
&gt; I don’t wish to be pedantic, but I believe that a bug like this one should
&gt; not have the [GTK] prefix. We try to use those prefixes for changes that are
&gt; truly specific to a particular port, changes that can’t possibly affect
&gt; other ports, as opposed to patches that are motivated by a problem that
&gt; shows only on one port, but are changes to cross-platform code.

Agreed; changed the summary to match what I did.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>316838</attachid>
            <date>2017-08-01 03:03:35 -0700</date>
            <delta_ts>2017-08-01 09:42:21 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-174610-20170801120334.patch</filename>
            <type>text/plain</type>
            <size>3668</size>
            <attacher name="Ms2ger (he/him; ⌚ UTC+1/+2)">Ms2ger</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjIwMDU0CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu
Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggYTEyYTBhNGUwZjkyNzU0NGU1NTNk
NzU0ZTc0YjkwNzVhODBlYWIwNS4uYjc5ZTMyMjU2Njg2ZmQzM2NhODk0MmJmNTE1YjUyNDg5MzZm
M2MyNSAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz
L0NoYW5nZUxvZwpAQCAtMSw1ICsxLDIyIEBACiAyMDE3LTA4LTAxICBNczJnZXIgIDxNczJnZXJA
aWdhbGlhLmNvbT4KIAorICAgICAgICBbR1RLXSBMYXlvdXQgdGVzdCBmYXN0L2Zvcm1zL2NvbnRl
bnQtd2l0aC1tYXJnaW5zLWluc2lkZS1idXR0b24uaHRtbCBpcyBmYWlsaW5nCisgICAgICAgIGh0
dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNzQ2MTAKKworICAgICAgICBS
ZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBVc2UgYW4gZXhwbGljaXQgdGV4
dCBjb2xvciBmb3IgY29udGVudC13aXRoLW1hcmdpbnMtaW5zaWRlLWJ1dHRvbi5odG1sLgorCisg
ICAgICAgIFRoaXMgbWFrZXMgdGhlIHRlc3QgcGFzcyBpbiB0aGUgR1RLIHBvcnQuCisKKyAgICAg
ICAgSSBhbHNvIGNoYW5nZWQgdGhlIGZvcm1hdHRpbmcgb2YgdGhlIENTUyB0byBtYWtlIGl0IG1v
cmUgcmVhZGFibGUuCisKKyAgICAgICAgKiBmYXN0L2Zvcm1zL2NvbnRlbnQtd2l0aC1tYXJnaW5z
LWluc2lkZS1idXR0b24tZXhwZWN0ZWQuaHRtbDoKKyAgICAgICAgKiBmYXN0L2Zvcm1zL2NvbnRl
bnQtd2l0aC1tYXJnaW5zLWluc2lkZS1idXR0b24uaHRtbDoKKyAgICAgICAgKiBwbGF0Zm9ybS9n
dGsvVGVzdEV4cGVjdGF0aW9uczoKKworMjAxNy0wOC0wMSAgTXMyZ2VyICA8TXMyZ2VyQGlnYWxp
YS5jb20+CisKICAgICAgICAgUmVtb3ZlIG9ic29sZXRlIGV4cGVjdGF0aW9ucyBmb3Igc3ludGF4
LTAyMS54bWwuCiAgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p
ZD04NjE0MgogCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9mYXN0L2Zvcm1zL2NvbnRlbnQtd2l0
aC1tYXJnaW5zLWluc2lkZS1idXR0b24tZXhwZWN0ZWQuaHRtbCBiL0xheW91dFRlc3RzL2Zhc3Qv
Zm9ybXMvY29udGVudC13aXRoLW1hcmdpbnMtaW5zaWRlLWJ1dHRvbi1leHBlY3RlZC5odG1sCmlu
ZGV4IDQ0NjIzZWM2MjYxNzA3MjUzNmI4MGFjMmI0YjMzMjdiZGFhYWFhNGMuLjgwY2ZjZjYxMzRm
OTZjNDc0M2ZiZWZjYTE5MTQwMjkwY2JhNWI1ZDUgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3RzL2Zh
c3QvZm9ybXMvY29udGVudC13aXRoLW1hcmdpbnMtaW5zaWRlLWJ1dHRvbi1leHBlY3RlZC5odG1s
CisrKyBiL0xheW91dFRlc3RzL2Zhc3QvZm9ybXMvY29udGVudC13aXRoLW1hcmdpbnMtaW5zaWRl
LWJ1dHRvbi1leHBlY3RlZC5odG1sCkBAIC02LDcgKzYsMTYgQEAKICAgICBtYXJnaW4tYm90dG9t
OiAxMDBweDsKICAgfQogICAKLSAgLm91dGVyIHsgdGV4dC1hbGlnbjpjZW50ZXI7IGZsb2F0OiBs
ZWZ0OyBiYWNrZ3JvdW5kLWNvbG9yOiAjY2NjY2NjOyBib3JkZXI6MXB4IHNvbGlkIGJsYWNrOyBw
YWRkaW5nOjA7IGZvbnQtZmFtaWx5OiBUaW1lczsgZm9udC1zaXplOjE2cHggfQorICAub3V0ZXIg
eworICAgIGZsb2F0OiBsZWZ0OworICAgIGJhY2tncm91bmQtY29sb3I6ICNjY2NjY2M7CisgICAg
Ym9yZGVyOiAxcHggc29saWQgYmxhY2s7CisgICAgcGFkZGluZzogMDsKKyAgICBmb250LWZhbWls
eTogVGltZXM7CisgICAgZm9udC1zaXplOiAxNnB4OworICAgIGNvbG9yOiBibGFjazsKKyAgICB0
ZXh0LWFsaWduOiBjZW50ZXI7CisgIH0KIDwvc3R5bGU+CiAKLTxkaXYgY2xhc3M9Im91dGVyIj48
ZGl2IGNsYXNzPSJpbm5lciI+Q29udGVudDwvZGl2PjwvYnV0dG9uPgpcIE5vIG5ld2xpbmUgYXQg
ZW5kIG9mIGZpbGUKKzxkaXYgY2xhc3M9Im91dGVyIj48ZGl2IGNsYXNzPSJpbm5lciI+Q29udGVu
dDwvZGl2PjwvYnV0dG9uPgpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvZmFzdC9mb3Jtcy9jb250
ZW50LXdpdGgtbWFyZ2lucy1pbnNpZGUtYnV0dG9uLmh0bWwgYi9MYXlvdXRUZXN0cy9mYXN0L2Zv
cm1zL2NvbnRlbnQtd2l0aC1tYXJnaW5zLWluc2lkZS1idXR0b24uaHRtbAppbmRleCBkMmY2YWI0
ODNhMTU1YjBhMjgyMTQ0YTA3NGM1YzUzYTczYmU1YWNkLi5jZDkxMzJjNDk4ZjhhNDZiNDMyYTk1
ZDJkOTkyNzU3YjM0YTllNjczIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9mYXN0L2Zvcm1zL2Nv
bnRlbnQtd2l0aC1tYXJnaW5zLWluc2lkZS1idXR0b24uaHRtbAorKysgYi9MYXlvdXRUZXN0cy9m
YXN0L2Zvcm1zL2NvbnRlbnQtd2l0aC1tYXJnaW5zLWluc2lkZS1idXR0b24uaHRtbApAQCAtNiw3
ICs2LDE2IEBACiAgICAgbWFyZ2luLWJvdHRvbTogMTAwcHg7CiAgIH0KICAgCi0gIGJ1dHRvbiB7
IGZsb2F0OmxlZnQ7IG1hcmdpbjowOyBiYWNrZ3JvdW5kLWNvbG9yOiAjY2NjY2NjOyBib3JkZXI6
MXB4IHNvbGlkIGJsYWNrOyBwYWRkaW5nOjA7IGZvbnQtZmFtaWx5OiBUaW1lczsgZm9udC1zaXpl
OjE2cHggfQorICBidXR0b24geworICAgIGZsb2F0OiBsZWZ0OworICAgIGJhY2tncm91bmQtY29s
b3I6ICNjY2NjY2M7CisgICAgYm9yZGVyOiAxcHggc29saWQgYmxhY2s7CisgICAgcGFkZGluZzog
MDsKKyAgICBmb250LWZhbWlseTogVGltZXM7CisgICAgZm9udC1zaXplOiAxNnB4OworICAgIGNv
bG9yOiBibGFjazsKKyAgICBtYXJnaW46IDA7CisgIH0KIDwvc3R5bGU+CiAKLTxidXR0b24+PGRp
dj5Db250ZW50PC9kaXY+PC9idXR0b24+ClwgTm8gbmV3bGluZSBhdCBlbmQgb2YgZmlsZQorPGJ1
dHRvbj48ZGl2PkNvbnRlbnQ8L2Rpdj48L2J1dHRvbj4KZGlmZiAtLWdpdCBhL0xheW91dFRlc3Rz
L3BsYXRmb3JtL2d0ay9UZXN0RXhwZWN0YXRpb25zIGIvTGF5b3V0VGVzdHMvcGxhdGZvcm0vZ3Rr
L1Rlc3RFeHBlY3RhdGlvbnMKaW5kZXggYmQyZmVkZGM5NWUxMTk0NGM3ZjQzNzFiN2MzZDgzZTE0
MzIwYmRlZS4uMjhlMTlmMmFhMWRlNTNmMGVkOThiNWJkZjg4ZDVlNWU2NzE5NjdmMCAxMDA2NDQK
LS0tIGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vZ3RrL1Rlc3RFeHBlY3RhdGlvbnMKKysrIGIvTGF5
b3V0VGVzdHMvcGxhdGZvcm0vZ3RrL1Rlc3RFeHBlY3RhdGlvbnMKQEAgLTM0MDksOCArMzQwOSw2
IEBAIHdlYmtpdC5vcmcvYi8xNzQ0NjEgaW5zcGVjdG9yL2NhbnZhcy9yZXF1ZXN0Q29udGVudC13
ZWJnbC5odG1sIFsgRmFpbHVyZSBdCiAKIHdlYmtpdC5vcmcvYi8xNzQ2MDkgYWNjZXNzaWJpbGl0
eS9wcmVzZW50YXRpb24tcm9sZS1pZnJhbWUuaHRtbCBbIEZhaWx1cmUgXQogCi13ZWJraXQub3Jn
L2IvMTc0NjEwIGZhc3QvZm9ybXMvY29udGVudC13aXRoLW1hcmdpbnMtaW5zaWRlLWJ1dHRvbi5o
dG1sIFsgRmFpbHVyZSBdCi0KICMvLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8v
Ly8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vCiAjIEVu
ZCBvZiBub24tY3Jhc2hpbmcsIG5vbi1mbGFreSB0ZXN0cyBmYWlsaW5nCiAjLy8vLy8vLy8vLy8v
Ly8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8v
Ly8vLy8vLy8vLy8vLy8vLy8vLwo=
</data>

          </attachment>
      

    </bug>

</bugzilla>