Chromium Code Reviews| Index: mojo/public/cpp/bindings/lib/pipe_control_message_proxy.cc |
| diff --git a/mojo/public/cpp/bindings/lib/pipe_control_message_proxy.cc b/mojo/public/cpp/bindings/lib/pipe_control_message_proxy.cc |
| index 701108e1637cb7dd40e66fb6d7ade1c38301bf4d..1e691e35fdfe62f15b4d56864316277d5b8e9f13 100644 |
| --- a/mojo/public/cpp/bindings/lib/pipe_control_message_proxy.cc |
| +++ b/mojo/public/cpp/bindings/lib/pipe_control_message_proxy.cc |
| @@ -9,6 +9,7 @@ |
| #include "base/compiler_specific.h" |
|
yzshen1
2017/03/21 16:10:00
Is this still needed?
watk
2017/03/22 07:09:15
Removed, thanks
|
| #include "base/logging.h" |
| +#include "base/macros.h" |
| #include "mojo/public/cpp/bindings/lib/message_builder.h" |
| #include "mojo/public/cpp/bindings/lib/serialization.h" |
| #include "mojo/public/interfaces/bindings/pipe_control_messages.mojom.h" |
| @@ -44,8 +45,7 @@ void PipeControlMessageProxy::NotifyPeerEndpointClosed( |
| InterfaceId id, |
| const base::Optional<DisconnectReason>& reason) { |
| Message message(ConstructPeerEndpointClosedMessage(id, reason)); |
| - bool ok = receiver_->Accept(&message); |
| - ALLOW_UNUSED_LOCAL(ok); |
| + ignore_result(receiver_->Accept(&message)); |
| } |
| // static |