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

Side by Side Diff: mojo/public/cpp/bindings/tests/router_unittest.cc

Issue 1460463003: Move mojo::internal::MessageBuilder out of the |internal| namespace. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: More comments for MessageBuilder + a unittest. 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <stdlib.h> 5 #include <stdlib.h>
6 #include <string.h> 6 #include <string.h>
7 7
8 #include "mojo/public/cpp/bindings/lib/message_builder.h" 8 #include "mojo/public/cpp/bindings/lib/message_builder.h"
9 #include "mojo/public/cpp/bindings/lib/router.h" 9 #include "mojo/public/cpp/bindings/lib/router.h"
10 #include "mojo/public/cpp/bindings/tests/message_queue.h" 10 #include "mojo/public/cpp/bindings/tests/message_queue.h"
11 #include "mojo/public/cpp/environment/environment.h" 11 #include "mojo/public/cpp/environment/environment.h"
12 #include "mojo/public/cpp/system/macros.h" 12 #include "mojo/public/cpp/system/macros.h"
13 #include "mojo/public/cpp/utility/run_loop.h" 13 #include "mojo/public/cpp/utility/run_loop.h"
14 #include "testing/gtest/include/gtest/gtest.h" 14 #include "testing/gtest/include/gtest/gtest.h"
15 15
16 namespace mojo { 16 namespace mojo {
17 namespace test { 17 namespace test {
18 namespace { 18 namespace {
19 19
20 void AllocRequestMessage(uint32_t name, const char* text, Message* message) { 20 void AllocRequestMessage(uint32_t name, const char* text, Message* message) {
21 size_t payload_size = strlen(text) + 1; // Plus null terminator. 21 size_t payload_size = strlen(text) + 1; // Plus null terminator.
22 internal::RequestMessageBuilder builder(name, payload_size); 22 RequestMessageBuilder builder(name, payload_size);
23 memcpy(builder.buffer()->Allocate(payload_size), text, payload_size); 23 memcpy(builder.buffer()->Allocate(payload_size), text, payload_size);
24 24
25 builder.message()->MoveTo(message); 25 builder.message()->MoveTo(message);
26 } 26 }
27 27
28 void AllocResponseMessage(uint32_t name, 28 void AllocResponseMessage(uint32_t name,
29 const char* text, 29 const char* text,
30 uint64_t request_id, 30 uint64_t request_id,
31 Message* message) { 31 Message* message) {
32 size_t payload_size = strlen(text) + 1; // Plus null terminator. 32 size_t payload_size = strlen(text) + 1; // Plus null terminator.
33 internal::ResponseMessageBuilder builder(name, payload_size, request_id); 33 ResponseMessageBuilder builder(name, payload_size, request_id);
34 memcpy(builder.buffer()->Allocate(payload_size), text, payload_size); 34 memcpy(builder.buffer()->Allocate(payload_size), text, payload_size);
35 35
36 builder.message()->MoveTo(message); 36 builder.message()->MoveTo(message);
37 } 37 }
38 38
39 class MessageAccumulator : public MessageReceiver { 39 class MessageAccumulator : public MessageReceiver {
40 public: 40 public:
41 explicit MessageAccumulator(MessageQueue* queue) : queue_(queue) {} 41 explicit MessageAccumulator(MessageQueue* queue) : queue_(queue) {}
42 42
43 bool Accept(Message* message) override { 43 bool Accept(Message* message) override {
(...skipping 321 matching lines...) Expand 10 before | Expand all | Expand 10 after
365 EXPECT_TRUE(generator.has_responder()); 365 EXPECT_TRUE(generator.has_responder());
366 } 366 }
367 367
368 EXPECT_FALSE(generator.responder_is_valid()); 368 EXPECT_FALSE(generator.responder_is_valid());
369 generator.CompleteWithResponse(); // This should end up doing nothing. 369 generator.CompleteWithResponse(); // This should end up doing nothing.
370 } 370 }
371 371
372 } // namespace 372 } // namespace
373 } // namespace test 373 } // namespace test
374 } // namespace mojo 374 } // namespace mojo
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698