Mojo JS bindings: draining a DataPipe

mojo/bindings/js/drain_data.h

Index: mojo/bindings/js/drain_data.h
diff --git a/mojo/bindings/js/drain_data.h b/mojo/bindings/js/drain_data.h
new file mode 100644
index 0000000000000000000000000000000000000000..11af4a76edfa800b11fd6e7941120fdcca5e26ea
--- /dev/null
+++ b/mojo/bindings/js/drain_data.h
@@ -0,0 +1,46 @@
+// Copyright 2014 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 MOJO_BINDINGS_JS_DRAIN_DATA_H_
+#define MOJO_BINDINGS_JS_DRAIN_DATA_H_
+
+#include "gin/runner.h"
+#include "mojo/public/c/environment/async_waiter.h"
+#include "mojo/public/cpp/system/core.h"
+#include "v8/include/v8.h"
+
+
+namespace mojo {
+namespace js {
+
+class DrainData {

[Matt Perry, 2014/09/24 00:39:42]

Please add some comments to this class and methods.

[hansmuller, 2014/09/24 17:47:53]

Done.

+ public:
+  DrainData(v8::Isolate* isolate, mojo::Handle handle);
+  v8::Handle<v8::Value> GetPromise();
+
+ private:
+  ~DrainData();
+
+  void WaitForData();
+  static void WaitCompleted(void* thisDrainData, MojoResult result) {

[Matt Perry, 2014/09/24 00:39:42]

this_drain_data (or maybe self?)

[hansmuller, 2014/09/24 17:47:53]

Done.

+    static_cast<DrainData*>(thisDrainData)->DataReady(result);
+  }
+  void DataReady(MojoResult result);
+  MojoResult ReadData();
+  void DeliverData(MojoResult result);
+
+  typedef std::vector<char> DataBuffer;
+
+  v8::Isolate* isolate_;
+  ScopedDataPipeConsumerHandle handle_;
+  MojoAsyncWaitID wait_id_;
+  base::WeakPtr<gin::Runner> runner_;
+  v8::UniquePersistent<v8::Promise::Resolver> resolver_;
+  std::vector<DataBuffer> data_buffers_;
+};
+
+}  // namespace js
+}  // namespace mojo
+
+#endif  // MOJO_BINDINGS_JS_DRAIN_DATA_H_