<?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>36200</bug_id>
          
          <creation_ts>2010-03-16 16:17:49 -0700</creation_ts>
          <short_desc>Temporarily disable WebGL layout tests failing due to multisampling</short_desc>
          <delta_ts>2010-03-16 16:39:26 -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>WebGL</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P1</priority>
          <bug_severity>Critical</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Kenneth Russell">kbr</reporter>
          <assigned_to name="Kenneth Russell">kbr</assigned_to>
          <cc>cmarrin</cc>
    
    <cc>eric</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>zmo</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>200494</commentid>
    <comment_count>0</comment_count>
    <who name="Kenneth Russell">kbr</who>
    <bug_when>2010-03-16 16:17:49 -0700</bug_when>
    <thetext>After the introduction of multisampling support for WebGL in https://bugs.webkit.org/show_bug.cgi?id=33416 , a few layout tests are failing on some of the commit bots. Detection of buggy OpenGL implementations and workarounds for them are being investigated under https://bugs.webkit.org/show_bug.cgi?id=36194 , but in the interim to unblock the commit queue we need to disable these tests. (The overall code changes in 33416 were blocking other work and it is highly undesirable to roll them back.)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200499</commentid>
    <comment_count>1</comment_count>
      <attachid>50850</attachid>
    <who name="Kenneth Russell">kbr</who>
    <bug_when>2010-03-16 16:26:47 -0700</bug_when>
    <thetext>Created attachment 50850
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200501</commentid>
    <comment_count>2</comment_count>
      <attachid>50850</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-03-16 16:28:28 -0700</bug_when>
    <thetext>Comment on attachment 50850
Patch

I think Kenneth meant &quot;build bots&quot;.  I don&apos;t think the commit-bot is failing these tests, but one of the build bots certainly is:
http://build.webkit.org/results/Leopard%20Intel%20Debug%20(Tests)/r56090%20(11512)/results.html

Either way, I think this is OK.

I would like to see the &quot;fix it&quot; bug instead of this bug listed in the Skipped list.  The ChangeLog could list both if you like.

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200503</commentid>
    <comment_count>3</comment_count>
      <attachid>50852</attachid>
    <who name="Kenneth Russell">kbr</who>
    <bug_when>2010-03-16 16:36:11 -0700</bug_when>
    <thetext>Created attachment 50852
Revised patch

Revised patch mentioning the &quot;real&quot; bug for this fix (36194) rather than this one in the Skipped file. Leaving the ChangeLog to avoid confusing webkit-patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200504</commentid>
    <comment_count>4</comment_count>
      <attachid>50852</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-03-16 16:37:20 -0700</bug_when>
    <thetext>Comment on attachment 50852
Revised patch

I shoudl have been more clear.  no need to re-post.  But still looks good!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200505</commentid>
    <comment_count>5</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-03-16 16:38:53 -0700</bug_when>
    <thetext>webkit-patch finds the &quot;bug&quot; for a diff by grepping the whole diff for bug numbers.  The first one it finds is the winner.

We might yet change that to only grep ChangeLogs, but for now that&apos;s how it works.
http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/webkitpy/bugzilla.py#L51</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200506</commentid>
    <comment_count>6</comment_count>
    <who name="Kenneth Russell">kbr</who>
    <bug_when>2010-03-16 16:39:26 -0700</bug_when>
    <thetext>Committed r56093: &lt;http://trac.webkit.org/changeset/56093&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>50850</attachid>
            <date>2010-03-16 16:26:47 -0700</date>
            <delta_ts>2010-03-16 16:36:11 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>tests.patch</filename>
            <type>text/plain</type>
            <size>1364</size>
            <attacher name="Kenneth Russell">kbr</attacher>
            
              <data encoding="base64">SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D
aGFuZ2VMb2cJKHJldmlzaW9uIDU2MDkyKQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr
aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTIgQEAKKzIwMTAtMDMtMTYgIEtlbm5ldGggUnVzc2VsbCAg
PGtickBnb29nbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor
CisgICAgICAgIFRlbXBvcmFyaWx5IGRpc2FibGUgV2ViR0wgbGF5b3V0IHRlc3RzIGZhaWxpbmcg
ZHVlIHRvIG11bHRpc2FtcGxpbmcKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv
d19idWcuY2dpP2lkPTM2MjAwCisKKyAgICAgICAgKiBwbGF0Zm9ybS9tYWMvU2tpcHBlZDoKKwog
MjAxMC0wMy0xNSAgT2phbiBWYWZhaSAgPG9qYW5AY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJl
dmlld2VkIGJ5IEFsZXhleSBQcm9za3VyeWFrb3YuCkluZGV4OiBMYXlvdXRUZXN0cy9wbGF0Zm9y
bS9tYWMvU2tpcHBlZAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMvU2tp
cHBlZAkocmV2aXNpb24gNTYwOTIpCisrKyBMYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMvU2tpcHBl
ZAkod29ya2luZyBjb3B5KQpAQCAtMTM3LDMgKzEzNywxMCBAQCBmYXN0L2RvbS9IVE1MUHJvZ3Jl
c3NFbGVtZW50L3NldC1wcm9ncmVzCiAjIGV2ZW50LmRhdGFUcmFuc2Zlci5zZXREYXRhKCd0ZXh0
L3VyaS1saXN0Jykgbm90IGNvcnJlY3RseSBpbXBsZW1lbnRlZCBvbiBNYWMKIGVkaXRpbmcvcGFz
dGVib2FyZC9kYXRhVHJhbnNmZXItc2V0RGF0YS1nZXREYXRhLmh0bWwKIAorIyBUZW1wb3Jhcmls
eSBjb21tZW50aW5nIG91dCBhIGZldyBXZWJHTCBsYXlvdXQgdGVzdHMgZmFpbGluZyBvbiBzb21l
IG9mIHRoZSBib3RzLgorIyBTZWUgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp
P2lkPTM2MjAwIC4KK2Zhc3QvY2FudmFzL3dlYmdsL2NvbnRleHQtYXR0cmlidXRlcy1hbHBoYS1k
ZXB0aC1zdGVuY2lsLWFudGlhbGlhcy5odG1sCitmYXN0L2NhbnZhcy93ZWJnbC90ZXgtaW1hZ2Ut
YW5kLXN1Yi1pbWFnZS0yZC13aXRoLWltYWdlLWRhdGEuaHRtbAorZmFzdC9jYW52YXMvd2ViZ2wv
dGV4LXN1Yi1pbWFnZS0yZC5odG1sCitmYXN0L2NhbnZhcy93ZWJnbC90cmlhbmdsZS5odG1sCitm
YXN0L2NhbnZhcy93ZWJnbC92aWV3cG9ydC11bmNoYW5nZWQtdXBvbi1yZXNpemUuaHRtbAo=
</data>
<flag name="review"
          id="34021"
          type_id="1"
          status="+"
          setter="eric"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>50852</attachid>
            <date>2010-03-16 16:36:11 -0700</date>
            <delta_ts>2010-03-16 16:37:20 -0700</delta_ts>
            <desc>Revised patch</desc>
            <filename>tests.patch</filename>
            <type>text/plain</type>
            <size>1364</size>
            <attacher name="Kenneth Russell">kbr</attacher>
            
              <data encoding="base64">SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D
aGFuZ2VMb2cJKHJldmlzaW9uIDU2MDkyKQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr
aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTIgQEAKKzIwMTAtMDMtMTYgIEtlbm5ldGggUnVzc2VsbCAg
PGtickBnb29nbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor
CisgICAgICAgIFRlbXBvcmFyaWx5IGRpc2FibGUgV2ViR0wgbGF5b3V0IHRlc3RzIGZhaWxpbmcg
ZHVlIHRvIG11bHRpc2FtcGxpbmcKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv
d19idWcuY2dpP2lkPTM2MjAwCisKKyAgICAgICAgKiBwbGF0Zm9ybS9tYWMvU2tpcHBlZDoKKwog
MjAxMC0wMy0xNSAgT2phbiBWYWZhaSAgPG9qYW5AY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJl
dmlld2VkIGJ5IEFsZXhleSBQcm9za3VyeWFrb3YuCkluZGV4OiBMYXlvdXRUZXN0cy9wbGF0Zm9y
bS9tYWMvU2tpcHBlZAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMvU2tp
cHBlZAkocmV2aXNpb24gNTYwOTIpCisrKyBMYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMvU2tpcHBl
ZAkod29ya2luZyBjb3B5KQpAQCAtMTM3LDMgKzEzNywxMCBAQCBmYXN0L2RvbS9IVE1MUHJvZ3Jl
c3NFbGVtZW50L3NldC1wcm9ncmVzCiAjIGV2ZW50LmRhdGFUcmFuc2Zlci5zZXREYXRhKCd0ZXh0
L3VyaS1saXN0Jykgbm90IGNvcnJlY3RseSBpbXBsZW1lbnRlZCBvbiBNYWMKIGVkaXRpbmcvcGFz
dGVib2FyZC9kYXRhVHJhbnNmZXItc2V0RGF0YS1nZXREYXRhLmh0bWwKIAorIyBUZW1wb3Jhcmls
eSBjb21tZW50aW5nIG91dCBhIGZldyBXZWJHTCBsYXlvdXQgdGVzdHMgZmFpbGluZyBvbiBzb21l
IG9mIHRoZSBib3RzLgorIyBTZWUgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp
P2lkPTM2MTk0IC4KK2Zhc3QvY2FudmFzL3dlYmdsL2NvbnRleHQtYXR0cmlidXRlcy1hbHBoYS1k
ZXB0aC1zdGVuY2lsLWFudGlhbGlhcy5odG1sCitmYXN0L2NhbnZhcy93ZWJnbC90ZXgtaW1hZ2Ut
YW5kLXN1Yi1pbWFnZS0yZC13aXRoLWltYWdlLWRhdGEuaHRtbAorZmFzdC9jYW52YXMvd2ViZ2wv
dGV4LXN1Yi1pbWFnZS0yZC5odG1sCitmYXN0L2NhbnZhcy93ZWJnbC90cmlhbmdsZS5odG1sCitm
YXN0L2NhbnZhcy93ZWJnbC92aWV3cG9ydC11bmNoYW5nZWQtdXBvbi1yZXNpemUuaHRtbAo=
</data>
<flag name="review"
          id="34023"
          type_id="1"
          status="+"
          setter="eric"
    />
          </attachment>
      

    </bug>

</bugzilla>