<?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>215298</bug_id>
          
          <creation_ts>2020-08-07 16:04:06 -0700</creation_ts>
          <short_desc>Off by one error in transform feedback buffer binding</short_desc>
          <delta_ts>2020-08-07 17:39:11 -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>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</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>
          
          <blocked>126404</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="James Darpinian">jdarpinian</reporter>
          <assigned_to name="James Darpinian">jdarpinian</assigned_to>
          <cc>cdumez</cc>
    
    <cc>changseok</cc>
    
    <cc>dino</cc>
    
    <cc>esprehn+autocc</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>graouts</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>kbr</cc>
    
    <cc>kondapallykalyan</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1678789</commentid>
    <comment_count>0</comment_count>
    <who name="James Darpinian">jdarpinian</who>
    <bug_when>2020-08-07 16:04:06 -0700</bug_when>
    <thetext>Off by one error in transform feedback buffer binding</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678791</commentid>
    <comment_count>1</comment_count>
      <attachid>406224</attachid>
    <who name="James Darpinian">jdarpinian</who>
    <bug_when>2020-08-07 16:05:17 -0700</bug_when>
    <thetext>Created attachment 406224
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678798</commentid>
    <comment_count>2</comment_count>
      <attachid>406224</attachid>
    <who name="Kenneth Russell">kbr</who>
    <bug_when>2020-08-07 16:07:09 -0700</bug_when>
    <thetext>Comment on attachment 406224
Patch

Looks good to me. I&apos;d r+ but my reviewer status hasn&apos;t been reinstated.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678799</commentid>
    <comment_count>3</comment_count>
      <attachid>406224</attachid>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2020-08-07 16:08:00 -0700</bug_when>
    <thetext>Comment on attachment 406224
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=406224&amp;action=review

&gt; Source/WebCore/ChangeLog:8
&gt; +        Caught by conformance test deqp/functional/gles3/negativebufferapi.html

This change is missing a test.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678802</commentid>
    <comment_count>4</comment_count>
      <attachid>406224</attachid>
    <who name="James Darpinian">jdarpinian</who>
    <bug_when>2020-08-07 16:19:27 -0700</bug_when>
    <thetext>Comment on attachment 406224
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=406224&amp;action=review

&gt;&gt; Source/WebCore/ChangeLog:8
&gt;&gt; +        Caught by conformance test deqp/functional/gles3/negativebufferapi.html
&gt; 
&gt; This change is missing a test.

The test is present. The suite it&apos;s in (deqp) is currently skipped in the test expectations because it takes a long time to run. I&apos;m looking into whether we can make it faster.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678812</commentid>
    <comment_count>5</comment_count>
      <attachid>406224</attachid>
    <who name="Dean Jackson">dino</who>
    <bug_when>2020-08-07 16:31:48 -0700</bug_when>
    <thetext>Comment on attachment 406224
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=406224&amp;action=review

&gt;&gt;&gt; Source/WebCore/ChangeLog:8
&gt;&gt;&gt; +        Caught by conformance test deqp/functional/gles3/negativebufferapi.html
&gt;&gt; 
&gt;&gt; This change is missing a test.
&gt; 
&gt; The test is present. The suite it&apos;s in (deqp) is currently skipped in the test expectations because it takes a long time to run. I&apos;m looking into whether we can make it faster.

We have a separate bot that runs the deqp tests.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678827</commentid>
    <comment_count>6</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2020-08-07 16:56:34 -0700</bug_when>
    <thetext>Committed r265399: &lt;https://trac.webkit.org/changeset/265399&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 406224.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678828</commentid>
    <comment_count>7</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2020-08-07 16:57:18 -0700</bug_when>
    <thetext>&lt;rdar://problem/66706728&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>406224</attachid>
            <date>2020-08-07 16:05:17 -0700</date>
            <delta_ts>2020-08-07 16:56:34 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-215298-20200807160516.patch</filename>
            <type>text/plain</type>
            <size>1615</size>
            <attacher name="James Darpinian">jdarpinian</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjY1MzQ5CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggMTRlZGUzM2JiNTVmYjQ4
YzgxZDE3ZjU4YmUzNDBmYmZhOTU5N2JhMi4uNjc4ZTExOThlYTYxYmUyZDgzNDM5NzMyNzExZWU2
YzNkOWNkYjY4OSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDIwLTA4LTA3ICBKYW1l
cyBEYXJwaW5pYW4gIDxqZGFycGluaWFuQGNocm9taXVtLm9yZz4KKworICAgICAgICBPZmYgYnkg
b25lIGVycm9yIGluIHRyYW5zZm9ybSBmZWVkYmFjayBidWZmZXIgYmluZGluZworICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjE1Mjk4CisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgQ2F1Z2h0IGJ5IGNvbmZvcm1h
bmNlIHRlc3QgZGVxcC9mdW5jdGlvbmFsL2dsZXMzL25lZ2F0aXZlYnVmZmVyYXBpLmh0bWwKKwor
ICAgICAgICAqIGh0bWwvY2FudmFzL1dlYkdMMlJlbmRlcmluZ0NvbnRleHQuY3BwOgorICAgICAg
ICAoV2ViQ29yZTo6V2ViR0wyUmVuZGVyaW5nQ29udGV4dDo6c2V0SW5kZXhlZEJ1ZmZlckJpbmRp
bmcpOgorCiAyMDIwLTA4LTA2ICBDaHJpcyBEdW1leiAgPGNkdW1lekBhcHBsZS5jb20+CiAKICAg
ICAgICAgRHJvcCBub24tc3RhbmRhcmQgY3JlYXRlQnVmZmVyKEFycmF5QnVmZmVyLCBib29sZWFu
KSBvdmVybG9hZCBmcm9tIEF1ZGlvQ29udGV4dApkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUv
aHRtbC9jYW52YXMvV2ViR0wyUmVuZGVyaW5nQ29udGV4dC5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9o
dG1sL2NhbnZhcy9XZWJHTDJSZW5kZXJpbmdDb250ZXh0LmNwcAppbmRleCBhZWYxMzI4MDBhNGYx
MTBjYTg3OTc4ZDc1YTY3MDk5ZWFjM2FmY2ZhLi4xOTgzNGNlNWMwYzhkMmNmNzMwMjNkYzNkYTI0
ZDIzMDNiNGQyNjRjIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9odG1sL2NhbnZhcy9XZWJH
TDJSZW5kZXJpbmdDb250ZXh0LmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9odG1sL2NhbnZhcy9X
ZWJHTDJSZW5kZXJpbmdDb250ZXh0LmNwcApAQCAtMjM0MCw3ICsyMzQwLDcgQEAgYm9vbCBXZWJH
TDJSZW5kZXJpbmdDb250ZXh0OjpzZXRJbmRleGVkQnVmZmVyQmluZGluZyhjb25zdCBjaGFyICpm
dW5jdGlvbk5hbWUsIEcKIAogICAgIHN3aXRjaCAodGFyZ2V0KSB7CiAgICAgY2FzZSBHcmFwaGlj
c0NvbnRleHRHTDo6VFJBTlNGT1JNX0ZFRURCQUNLX0JVRkZFUjoKLSAgICAgICAgaWYgKGluZGV4
ID4gbV9tYXhUcmFuc2Zvcm1GZWVkYmFja1NlcGFyYXRlQXR0cmlicykgeworICAgICAgICBpZiAo
aW5kZXggPj0gbV9tYXhUcmFuc2Zvcm1GZWVkYmFja1NlcGFyYXRlQXR0cmlicykgewogICAgICAg
ICAgICAgc3ludGhlc2l6ZUdMRXJyb3IoR3JhcGhpY3NDb250ZXh0R0w6OklOVkFMSURfVkFMVUUs
IGZ1bmN0aW9uTmFtZSwgImluZGV4IG91dCBvZiByYW5nZSIpOwogICAgICAgICAgICAgcmV0dXJu
IGZhbHNlOwogICAgICAgICB9Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>