<?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>46870</bug_id>
          
          <creation_ts>2010-09-29 20:36:49 -0700</creation_ts>
          <short_desc>fast/canvas/webgl/draw-elements-out-of-bounds.html is crashing on Snow Leopard</short_desc>
          <delta_ts>2010-09-30 16:36:49 -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>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>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>45769</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Eric Seidel (no email)">eric</reporter>
          <assigned_to name="Zhenyao Mo">zmo</assigned_to>
          <cc>alokp</cc>
    
    <cc>cmarrin</cc>
    
    <cc>commit-queue</cc>
    
    <cc>enne</cc>
    
    <cc>kbr</cc>
    
    <cc>zmo</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>287358</commentid>
    <comment_count>0</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-09-29 20:36:49 -0700</bug_when>
    <thetext>fast/canvas/webgl/draw-elements-out-of-bounds.html is crashing on Snow Leopard

http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(Tests)/r68740%20(18485)/fast/canvas/webgl/draw-elements-out-of-bounds-stderr.txt

Assertion failed: (0 &amp;&amp; &quot;InitThread(): Process hasn&apos;t been initalised.&quot;), function InitThread, file /Volumes/Data/WebKit-BuildSlave/snowleopard-intel-release/build/ANGLE/src/compiler/InitializeDll.cpp, line 71.

The crashing started after I turned off some canvas tests as part of this rollout:
http://trac.webkit.org/changeset/68737

Since skipping tests should not cause crashes, there was likely a crash here before, masked by other factors.

Going to skip this canvas test until zmo/kbr can take a look.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>287359</commentid>
    <comment_count>1</comment_count>
      <attachid>69307</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-09-29 20:40:44 -0700</bug_when>
    <thetext>Created attachment 69307
Skip the test</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>287360</commentid>
    <comment_count>2</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-09-29 20:42:22 -0700</bug_when>
    <thetext>Committed r68745: &lt;http://trac.webkit.org/changeset/68745&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>287370</commentid>
    <comment_count>3</comment_count>
    <who name="Zhenyao Mo">zmo</who>
    <bug_when>2010-09-29 21:08:09 -0700</bug_when>
    <thetext>I&apos;ll take a look.  Looks like Angle issue, but it could be misleading.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>287498</commentid>
    <comment_count>4</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2010-09-30 05:25:15 -0700</bug_when>
    <thetext>&lt;rdar://problem/8496585&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>287776</commentid>
    <comment_count>5</comment_count>
      <attachid>69374</attachid>
    <who name="Zhenyao Mo">zmo</who>
    <bug_when>2010-09-30 13:35:18 -0700</bug_when>
    <thetext>Created attachment 69374
patch

After discussing with Alok and Dimitri, this is an educated attempt.

The cause (from the assertion failure) could be the previous test cleanup is delayed and the current test called ShInitialize() before the ShFinalize() from the previous test.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>287779</commentid>
    <comment_count>6</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-09-30 13:37:57 -0700</bug_when>
    <thetext>Do we finalize in some other way? or after this change would we never finalize?  And what are the consequences of such?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>287825</commentid>
    <comment_count>7</comment_count>
      <attachid>69374</attachid>
    <who name="Kenneth Russell">kbr</who>
    <bug_when>2010-09-30 14:27:17 -0700</bug_when>
    <thetext>Comment on attachment 69374
patch

We&apos;ve discussed this earlier via email and this is the correct fix at the present time.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>287858</commentid>
    <comment_count>8</comment_count>
    <who name="Zhenyao Mo">zmo</who>
    <bug_when>2010-09-30 14:48:38 -0700</bug_when>
    <thetext>Committed r68833: &lt;http://trac.webkit.org/changeset/68833&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>287920</commentid>
    <comment_count>9</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-09-30 16:31:19 -0700</bug_when>
    <thetext>My questions were never answered, but I guess I&apos;m not on the right mailing list. :)

Does this need a FIXME for the future?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>287921</commentid>
    <comment_count>10</comment_count>
    <who name="Kenneth Russell">kbr</who>
    <bug_when>2010-09-30 16:32:55 -0700</bug_when>
    <thetext>(In reply to comment #9)
&gt; My questions were never answered, but I guess I&apos;m not on the right mailing list. :)
&gt; 
&gt; Does this need a FIXME for the future?

Not really. At best we would need an at-exit hook, but OS process separation will take care of the cleanup of the needed static and thread-local resources.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>287924</commentid>
    <comment_count>11</comment_count>
    <who name="Zhenyao Mo">zmo</who>
    <bug_when>2010-09-30 16:36:49 -0700</bug_when>
    <thetext>(In reply to comment #9)
&gt; My questions were never answered, but I guess I&apos;m not on the right mailing list. :)
&gt; 
&gt; Does this need a FIXME for the future?

alokp is working on getting rid of ShInitialize and ShFinalize.  We&apos;ll just live with this until that happens.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>69307</attachid>
            <date>2010-09-29 20:40:44 -0700</date>
            <delta_ts>2010-09-30 13:35:18 -0700</delta_ts>
            <desc>Skip the test</desc>
            <filename>bug-46870-20100929204043.patch</filename>
            <type>text/plain</type>
            <size>1388</size>
            <attacher name="WebKit Commit Bot">commit-queue</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCBkNWQ0MWRkZWJmZWViYjRiOTZiOGYwNzRmZDE5MWE1NWEwMmZiYTk4Li4wMGIyMDZh
OTFlNmRmYjc3OWY2NDA3YzY2YjY3MTM1YmZhZjBmMWI4IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0
cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAK
KzIwMTAtMDktMjkgIEVyaWMgU2VpZGVsICA8ZXJpY0B3ZWJraXQub3JnPgorCisgICAgICAgIFVu
cmV2aWV3ZWQuICBTa2lwcGluZyBjcmFzaGluZyB0ZXN0IHRvIGdyZWVuIHRoZSB0cmVlLgorICAg
ICAgICBUaGlzIGNyYXNoIHN0YXJ0ZWQgd2hlbiBJIHNraXBwZWQgc29tZSBjYW52YXMgdGVzdHMK
KyAgICAgICAgYXMgcGFydCBvZiByb2xsaW5nIG91dCByNjg1NjguCisKKyAgICAgICAgZmFzdC9j
YW52YXMvd2ViZ2wvZHJhdy1lbGVtZW50cy1vdXQtb2YtYm91bmRzLmh0bWwgaXMgY3Jhc2hpbmcg
b24gU25vdyBMZW9wYXJkCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVn
LmNnaT9pZD00Njg3MAorCisgICAgICAgICogcGxhdGZvcm0vbWFjLXNub3dsZW9wYXJkL1NraXBw
ZWQ6CisKIDIwMTAtMDktMjkgIE1paGFpIFBhcnBhcml0YSAgPG1paGFpcEBjaHJvbWl1bS5vcmc+
CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgRGFyaW4gRmlzaGVyLgpkaWZmIC0tZ2l0IGEvTGF5b3V0
VGVzdHMvcGxhdGZvcm0vbWFjLXNub3dsZW9wYXJkL1NraXBwZWQgYi9MYXlvdXRUZXN0cy9wbGF0
Zm9ybS9tYWMtc25vd2xlb3BhcmQvU2tpcHBlZAppbmRleCBmZDg5YzE3MDcxZWNjYzVlNzhhMTdl
Zjk5YzAzM2ZjODQwZDE1ZGQ1Li42MzI3NjBhNDZlM2I2OWQzMzRiODdmYWZmYWYyZTM4YTE5NjFi
NGM2IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMtc25vd2xlb3BhcmQvU2tp
cHBlZAorKysgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMtc25vd2xlb3BhcmQvU2tpcHBlZApA
QCAtMTM3LDMgKzEzNyw3IEBAIHN0b3JhZ2UvaW5kZXhlZGRiCiAjIFBoaWxpcCdzIGNhbnZhcyB0
ZXN0cyB0aGF0IGZhaWwgb24gU25vd0xlb3BhcmQgb25seQogY2FudmFzL3BoaWxpcC90ZXN0cy8y
ZC5ncmFkaWVudC5pbnRlcnBvbGF0ZS5vdmVybGFwLmh0bWwKIGNhbnZhcy9waGlsaXAvdGVzdHMv
MmQuZ3JhZGllbnQuaW50ZXJwb2xhdGUub3ZlcmxhcDIuaHRtbAorCisjIENyYXNoaW5nIGR1ZSB0
byBpbnRlcmFjdGlvbiB3aXRoIGVhcmxpZXIgdGVzdHMKKyMgaHR0cHM6Ly9idWdzLndlYmtpdC5v
cmcvc2hvd19idWcuY2dpP2lkPTQ2ODcwCitmYXN0L2NhbnZhcy93ZWJnbC9kcmF3LWVsZW1lbnRz
LW91dC1vZi1ib3VuZHMuaHRtbAo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>69374</attachid>
            <date>2010-09-30 13:35:18 -0700</date>
            <delta_ts>2010-09-30 14:27:17 -0700</delta_ts>
            <desc>patch</desc>
            <filename>angle.patch</filename>
            <type>text/plain</type>
            <size>2407</size>
            <attacher name="Zhenyao Mo">zmo</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA2ODgxOCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTMgQEAKKzIwMTAtMDktMzAgIFpoZW55YW8gTW8gIDx6bW9AZ29vZ2xlLmNvbT4K
KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBmYXN0L2Nh
bnZhcy93ZWJnbC9kcmF3LWVsZW1lbnRzLW91dC1vZi1ib3VuZHMuaHRtbCBpcyBjcmFzaGluZyBv
biBTbm93IExlb3BhcmQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu
Y2dpP2lkPTQ2ODcwCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy9BTkdMRVdlYktpdEJy
aWRnZS5jcHA6CisgICAgICAgIChXZWJDb3JlOjpBTkdMRVdlYktpdEJyaWRnZTo6fkFOR0xFV2Vi
S2l0QnJpZGdlKTogUmVtb3ZlIFNoRmluYWxpemUoKS4KKwogMjAxMC0wOS0zMCAgRGF2aWQgSHlh
dHQgIDxoeWF0dEBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgQmV0aCBEYWtpbi4K
SW5kZXg6IFdlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvQU5HTEVXZWJLaXRCcmlkZ2UuY3BwCj09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT0KLS0tIFdlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvQU5HTEVXZWJLaXRCcmlkZ2Uu
Y3BwCShyZXZpc2lvbiA2ODgwNSkKKysrIFdlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvQU5HTEVX
ZWJLaXRCcmlkZ2UuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC00NCw4ICs0NCw2IEBAIEFOR0xFV2Vi
S2l0QnJpZGdlOjp+QU5HTEVXZWJLaXRCcmlkZ2UoKQogICAgICAgICBTaERlc3RydWN0KG1fZnJh
Z21lbnRDb21waWxlcik7CiAgICAgICAgIFNoRGVzdHJ1Y3QobV92ZXJ0ZXhDb21waWxlcik7CiAg
ICAgfQotICAgIAotICAgIFNoRmluYWxpemUoKTsKIH0KIAogYm9vbCBBTkdMRVdlYktpdEJyaWRn
ZTo6dmFsaWRhdGVTaGFkZXJTb3VyY2UoY29uc3QgY2hhciogc2hhZGVyU291cmNlLCBBTkdMRVNo
YWRlclR5cGUgc2hhZGVyVHlwZSwgU3RyaW5nJiB0cmFuc2xhdGVkU2hhZGVyU291cmNlLCBTdHJp
bmcmIHNoYWRlclZhbGlkYXRpb25Mb2cpCkluZGV4OiBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cKPT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PQotLS0gTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCShyZXZpc2lvbiA2ODgxOCkKKysrIExh
eW91dFRlc3RzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDEyIEBACisyMDEw
LTA5LTMwICBaaGVueWFvIE1vICA8em1vQGdvb2dsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQg
YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgZmFzdC9jYW52YXMvd2ViZ2wvZHJhdy1lbGVt
ZW50cy1vdXQtb2YtYm91bmRzLmh0bWwgaXMgY3Jhc2hpbmcgb24gU25vdyBMZW9wYXJkCisgICAg
ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD00Njg3MAorCisgICAg
ICAgICogcGxhdGZvcm0vbWFjLXNub3dsZW9wYXJkL1NraXBwZWQ6IFJlLWVuYWJsZSB0aGUgdGVz
dC4KKwogMjAxMC0wOS0zMCAgQWxwaGEgTGFtICA8aGNsYW1AY2hyb21pdW0ub3JnPgogCiAgICAg
ICAgIFVucmV2aWV3ZWQuIEJ1aWxkIGZpeC4KSW5kZXg6IExheW91dFRlc3RzL3BsYXRmb3JtL21h
Yy1zbm93bGVvcGFyZC9Ta2lwcGVkCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL3BsYXRmb3Jt
L21hYy1zbm93bGVvcGFyZC9Ta2lwcGVkCShyZXZpc2lvbiA2ODgwNSkKKysrIExheW91dFRlc3Rz
L3BsYXRmb3JtL21hYy1zbm93bGVvcGFyZC9Ta2lwcGVkCSh3b3JraW5nIGNvcHkpCkBAIC0xMzcs
NyArMTM3LDMgQEAgc3RvcmFnZS9pbmRleGVkZGIKICMgUGhpbGlwJ3MgY2FudmFzIHRlc3RzIHRo
YXQgZmFpbCBvbiBTbm93TGVvcGFyZCBvbmx5CiBjYW52YXMvcGhpbGlwL3Rlc3RzLzJkLmdyYWRp
ZW50LmludGVycG9sYXRlLm92ZXJsYXAuaHRtbAogY2FudmFzL3BoaWxpcC90ZXN0cy8yZC5ncmFk
aWVudC5pbnRlcnBvbGF0ZS5vdmVybGFwMi5odG1sCi0KLSMgQ3Jhc2hpbmcgZHVlIHRvIGludGVy
YWN0aW9uIHdpdGggZWFybGllciB0ZXN0cwotIyBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93
X2J1Zy5jZ2k/aWQ9NDY4NzAKLWZhc3QvY2FudmFzL3dlYmdsL2RyYXctZWxlbWVudHMtb3V0LW9m
LWJvdW5kcy5odG1sCg==
</data>
<flag name="review"
          id="59057"
          type_id="1"
          status="+"
          setter="kbr"
    />
    <flag name="commit-queue"
          id="59058"
          type_id="3"
          status="-"
          setter="zmo"
    />
          </attachment>
      

    </bug>

</bugzilla>