<?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>48677</bug_id>
          
          <creation_ts>2010-10-29 14:20:40 -0700</creation_ts>
          <short_desc>vertexAttribPointer should generate INVALID_VALUE if the stride exceeds 255</short_desc>
          <delta_ts>2010-11-02 09:44:51 -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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Zhenyao Mo">zmo</reporter>
          <assigned_to name="Zhenyao Mo">zmo</assigned_to>
          <cc>cmarrin</cc>
    
    <cc>enne</cc>
    
    <cc>kbr</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>302054</commentid>
    <comment_count>0</comment_count>
    <who name="Zhenyao Mo">zmo</who>
    <bug_when>2010-10-29 14:20:40 -0700</bug_when>
    <thetext>WebGL spec:

6.6 Vertex Attribute Data Stride

The WebGL API supports vertex attribute data strides up to 255 bytes. A call to vertexAttribPointer will generate an INVALID_VALUE error if the value for the stride parameter exceeds 255.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>303000</commentid>
    <comment_count>1</comment_count>
      <attachid>72576</attachid>
    <who name="Zhenyao Mo">zmo</who>
    <bug_when>2010-11-01 15:42:47 -0700</bug_when>
    <thetext>Created attachment 72576
patch

Test synched with khronos.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>303100</commentid>
    <comment_count>2</comment_count>
      <attachid>72576</attachid>
    <who name="Andreas Kling">kling</who>
    <bug_when>2010-11-01 17:10:27 -0700</bug_when>
    <thetext>Comment on attachment 72576
patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>303420</commentid>
    <comment_count>3</comment_count>
    <who name="Zhenyao Mo">zmo</who>
    <bug_when>2010-11-02 09:44:51 -0700</bug_when>
    <thetext>Committed r71123: &lt;http://trac.webkit.org/changeset/71123&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>72576</attachid>
            <date>2010-11-01 15:42:47 -0700</date>
            <delta_ts>2010-11-01 17:10:27 -0700</delta_ts>
            <desc>patch</desc>
            <filename>vap.patch</filename>
            <type>text/plain</type>
            <size>5053</size>
            <attacher name="Zhenyao Mo">zmo</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA3MTA2NSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTUgQEAKKzIwMTAtMTEtMDEgIFpoZW55YW8gTW8gIDx6bW9AZ29vZ2xlLmNvbT4K
KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICB2ZXJ0ZXhB
dHRyaWJQb2ludGVyIHNob3VsZCBnZW5lcmF0ZSBJTlZBTElEX1ZBTFVFIGlmIHRoZSBzdHJpZGUg
ZXhjZWVkcyAyNTUKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp
P2lkPTQ4Njc3CisKKyAgICAgICAgVGVzdDogZmFzdC9jYW52YXMvd2ViZ2wvZ2wtdmVydGV4YXR0
cmlicG9pbnRlci5odG1sCisKKyAgICAgICAgKiBodG1sL2NhbnZhcy9XZWJHTFJlbmRlcmluZ0Nv
bnRleHQuY3BwOgorICAgICAgICAoV2ViQ29yZTo6V2ViR0xSZW5kZXJpbmdDb250ZXh0Ojp2ZXJ0
ZXhBdHRyaWJQb2ludGVyKTogR2VuZXJhdGUgSU5WQUxJRF9WQUxVRSBpZiBzdHJpZGUgPiAyNTUu
CisKIDIwMTAtMTAtMzEgIEFkYW0gQmFydGggIDxhYmFydGhAd2Via2l0Lm9yZz4KIAogICAgICAg
ICBSZXZpZXdlZCBieSBFcmljIFNlaWRlbC4KSW5kZXg6IFdlYkNvcmUvaHRtbC9jYW52YXMvV2Vi
R0xSZW5kZXJpbmdDb250ZXh0LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL2h0bWwvY2FudmFz
L1dlYkdMUmVuZGVyaW5nQ29udGV4dC5jcHAJKHJldmlzaW9uIDcxMDQ2KQorKysgV2ViQ29yZS9o
dG1sL2NhbnZhcy9XZWJHTFJlbmRlcmluZ0NvbnRleHQuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0y
ODQ1LDcgKzI4NDUsNyBAQCB2b2lkIFdlYkdMUmVuZGVyaW5nQ29udGV4dDo6dmVydGV4QXR0cmli
CiAgICAgICAgIG1fY29udGV4dC0+c3ludGhlc2l6ZUdMRXJyb3IoR3JhcGhpY3NDb250ZXh0M0Q6
OklOVkFMSURfVkFMVUUpOwogICAgICAgICByZXR1cm47CiAgICAgfQotICAgIGlmIChzaXplIDwg
MSB8fCBzaXplID4gNCB8fCBzdHJpZGUgPCAwIHx8IG9mZnNldCA8IDApIHsKKyAgICBpZiAoc2l6
ZSA8IDEgfHwgc2l6ZSA+IDQgfHwgc3RyaWRlIDwgMCB8fCBzdHJpZGUgPiAyNTUgfHwgb2Zmc2V0
IDwgMCkgewogICAgICAgICBtX2NvbnRleHQtPnN5bnRoZXNpemVHTEVycm9yKEdyYXBoaWNzQ29u
dGV4dDNEOjpJTlZBTElEX1ZBTFVFKTsKICAgICAgICAgcmV0dXJuOwogICAgIH0KSW5kZXg6IExh
eW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJ
KHJldmlzaW9uIDcxMDY1KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkp
CkBAIC0xLDMgKzEsMTMgQEAKKzIwMTAtMTEtMDEgIFpoZW55YW8gTW8gIDx6bW9AZ29vZ2xlLmNv
bT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICB2ZXJ0
ZXhBdHRyaWJQb2ludGVyIHNob3VsZCBnZW5lcmF0ZSBJTlZBTElEX1ZBTFVFIGlmIHRoZSBzdHJp
ZGUgZXhjZWVkcyAyNTUKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu
Y2dpP2lkPTQ4Njc3CisKKyAgICAgICAgKiBmYXN0L2NhbnZhcy93ZWJnbC9nbC12ZXJ0ZXhhdHRy
aWJwb2ludGVyLWV4cGVjdGVkLnR4dDogQWRkZWQuCisgICAgICAgICogZmFzdC9jYW52YXMvd2Vi
Z2wvZ2wtdmVydGV4YXR0cmlicG9pbnRlci5odG1sOiBBZGRlZC4KKwogMjAxMC0xMS0wMSAgRG1p
dHJ5IFRpdG92ICA8ZGltaWNoQGNocm9taXVtLm9yZz4KIAogICAgICAgICBbQ2hyb21pdW1dIFVu
cmV2aWV3ZWQgdGVzdCByZWJhc2VsaW5lLgpJbmRleDogTGF5b3V0VGVzdHMvZmFzdC9jYW52YXMv
d2ViZ2wvZ2wtdmVydGV4YXR0cmlicG9pbnRlci1leHBlY3RlZC50eHQKPT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0g
TGF5b3V0VGVzdHMvZmFzdC9jYW52YXMvd2ViZ2wvZ2wtdmVydGV4YXR0cmlicG9pbnRlci1leHBl
Y3RlZC50eHQJKHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy9mYXN0L2NhbnZhcy93ZWJnbC9n
bC12ZXJ0ZXhhdHRyaWJwb2ludGVyLWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkKQEAgLTAsMCAr
MSwxNyBAQAorVGhpcyB0ZXN0IGNoZWNrcyB2ZXJ0ZXhBdHRyaWJQb2ludGVyIGJlaGF2aW9ycyBp
biBXZWJHTC4KKworT24gc3VjY2VzcywgeW91IHdpbGwgc2VlIGEgc2VyaWVzIG9mICJQQVNTIiBt
ZXNzYWdlcywgZm9sbG93ZWQgYnkgIlRFU1QgQ09NUExFVEUiLgorCisKK0NhbnZhcy5nZXRDb250
ZXh0CitQQVNTIGNvbnRleHQgZXhpc3RzCisKK0NoZWNraW5nIGdsLnZlcnRleEF0dHJpYlBvaW50
ZXIuCitQQVNTIGdldEVycm9yIHdhcyBleHBlY3RlZCB2YWx1ZTogSU5WQUxJRF9PUEVSQVRJT04g
OiB2ZXJ0ZXhBdHRyaWJQb2ludGVyIHNob3VsZCBmYWlsIGlmIG5vIGJ1ZmZlciBpcyBib3VuZAor
UEFTUyBnZXRFcnJvciB3YXMgZXhwZWN0ZWQgdmFsdWU6IElOVkFMSURfVkFMVUUgOiBXZWJHTCBB
UEkgc3VwcG9ydHMgdmVydGV4IGF0dHJpYnV0ZSBkYXRhIHN0cmlkZXMgdXAgdG8gMjU1IGJ5dGVz
CitQQVNTIGdldEVycm9yIHdhcyBleHBlY3RlZCB2YWx1ZTogTk9fRVJST1IgOiB2ZXJ0ZXhBdHRy
aWJQb2ludGVyIHdpdGggc3RyaWRlIDw9IDI1NSBzaG91bGQgc3VjY2VlZAorCitQQVNTIHN1Y2Nl
c3NmdWxseVBhcnNlZCBpcyB0cnVlCisKK1RFU1QgQ09NUExFVEUKKwpJbmRleDogTGF5b3V0VGVz
dHMvZmFzdC9jYW52YXMvd2ViZ2wvZ2wtdmVydGV4YXR0cmlicG9pbnRlci5odG1sCj09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT0KLS0tIExheW91dFRlc3RzL2Zhc3QvY2FudmFzL3dlYmdsL2dsLXZlcnRleGF0dHJpYnBvaW50
ZXIuaHRtbAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvY2FudmFzL3dlYmdsL2ds
LXZlcnRleGF0dHJpYnBvaW50ZXIuaHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSw1NiBAQAor
PCFET0NUWVBFIEhUTUwgUFVCTElDICItLy9XM0MvL0RURCBIVE1MIDQuMDEgVHJhbnNpdGlvbmFs
Ly9FTiIKKyAgImh0dHA6Ly93d3cudzMub3JnL1RSL2h0bWw0L2xvb3NlLmR0ZCI+Cis8aHRtbD4K
KzxoZWFkPgorPG1ldGEgaHR0cC1lcXVpdj0iQ29udGVudC1UeXBlIiBjb250ZW50PSJ0ZXh0L2h0
bWw7IGNoYXJzZXQ9dXRmLTgiPgorPHRpdGxlPldlYkdMIHZlcnRleEF0dHJpYlBvaW50ZXIgQ29u
Zm9ybWFuY2UgVGVzdHM8L3RpdGxlPgorPGxpbmsgcmVsPSJzdHlsZXNoZWV0IiBocmVmPSIuLi8u
Li9qcy9yZXNvdXJjZXMvanMtdGVzdC1zdHlsZS5jc3MiLz4KKzxzY3JpcHQgc3JjPSJyZXNvdXJj
ZXMvZGVza3RvcC1nbC1jb25zdGFudHMuanMiIHR5cGU9InRleHQvamF2YXNjcmlwdCI+PC9zY3Jp
cHQ+Cis8c2NyaXB0IHNyYz0iLi4vLi4vanMvcmVzb3VyY2VzL2pzLXRlc3QtcHJlLmpzIj48L3Nj
cmlwdD4KKzxzY3JpcHQgc3JjPSJyZXNvdXJjZXMvd2ViZ2wtdGVzdC5qcyI+PC9zY3JpcHQ+Cis8
L2hlYWQ+Cis8Ym9keT4KKzxkaXYgaWQ9ImRlc2NyaXB0aW9uIj48L2Rpdj4KKzxkaXYgaWQ9ImNv
bnNvbGUiPjwvZGl2PgorPGNhbnZhcyBpZD0iY2FudmFzIiB3aWR0aD0iMiIgaGVpZ2h0PSIyIj4g
PC9jYW52YXM+Cis8c2NyaXB0PgorZGVzY3JpcHRpb24oIlRoaXMgdGVzdCBjaGVja3MgdmVydGV4
QXR0cmliUG9pbnRlciBiZWhhdmlvcnMgaW4gV2ViR0wuIik7CisKK2RlYnVnKCIiKTsKK2RlYnVn
KCJDYW52YXMuZ2V0Q29udGV4dCIpOworCit2YXIgZ2wgPSBjcmVhdGUzRENvbnRleHQoZG9jdW1l
bnQuZ2V0RWxlbWVudEJ5SWQoImNhbnZhcyIpKTsKK2lmICghZ2wpIHsKKyAgdGVzdEZhaWxlZCgi
Y29udGV4dCBkb2VzIG5vdCBleGlzdCIpOworfSBlbHNlIHsKKyAgdGVzdFBhc3NlZCgiY29udGV4
dCBleGlzdHMiKTsKKworICBkZWJ1ZygiIik7CisgIGRlYnVnKCJDaGVja2luZyBnbC52ZXJ0ZXhB
dHRyaWJQb2ludGVyLiIpOworCisgIGdsLnZlcnRleEF0dHJpYlBvaW50ZXIoMCwgMywgZ2wuRkxP
QVQsIDAsIDAsIDEyKTsKKyAgZ2xFcnJvclNob3VsZEJlKGdsLCBnbC5JTlZBTElEX09QRVJBVElP
TiwKKyAgICAgICJ2ZXJ0ZXhBdHRyaWJQb2ludGVyIHNob3VsZCBmYWlsIGlmIG5vIGJ1ZmZlciBp
cyBib3VuZCIpOworCisgIHZhciB2ZXJ0ZXhPYmplY3QgPSBnbC5jcmVhdGVCdWZmZXIoKTsKKyAg
Z2wuYmluZEJ1ZmZlcihnbC5BUlJBWV9CVUZGRVIsIHZlcnRleE9iamVjdCk7CisgIGdsLmJ1ZmZl
ckRhdGEoZ2wuQVJSQVlfQlVGRkVSLCBuZXcgRmxvYXQzMkFycmF5KDUxMiksIGdsLlNUQVRJQ19E
UkFXKTsKKyAgZ2wudmVydGV4QXR0cmliUG9pbnRlcigwLCAxLCBnbC5GTE9BVCwgMCwgMjU2LCAw
KTsKKyAgZ2xFcnJvclNob3VsZEJlKGdsLCBnbC5JTlZBTElEX1ZBTFVFLAorICAgICAgIldlYkdM
IEFQSSBzdXBwb3J0cyB2ZXJ0ZXggYXR0cmlidXRlIGRhdGEgc3RyaWRlcyB1cCB0byAyNTUgYnl0
ZXMiKTsKKyAgZ2wudmVydGV4QXR0cmliUG9pbnRlcigwLCAxLCBnbC5GTE9BVCwgMCwgMjU1LCAw
KTsKKyAgZ2xFcnJvclNob3VsZEJlKGdsLCBnbC5OT19FUlJPUiwKKyAgICAgICJ2ZXJ0ZXhBdHRy
aWJQb2ludGVyIHdpdGggc3RyaWRlIDw9IDI1NSBzaG91bGQgc3VjY2VlZCIpOworfQorCitkZWJ1
ZygiIik7CitzdWNjZXNzZnVsbHlQYXJzZWQgPSB0cnVlOworCis8L3NjcmlwdD4KKzxzY3JpcHQg
c3JjPSIuLi8uLi9qcy9yZXNvdXJjZXMvanMtdGVzdC1wb3N0LmpzIj48L3NjcmlwdD4KKworPHNj
cmlwdD4KKzwvc2NyaXB0PgorCis8L2JvZHk+Cis8L2h0bWw+Cg==
</data>
<flag name="review"
          id="62850"
          type_id="1"
          status="+"
          setter="kling"
    />
    <flag name="commit-queue"
          id="62851"
          type_id="3"
          status="-"
          setter="zmo"
    />
          </attachment>
      

    </bug>

</bugzilla>