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

Unified Diff: blimp/net/blimp_message_dispatcher.h

Issue 1429193002: Add interfaces for most major Blimp net components. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: blimp/net/blimp_message_dispatcher.h
diff --git a/blimp/net/blimp_message_dispatcher.h b/blimp/net/blimp_message_dispatcher.h
index bf12449df16c8931056a67060abdba45ba3570f8..e4f3dd6698d19e48a3e96280b318d1074ad28298 100644
--- a/blimp/net/blimp_message_dispatcher.h
+++ b/blimp/net/blimp_message_dispatcher.h
@@ -10,14 +10,14 @@
#include "base/containers/small_map.h"
#include "base/threading/thread_checker.h"
#include "blimp/common/proto/blimp_message.pb.h"
-#include "blimp/net/blimp_message_receiver.h"
+#include "blimp/net/blimp_message_processor.h"
#include "blimp/net/blimp_net_export.h"
namespace blimp {
-// Multiplexing BlimpMessageReceiver which routes BlimpMessages to other
-// receivers based on the messages' feature type.
-class BLIMP_NET_EXPORT BlimpMessageDispatcher : public BlimpMessageReceiver {
+// Multiplexing BlimpMessageProcessor which routes BlimpMessages to other
+// processors based on the messages' feature type.
+class BLIMP_NET_EXPORT BlimpMessageDispatcher : public BlimpMessageProcessor {
public:
BlimpMessageDispatcher();
~BlimpMessageDispatcher() override;
@@ -27,13 +27,14 @@ class BLIMP_NET_EXPORT BlimpMessageDispatcher : public BlimpMessageReceiver {
// Only one handler may be added per type.
//
// |handler| must be valid when OnBlimpMessage() is called.
- void AddReceiver(BlimpMessage::Type type, BlimpMessageReceiver* handler);
+ void AddReceiver(BlimpMessage::Type type, BlimpMessageProcessor* handler);
- // BlimpMessageReceiver implementation.
- net::Error OnBlimpMessage(const BlimpMessage& message) override;
+ // BlimpMessageProcessor implementation.
+ void ProcessMessage(const BlimpMessage& message,
+ const net::CompletionCallback& callback) override;
private:
- base::SmallMap<std::map<BlimpMessage::Type, BlimpMessageReceiver*>>
+ base::SmallMap<std::map<BlimpMessage::Type, BlimpMessageProcessor*>>
feature_receiver_map_;
DISALLOW_COPY_AND_ASSIGN(BlimpMessageDispatcher);

Powered by Google App Engine
This is Rietveld 408576698