Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(615)

Side by Side Diff: mojo/services/network/public/cpp/web_socket_write_queue.h

Issue 1148913002: Fix WebSocket{Read,Write}Queue. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef MOJO_SERVICES_NETWORK_PUBLIC_CPP_WEB_SOCKET_WRITE_QUEUE_H_ 5 #ifndef MOJO_SERVICES_NETWORK_PUBLIC_CPP_WEB_SOCKET_WRITE_QUEUE_H_
6 #define MOJO_SERVICES_NETWORK_PUBLIC_CPP_WEB_SOCKET_WRITE_QUEUE_H_ 6 #define MOJO_SERVICES_NETWORK_PUBLIC_CPP_WEB_SOCKET_WRITE_QUEUE_H_
7 7
8 #include "base/callback.h" 8 #include "base/callback.h"
9 #include "base/memory/scoped_vector.h" 9 #include "base/memory/scoped_vector.h"
10 #include "base/memory/weak_ptr.h"
10 #include "mojo/common/handle_watcher.h" 11 #include "mojo/common/handle_watcher.h"
11 #include "third_party/mojo/src/mojo/public/cpp/system/data_pipe.h" 12 #include "third_party/mojo/src/mojo/public/cpp/system/data_pipe.h"
12 13
13 namespace mojo { 14 namespace mojo {
14 15
15 // This class simplifies the handling of multiple Writes on a DataPipe. It 16 // This class simplifies the handling of multiple Writes on a DataPipe. It
16 // writes each chunk all at once (or waits until the pipe is ready before 17 // writes each chunk all at once (or waits until the pipe is ready before
17 // writing), calling the callback when finished. Callbacks are owned by this 18 // writing), calling the callback when finished. Callbacks are owned by this
18 // class, and are guaranteed not to be called after this class is destroyed. 19 // class, and are guaranteed not to be called after this class is destroyed.
19 // See also: WebSocketReadQueue 20 // See also: WebSocketReadQueue
20 class WebSocketWriteQueue { 21 class WebSocketWriteQueue {
21 public: 22 public:
22 WebSocketWriteQueue(DataPipeProducerHandle handle); 23 explicit WebSocketWriteQueue(DataPipeProducerHandle handle);
23 ~WebSocketWriteQueue(); 24 ~WebSocketWriteQueue();
24 25
25 void Write(const char* data, 26 void Write(const char* data,
26 uint32_t num_bytes, 27 uint32_t num_bytes,
27 base::Callback<void(const char*)> callback); 28 base::Callback<void(const char*)> callback);
28 29
29 private: 30 private:
30 struct Operation; 31 struct Operation;
31 32
32 MojoResult TryToWrite(); 33 void TryToWrite();
33 void Wait(); 34 void Wait();
34 void OnHandleReady(MojoResult result); 35 void OnHandleReady(MojoResult result);
35 36
36 DataPipeProducerHandle handle_; 37 DataPipeProducerHandle handle_;
37 common::HandleWatcher handle_watcher_; 38 common::HandleWatcher handle_watcher_;
38 ScopedVector<Operation> queue_; 39 ScopedVector<Operation> queue_;
39 bool is_waiting_; 40 bool is_busy_;
41 base::WeakPtrFactory<WebSocketWriteQueue> weak_factory_;
40 }; 42 };
41 43
42 } // namespace mojo 44 } // namespace mojo
43 45
44 #endif // MOJO_SERVICES_NETWORK_PUBLIC_CPP_WEB_SOCKET_WRITE_QUEUE_H_ 46 #endif // MOJO_SERVICES_NETWORK_PUBLIC_CPP_WEB_SOCKET_WRITE_QUEUE_H_
OLDNEW
« no previous file with comments | « mojo/services/network/public/cpp/web_socket_read_queue.cc ('k') | mojo/services/network/public/cpp/web_socket_write_queue.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698