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

Unified Diff: mojo/edk/system/child_broker.cc

Issue 1465183005: Rename mojo::TokenSerializer to mojo::Broker. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix win component 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: mojo/edk/system/child_broker.cc
diff --git a/mojo/edk/system/child_token_serializer_win.cc b/mojo/edk/system/child_broker.cc
similarity index 61%
rename from mojo/edk/system/child_token_serializer_win.cc
rename to mojo/edk/system/child_broker.cc
index 2e3f78eb309c44aaabd908001e711c9e923aa1bf..6523c939cce3e3f3d551725ce31b10fbcd0b7cd1 100644
--- a/mojo/edk/system/child_token_serializer_win.cc
+++ b/mojo/edk/system/child_broker.cc
@@ -2,31 +2,30 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "mojo/edk/system/child_token_serializer_win.h"
+#include "mojo/edk/system/child_broker.h"
#include "base/logging.h"
#include "mojo/edk/embedder/embedder_internal.h"
-#include "mojo/edk/system/token_serializer_messages_win.h"
+#include "mojo/edk/system/broker_messages.h"
namespace mojo {
namespace edk {
-
-ChildTokenSerializer* ChildTokenSerializer::GetInstance() {
- return base::Singleton<
- ChildTokenSerializer,
- base::LeakySingletonTraits<ChildTokenSerializer>>::get();
+
+ChildBroker* ChildBroker::GetInstance() {
+ return base::Singleton<
+ ChildBroker, base::LeakySingletonTraits<ChildBroker>>::get();
}
-void ChildTokenSerializer::SetParentTokenSerializerHandle(
- ScopedPlatformHandle handle) {
+void ChildBroker::SetChildBrokerHostHandle(ScopedPlatformHandle handle) {
handle_ = handle.Pass();
lock_.Unlock();
}
-void ChildTokenSerializer::CreatePlatformChannelPair(
+#if defined(OS_WIN)
+void ChildBroker::CreatePlatformChannelPair(
ScopedPlatformHandle* server, ScopedPlatformHandle* client) {
- TokenSerializerMessage message;
- message.size = kTokenSerializerMessageHeaderSize;
+ BrokerMessage message;
+ message.size = kBrokerMessageHeaderSize;
message.id = CREATE_PLATFORM_CHANNEL_PAIR;
uint32_t response_size = 2 * sizeof(HANDLE);
@@ -37,14 +36,13 @@ void ChildTokenSerializer::CreatePlatformChannelPair(
}
}
-void ChildTokenSerializer::HandleToToken(const PlatformHandle* platform_handles,
- size_t count,
- uint64_t* tokens) {
- uint32_t size = kTokenSerializerMessageHeaderSize +
- static_cast<int>(count) * sizeof(HANDLE);
+void ChildBroker::HandleToToken(const PlatformHandle* platform_handles,
+ size_t count,
+ uint64_t* tokens) {
+ uint32_t size = kBrokerMessageHeaderSize +
+ static_cast<int>(count) * sizeof(HANDLE);
std::vector<char> message_buffer(size);
- TokenSerializerMessage* message =
- reinterpret_cast<TokenSerializerMessage*>(&message_buffer[0]);
+ BrokerMessage* message = reinterpret_cast<BrokerMessage*>(&message_buffer[0]);
message->size = size;
message->id = HANDLE_TO_TOKEN;
for (size_t i = 0; i < count; ++i)
@@ -54,14 +52,14 @@ void ChildTokenSerializer::HandleToToken(const PlatformHandle* platform_handles,
WriteAndReadResponse(message, tokens, response_size);
}
-void ChildTokenSerializer::TokenToHandle(const uint64_t* tokens,
- size_t count,
- PlatformHandle* handles) {
- uint32_t size = kTokenSerializerMessageHeaderSize +
+void ChildBroker::TokenToHandle(const uint64_t* tokens,
+ size_t count,
+ PlatformHandle* handles) {
+ uint32_t size = kBrokerMessageHeaderSize +
static_cast<int>(count) * sizeof(uint64_t);
std::vector<char> message_buffer(size);
- TokenSerializerMessage* message =
- reinterpret_cast<TokenSerializerMessage*>(&message_buffer[0]);
+ BrokerMessage* message =
+ reinterpret_cast<BrokerMessage*>(&message_buffer[0]);
message->size = size;
message->id = TOKEN_TO_HANDLE;
memcpy(&message->tokens[0], tokens, count * sizeof(uint64_t));
@@ -74,24 +72,26 @@ void ChildTokenSerializer::TokenToHandle(const uint64_t* tokens,
handles[i].handle = handles_temp[i];
}
}
+#endif
-ChildTokenSerializer::ChildTokenSerializer() {
- DCHECK(!internal::g_token_serializer);
- internal::g_token_serializer = this;
+ChildBroker::ChildBroker() {
+ DCHECK(!internal::g_broker);
+ internal::g_broker = this;
// Block any threads from calling this until we have a pipe to the parent.
lock_.Lock();
}
-ChildTokenSerializer::~ChildTokenSerializer() {
+ChildBroker::~ChildBroker() {
}
-bool ChildTokenSerializer::WriteAndReadResponse(TokenSerializerMessage* message,
- void* response,
- uint32_t response_size) {
+bool ChildBroker::WriteAndReadResponse(BrokerMessage* message,
+ void* response,
+ uint32_t response_size) {
lock_.Lock();
CHECK(handle_.is_valid());
bool result = true;
+#if defined(OS_WIN)
DWORD bytes_written = 0;
// This will always write in one chunk per
// https://msdn.microsoft.com/en-us/library/windows/desktop/aa365150.aspx.
@@ -114,6 +114,7 @@ bool ChildTokenSerializer::WriteAndReadResponse(TokenSerializerMessage* message,
response = static_cast<char*>(response) + bytes_read;
}
}
+#endif
lock_.Unlock();

Powered by Google App Engine
This is Rietveld 408576698