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

Unified Diff: mojo/public/cpp/bindings/lib/control_message_proxy.h

Issue 2649573004: Mojo bindings: merge the control messages of sending disconnect reason and notifying endpoint close… (Closed)
Patch Set: Created 3 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: mojo/public/cpp/bindings/lib/control_message_proxy.h
diff --git a/mojo/public/cpp/bindings/lib/control_message_proxy.h b/mojo/public/cpp/bindings/lib/control_message_proxy.h
index d2f154e77b5d51e59f3a2abb2dd6bbd427b2d364..2f9314ebf0023edf3646ce62e77e969dc0a72489 100644
--- a/mojo/public/cpp/bindings/lib/control_message_proxy.h
+++ b/mojo/public/cpp/bindings/lib/control_message_proxy.h
@@ -7,16 +7,15 @@
#include <stdint.h>
-#include <string>
-
#include "base/callback.h"
#include "base/macros.h"
#include "mojo/public/cpp/bindings/bindings_export.h"
#include "mojo/public/cpp/bindings/lib/serialization_context.h"
-#include "mojo/public/cpp/bindings/message.h"
namespace mojo {
+class MessageReceiverWithResponder;
+
namespace internal {
// Proxy for request messages defined in interface_control_messages.mojom.
@@ -30,15 +29,8 @@ class MOJO_CPP_BINDINGS_EXPORT ControlMessageProxy {
void RequireVersion(uint32_t version);
void FlushForTesting();
- void SendDisconnectReason(uint32_t custom_reason,
- const std::string& description);
-
void OnConnectionError();
- static Message ConstructDisconnectReasonMessage(
- uint32_t custom_reason,
- const std::string& description);
-
private:
void RunFlushForTestingClosure();

Powered by Google App Engine
This is Rietveld 408576698