WebGL 2: fix the bug in invalidate-framebuffer.html

gpu/command_buffer/build_gles2_cmd_buffer.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """code generator for GLES2 command buffers."""
 
 import itertools
 import os
 import os.path
@@ skipping 585 matching lines @@
       'GL_FRAMEBUFFER',
     ],
     'valid_es3': [
       'GL_DRAW_FRAMEBUFFER' ,
       'GL_READ_FRAMEBUFFER' ,
     ],
     'invalid': [
       'GL_RENDERBUFFER',
     ],
   },
-  'InvalidateFrameBufferTarget': {

[Zhenyao Mo, 2015/11/13 03:18:11]

You should not remove this.  For invalidateFramebuffer(), only GL_FRAMEBUFFER.

[Ken Russell (switch to Gerrit), 2015/11/17 23:06:37]

Actually it looks like this was changed in ES spec 3.0.1 (see section E.5 of the
ES 3.0.4 spec). The spec allows all 3 values to be passed. The man page is out
of date. Filed Khronos internal bug 15181 about this.

[yunchao, 2015/11/19 01:42:59]

Agreed.

-    'type': 'GLenum',
-    'valid': [
-      'GL_FRAMEBUFFER',
-    ],
-    'invalid': [
-      'GL_DRAW_FRAMEBUFFER' ,
-      'GL_READ_FRAMEBUFFER' ,
-    ],
-  },
   'RenderBufferTarget': {
     'type': 'GLenum',
     'valid': [
       'GL_RENDERBUFFER',
     ],
     'invalid': [
       'GL_FRAMEBUFFER',
     ],
   },
   'BufferTarget': {
@@ skipping 10488 matching lines @@
   Format(gen.generated_cpp_filenames)
 
   if gen.errors > 0:
     print "%d errors" % gen.errors
     return 1
   return 0
 
 
 if __name__ == '__main__':
   sys.exit(main(sys.argv[1:]))