Change data pipe wrappers used by SerialConnection to use message pipe.

device/serial/data_sink_receiver.h

Index: device/serial/data_sink_receiver.h
diff --git a/device/serial/data_sink_receiver.h b/device/serial/data_sink_receiver.h
index 75d85e690e7dd1c27eb6a1116a76ae4f4ebba46c..230c9ef564002d9d5866f58a78213bcd7999e449 100644
--- a/device/serial/data_sink_receiver.h
+++ b/device/serial/data_sink_receiver.h
@@ -17,8 +17,6 @@
 
 namespace device {
 
-class AsyncWaiter;
-
 class DataSinkReceiver : public base::RefCounted<DataSinkReceiver>,
                          public mojo::InterfaceImpl<serial::DataSink> {
  public:
@@ -49,15 +47,14 @@ class DataSinkReceiver : public base::RefCounted<DataSinkReceiver>,
   virtual ~DataSinkReceiver();
 
   // mojo::InterfaceImpl<serial::DataSink> overrides.
-  virtual void Init(mojo::ScopedDataPipeConsumerHandle handle) override;
+  virtual void Init(uint32_t error) override;

[raymes, 2014/10/17 03:10:12]

is the parameter meant to be the buffer size?

[Sam McNally, 2014/10/20 05:12:59]

Done.

   virtual void Cancel(int32_t error) override;
+  virtual void AcceptData(mojo::Array<uint8_t> data) override;
   virtual void OnConnectionError() override;
 
-  // Starts waiting for |handle_| to be ready for reads.
-  void StartWaiting();
-
-  // Invoked when |handle_| is ready for reads.
-  void OnDoneWaiting(MojoResult result);
+  // Checks whether there is data ready to dispatch and dispatches it if there
+  // is.
+  void CheckForData();
 
   // Reports a successful read of |bytes_read|.
   void Done(uint32_t bytes_read);
@@ -72,17 +69,14 @@ class DataSinkReceiver : public base::RefCounted<DataSinkReceiver>,
   // Sends an ReportBytesSentAndError message to the client.
   void ReportBytesSentAndError(uint32_t bytes_read, int32_t error);
 
-  // Invoked in response to an ReportBytesSentAndError call to the client with
-  // the number of bytes to flush.
-  void SetNumBytesToFlush(uint32_t bytes_to_flush);
+  // Invoked in response to an ReportBytesSentAndError call to the client at
+  // the point in the data stream to flush.
+  void DoFlush();
 
   // Reports a fatal error to the client and shuts down.
   void DispatchFatalError();
 
-  // The data connection to the data sender.
-  mojo::ScopedDataPipeConsumerHandle handle_;
-
-  // The callback to call when |handle_| has data ready to read.
+  // The callback to call when there is data ready to read.
   const ReadyCallback ready_callback_;
 
   // The callback to call when the client has requested cancellation.
@@ -91,16 +85,23 @@ class DataSinkReceiver : public base::RefCounted<DataSinkReceiver>,
   // The callback to call if a fatal error occurs.
   const ErrorCallback error_callback_;
 
-  // The queue of pending flushes.
-  std::queue<linked_ptr<PendingFlush> > pending_flushes_;
-
-  // A waiter used to wait until |handle_| is readable if we are waiting.
-  scoped_ptr<AsyncWaiter> waiter_;
+  // Whether we are waiting for a flush.
+  bool flush_pending_;
 
   // The buffer passed to |ready_callback_| if one exists. This is not owned,
   // but the Buffer will call Done or DoneWithError before being deleted.
   Buffer* buffer_in_use_;
 
+  bool waiting_for_data_;
+
+  bool initialized_;
+
+  uint32_t available_buffer_capacity_;
+
+  std::queue<linked_ptr<mojo::Array<uint8_t>>> pending_data_buffers_;
+
+  uint32_t pending_data_offset_;

[raymes, 2014/10/17 03:10:12]

Please add comments for these.

[Sam McNally, 2014/10/20 05:12:59]

Done.

+
   // Whether we have encountered a fatal error and shut down.
   bool shut_down_;