implement getBufferSubDataAsync prototype

third_party/WebKit/Source/modules/webgl/WebGL2RenderingContextBase.h

 // Copyright 2015 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.
 
 #ifndef WebGL2RenderingContextBase_h
 #define WebGL2RenderingContextBase_h
 
+#include "bindings/core/v8/ScriptPromise.h"
 #include "modules/webgl/WebGLExtension.h"
 #include "modules/webgl/WebGLRenderingContextBase.h"
 #include <memory>
 
 namespace blink {
 
 class WebGLTexture;
 
 class WebGLActiveInfo;
 class WebGLBuffer;
 class WebGLProgram;
 class WebGLQuery;
 class WebGLSampler;
 class WebGLSync;
 class WebGLTransformFeedback;
 class WebGLUniformLocation;
 class WebGLVertexArrayObject;
 
 class WebGL2RenderingContextBase : public WebGLRenderingContextBase {
  public:
   ~WebGL2RenderingContextBase() override;
 
+  void destroyContext() override;
+
   /* Buffer objects */
   void bufferData(GLenum, DOMArrayBufferView*, GLenum, GLuint, GLuint);
   void bufferSubData(GLenum, GLintptr, DOMArrayBufferView*, GLuint, GLuint);
   // Have to re-declare/re-define the following buffer{Sub}Data functions from
   // base class.  This is because the above buffer{Sub}Data() hides the name
   // from base class.
   void bufferData(GLenum target, long long size, GLenum usage);
   void bufferData(GLenum target, DOMArrayBuffer* data, GLenum usage);
   void bufferData(GLenum target, DOMArrayBufferView* data, GLenum usage);
   void bufferSubData(GLenum target, long long offset, DOMArrayBuffer* data);
   void bufferSubData(GLenum target,
                      long long offset,
                      const FlexibleArrayBufferView& data);
 
   void copyBufferSubData(GLenum, GLenum, long long, long long, long long);
   void getBufferSubData(GLenum, long long, DOMArrayBufferView*, GLuint, GLuint);
+  ScriptPromise getBufferSubDataAsync(ScriptState*,
+                                      GLenum,
+                                      GLintptr,
+                                      DOMArrayBufferView*,
+                                      GLuint,
+                                      GLuint);
 
   /* Framebuffer objects */
   bool validateTexFuncLayer(const char*, GLenum texTarget, GLint layer);
   void blitFramebuffer(GLint,
                        GLint,
                        GLint,
                        GLint,
                        GLint,
                        GLint,
                        GLint,
@@ skipping 748 matching lines @@
   // Helper function to check texture 3D target and texture bound to the target.
   // Generate GL errors and return 0 if target is invalid or texture bound is
   // null.  Otherwise, return the texture bound to the target.
   WebGLTexture* validateTexture3DBinding(const char* functionName,
                                          GLenum target);
 
   WebGLBuffer* validateBufferDataTarget(const char* functionName,
                                         GLenum target) override;
   bool validateBufferDataUsage(const char* functionName, GLenum usage) override;
 
+  class GetBufferSubDataAsyncCallback
+      : public RefCounted<GetBufferSubDataAsyncCallback> {

[esprehn, 2016/10/08 03:02:49]

Does this need to be a nested class?

[Kai Ninomiya, 2016/10/11 01:34:14]

I'll simplify this.

+   private:

[Ken Russell (switch to Gerrit), 2016/10/10 22:21:30]

Per the Google C++ style guide at
https://google.github.io/styleguide/cppguide.html#Declaration_Order , the public
members should go first in the class.

[Kai Ninomiya, 2016/10/11 01:34:14]

Done.

+    WeakPersistent<WebGL2RenderingContextBase> context;
+    Persistent<DOMArrayBufferView> dstData;

[esprehn, 2016/10/08 03:02:49]

We don't generally abbreviate in blink, does the spec use "dst" ?

[Kai Ninomiya, 2016/10/11 01:34:14]

Yes, the spec uses "dst." However, it does not need to be abbreviated this way
here, so I'm going to rename several things here.

+    Persistent<ScriptPromiseResolver> resolver;
+    void* subBaseAddress;

[Ken Russell (switch to Gerrit), 2016/10/10 22:21:30]

These variables could use some comments. Also, please group them together
according to relationship. dstData, subBaseAddress and subByteLength should be
together. mappedData should come either before or after, and have a separate
comment.

+    void* mappedData;
+    long long subByteLength;

[esprehn, 2016/10/08 03:02:49]

Very few things in blink can actually handle something as big as a long long,
ex. JS can't even represent that. What does this get used for?

[Ken Russell (switch to Gerrit), 2016/10/10 22:21:30]

It represents the amount of data read back from the GPU to the CPU. It could use
an OpenGL typedef like GLuint, which is what's used in the IDL and which is
typedef'd there to something representable in JavaScript. Kai, would you make
that change?

[Kai Ninomiya, 2016/10/11 01:34:14]

esprehn: Excellent point, I agree. If it's alright, I'm going to handle this in
another CL. This type comes from existing code (validateSubSourceAndGetData) and
I will need to change it and all of its (four) usages at once, as well as
closely inspect changes to validateSubSourceAndGetData, which is a security
sensitive range validation function.

+
+   public:
+    GetBufferSubDataAsyncCallback(WebGL2RenderingContextBase*,
+                                  DOMArrayBufferView*,
+                                  ScriptPromiseResolver*,
+                                  void*,
+                                  void*,
+                                  long long);
+    void destroy();
+    static void resolve(GetBufferSubDataAsyncCallback*);
+  };
+
+  std::set<GetBufferSubDataAsyncCallback*> m_getBufferSubDataAsyncCallbacks;
+
+  void registerGetBufferSubDataAsyncCallback(GetBufferSubDataAsyncCallback*);
+  void unregisterGetBufferSubDataAsyncCallback(GetBufferSubDataAsyncCallback*);
+
   bool canUseTexImageByGPU(TexImageFunctionID,
                            GLint internalformat,
                            GLenum type) override;
 
   void removeBoundBuffer(WebGLBuffer*) override;
 
   void resetUnpackParameters() override;
   void restoreUnpackParameters() override;
 
   void visitChildDOMWrappers(v8::Isolate*,
@@ skipping 43 matching lines @@
                   context,
                   context->is3d() &&
                       WebGLRenderingContextBase::getWebGLVersion(context) >= 2,
                   context.is3d() &&
                       WebGLRenderingContextBase::getWebGLVersion(&context) >=
                           2);
 
 }  // namespace blink
 
 #endif