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

Side by Side Diff: chrome/browser/media/router/browser_presentation_connection_proxy_unittest.cc

Issue 2706463002: [Presentation API] Mojo typemap for content::PresentationConnectionMessage (Closed)
Patch Set: Respond to dcheng@ comments Created 3 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 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 "chrome/browser/media/router/browser_presentation_connection_proxy.h" 5 #include "chrome/browser/media/router/browser_presentation_connection_proxy.h"
6 6
7 #include "base/run_loop.h" 7 #include "base/run_loop.h"
8 #include "base/test/mock_callback.h" 8 #include "base/test/mock_callback.h"
9 #include "chrome/browser/media/router/media_source.h" 9 #include "chrome/browser/media/router/media_source.h"
10 #include "chrome/browser/media/router/media_source_helper.h" 10 #include "chrome/browser/media/router/media_source_helper.h"
11 #include "chrome/browser/media/router/mock_media_router.h" 11 #include "chrome/browser/media/router/mock_media_router.h"
12 #include "content/public/common/presentation_connection_message.h"
12 #include "content/public/test/test_browser_thread_bundle.h" 13 #include "content/public/test/test_browser_thread_bundle.h"
13 #include "testing/gmock/include/gmock/gmock.h" 14 #include "testing/gmock/include/gmock/gmock.h"
14 15
15 using ::testing::_; 16 using ::testing::_;
16 17
17 namespace media_router { 18 namespace media_router {
18 19
19 constexpr char kMediaRouteId[] = "MockRouteId"; 20 constexpr char kMediaRouteId[] = "MockRouteId";
20 21
21 class MockPresentationConnectionProxy 22 class MockPresentationConnectionProxy
22 : public NON_EXPORTED_BASE(blink::mojom::PresentationConnection) { 23 : public NON_EXPORTED_BASE(blink::mojom::PresentationConnection) {
23 public: 24 public:
24 void OnMessage(blink::mojom::ConnectionMessagePtr message, 25 // PresentationConnectionMessage is move-only.
25 const OnMessageCallback& on_message_callback) override { 26 void OnMessage(content::PresentationConnectionMessage message,
26 OnMessageRaw(message.get(), on_message_callback); 27 const OnMessageCallback& cb) {
28 OnMessageInternal(message, cb);
27 } 29 }
28 30 MOCK_METHOD2(OnMessageInternal,
29 MOCK_METHOD2(OnMessageRaw, 31 void(const content::PresentationConnectionMessage&,
30 void(const blink::mojom::ConnectionMessage*,
31 const OnMessageCallback&)); 32 const OnMessageCallback&));
32 MOCK_METHOD1(DidChangeState, 33 MOCK_METHOD1(DidChangeState,
33 void(content::PresentationConnectionState state)); 34 void(content::PresentationConnectionState state));
34 }; 35 };
35 36
36 class BrowserPresentationConnectionProxyTest : public ::testing::Test { 37 class BrowserPresentationConnectionProxyTest : public ::testing::Test {
37 public: 38 public:
38 BrowserPresentationConnectionProxyTest() = default; 39 BrowserPresentationConnectionProxyTest() = default;
39 40
40 void SetUp() override { 41 void SetUp() override {
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
73 private: 74 private:
74 std::unique_ptr<MockPresentationConnectionProxy> 75 std::unique_ptr<MockPresentationConnectionProxy>
75 mock_controller_connection_proxy_; 76 mock_controller_connection_proxy_;
76 std::unique_ptr<BrowserPresentationConnectionProxy> browser_connection_proxy_; 77 std::unique_ptr<BrowserPresentationConnectionProxy> browser_connection_proxy_;
77 MockMediaRouter mock_router_; 78 MockMediaRouter mock_router_;
78 content::TestBrowserThreadBundle thread_bundle_; 79 content::TestBrowserThreadBundle thread_bundle_;
79 }; 80 };
80 81
81 TEST_F(BrowserPresentationConnectionProxyTest, TestOnMessageTextMessage) { 82 TEST_F(BrowserPresentationConnectionProxyTest, TestOnMessageTextMessage) {
82 std::string message = "test message"; 83 std::string message = "test message";
83 blink::mojom::ConnectionMessagePtr session_message = 84 content::PresentationConnectionMessage connection_message(message);
84 blink::mojom::ConnectionMessage::New();
85 session_message->type = blink::mojom::PresentationMessageType::TEXT;
86 session_message->message = message;
87 85
88 base::MockCallback<base::Callback<void(bool)>> mock_on_message_callback; 86 base::MockCallback<base::Callback<void(bool)>> mock_on_message_callback;
89 EXPECT_CALL(*mock_router(), SendRouteMessage(kMediaRouteId, message, _)); 87 EXPECT_CALL(*mock_router(), SendRouteMessage(kMediaRouteId, message, _));
90 88
91 browser_connection_proxy()->OnMessage(std::move(session_message), 89 browser_connection_proxy()->OnMessage(std::move(connection_message),
92 mock_on_message_callback.Get()); 90 mock_on_message_callback.Get());
93 } 91 }
94 92
95 TEST_F(BrowserPresentationConnectionProxyTest, TestOnMessageBinaryMessage) { 93 TEST_F(BrowserPresentationConnectionProxyTest, TestOnMessageBinaryMessage) {
96 std::vector<uint8_t> expected_data; 94 std::vector<uint8_t> expected_data;
97 expected_data.push_back(42); 95 expected_data.push_back(42);
98 expected_data.push_back(36); 96 expected_data.push_back(36);
99 97
100 blink::mojom::ConnectionMessagePtr session_message = 98 content::PresentationConnectionMessage connection_message(expected_data);
101 blink::mojom::ConnectionMessage::New();
102 session_message->type = blink::mojom::PresentationMessageType::BINARY;
103 session_message->data = expected_data;
104 99
105 base::MockCallback<base::Callback<void(bool)>> mock_on_message_callback; 100 base::MockCallback<base::Callback<void(bool)>> mock_on_message_callback;
106 EXPECT_CALL(*mock_router(), SendRouteBinaryMessageInternal(_, _, _)) 101 EXPECT_CALL(*mock_router(), SendRouteBinaryMessage(_, _, _))
107 .WillOnce(::testing::Invoke([&expected_data]( 102 .WillOnce(::testing::Invoke(
108 const MediaRoute::Id& route_id, std::vector<uint8_t>* data, 103 [&expected_data](
109 const BrowserPresentationConnectionProxy::OnMessageCallback& 104 const MediaRoute::Id& route_id, const std::vector<uint8_t>& data,
110 callback) { EXPECT_EQ(expected_data, *data); })); 105 const BrowserPresentationConnectionProxy::OnMessageCallback&
106 callback) { EXPECT_EQ(data, expected_data); }));
111 107
112 browser_connection_proxy()->OnMessage(std::move(session_message), 108 browser_connection_proxy()->OnMessage(std::move(connection_message),
113 mock_on_message_callback.Get()); 109 mock_on_message_callback.Get());
114 } 110 }
115 111
116 } // namespace media_router 112 } // namespace media_router
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698