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

Unified Diff: third_party/mojo/src/mojo/public/cpp/bindings/lib/router.cc

Issue 1410053006: Move third_party/mojo/src/mojo/public to mojo/public (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: merge 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: third_party/mojo/src/mojo/public/cpp/bindings/lib/router.cc
diff --git a/third_party/mojo/src/mojo/public/cpp/bindings/lib/router.cc b/third_party/mojo/src/mojo/public/cpp/bindings/lib/router.cc
deleted file mode 100644
index b28d2cefeee867e39c48e694e216c3073895e685..0000000000000000000000000000000000000000
--- a/third_party/mojo/src/mojo/public/cpp/bindings/lib/router.cc
+++ /dev/null
@@ -1,158 +0,0 @@
-// Copyright 2014 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "third_party/mojo/src/mojo/public/cpp/bindings/lib/router.h"
-
-#include "third_party/mojo/src/mojo/public/cpp/environment/logging.h"
-
-namespace mojo {
-namespace internal {
-
-// ----------------------------------------------------------------------------
-
-class ResponderThunk : public MessageReceiverWithStatus {
- public:
- explicit ResponderThunk(const SharedData<Router*>& router)
- : router_(router), accept_was_invoked_(false) {}
- ~ResponderThunk() override {
- if (!accept_was_invoked_) {
- // The Mojo application handled a message that was expecting a response
- // but did not send a response.
- Router* router = router_.value();
- if (router) {
- // We raise an error to signal the calling application that an error
- // condition occurred. Without this the calling application would have
- // no way of knowing it should stop waiting for a response.
- router->RaiseError();
- }
- }
- }
-
- // MessageReceiver implementation:
- bool Accept(Message* message) override {
- accept_was_invoked_ = true;
- MOJO_DCHECK(message->has_flag(kMessageIsResponse));
-
- bool result = false;
-
- Router* router = router_.value();
- if (router)
- result = router->Accept(message);
-
- return result;
- }
-
- // MessageReceiverWithStatus implementation:
- bool IsValid() override {
- Router* router = router_.value();
- return router && !router->encountered_error() && router->is_valid();
- }
-
- private:
- SharedData<Router*> router_;
- bool accept_was_invoked_;
-};
-
-// ----------------------------------------------------------------------------
-
-Router::HandleIncomingMessageThunk::HandleIncomingMessageThunk(Router* router)
- : router_(router) {
-}
-
-Router::HandleIncomingMessageThunk::~HandleIncomingMessageThunk() {
-}
-
-bool Router::HandleIncomingMessageThunk::Accept(Message* message) {
- return router_->HandleIncomingMessage(message);
-}
-
-// ----------------------------------------------------------------------------
-
-Router::Router(ScopedMessagePipeHandle message_pipe,
- FilterChain filters,
- const MojoAsyncWaiter* waiter)
- : thunk_(this),
- filters_(filters.Pass()),
- connector_(message_pipe.Pass(), waiter),
- weak_self_(this),
- incoming_receiver_(nullptr),
- next_request_id_(0),
- testing_mode_(false) {
- filters_.SetSink(&thunk_);
- connector_.set_incoming_receiver(filters_.GetHead());
-}
-
-Router::~Router() {
- weak_self_.set_value(nullptr);
-
- for (auto& pair : responders_)
- delete pair.second;
-}
-
-bool Router::Accept(Message* message) {
- MOJO_DCHECK(thread_checker_.CalledOnValidThread());
- MOJO_DCHECK(!message->has_flag(kMessageExpectsResponse));
- return connector_.Accept(message);
-}
-
-bool Router::AcceptWithResponder(Message* message, MessageReceiver* responder) {
- MOJO_DCHECK(thread_checker_.CalledOnValidThread());
- MOJO_DCHECK(message->has_flag(kMessageExpectsResponse));
-
- // Reserve 0 in case we want it to convey special meaning in the future.
- uint64_t request_id = next_request_id_++;
- if (request_id == 0)
- request_id = next_request_id_++;
-
- message->set_request_id(request_id);
- if (!connector_.Accept(message))
- return false;
-
- // We assume ownership of |responder|.
- responders_[request_id] = responder;
- return true;
-}
-
-void Router::EnableTestingMode() {
- MOJO_DCHECK(thread_checker_.CalledOnValidThread());
- testing_mode_ = true;
- connector_.set_enforce_errors_from_incoming_receiver(false);
-}
-
-bool Router::HandleIncomingMessage(Message* message) {
- MOJO_DCHECK(thread_checker_.CalledOnValidThread());
- if (message->has_flag(kMessageExpectsResponse)) {
- if (!incoming_receiver_)
- return false;
-
- MessageReceiverWithStatus* responder = new ResponderThunk(weak_self_);
- bool ok = incoming_receiver_->AcceptWithResponder(message, responder);
- if (!ok)
- delete responder;
- return ok;
-
- } else if (message->has_flag(kMessageIsResponse)) {
- uint64_t request_id = message->request_id();
- ResponderMap::iterator it = responders_.find(request_id);
- if (it == responders_.end()) {
- MOJO_DCHECK(testing_mode_);
- return false;
- }
- MessageReceiver* responder = it->second;
- responders_.erase(it);
- bool ok = responder->Accept(message);
- delete responder;
- return ok;
- } else {
- if (!incoming_receiver_)
- return false;
-
- return incoming_receiver_->Accept(message);
- }
-}
-
-// ----------------------------------------------------------------------------
-
-} // namespace internal
-} // namespace mojo

Powered by Google App Engine
This is Rietveld 408576698