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

Unified Diff: mojo/public/cpp/bindings/lib/binding_state.cc

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/binding_state.cc
diff --git a/mojo/public/cpp/bindings/lib/binding_state.cc b/mojo/public/cpp/bindings/lib/binding_state.cc
index 0ec445e89c924002cb37c9d786af55779bf4f030..c98fc6fec8707d22a8c0ef152dadc45e98c9e71b 100644
--- a/mojo/public/cpp/bindings/lib/binding_state.cc
+++ b/mojo/public/cpp/bindings/lib/binding_state.cc
@@ -48,8 +48,8 @@ void BindingStateBase::Close() {
void BindingStateBase::CloseWithReason(uint32_t custom_reason,
const std::string& description) {
if (endpoint_client_)
- endpoint_client_->control_message_proxy()->SendDisconnectReason(
- custom_reason, description);
+ endpoint_client_->CloseWithReason(custom_reason, description);
+
Close();
}

Powered by Google App Engine
This is Rietveld 408576698