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

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: Fix compile error after rebase 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 MOCK_METHOD0(OnClose, void()); 35 MOCK_METHOD0(OnClose, void());
35 }; 36 };
36 37
37 class BrowserPresentationConnectionProxyTest : public ::testing::Test { 38 class BrowserPresentationConnectionProxyTest : public ::testing::Test {
38 public: 39 public:
39 BrowserPresentationConnectionProxyTest() = default; 40 BrowserPresentationConnectionProxyTest() = default;
40 41
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 private: 75 private:
75 std::unique_ptr<MockPresentationConnectionProxy> 76 std::unique_ptr<MockPresentationConnectionProxy>
76 mock_controller_connection_proxy_; 77 mock_controller_connection_proxy_;
77 std::unique_ptr<BrowserPresentationConnectionProxy> browser_connection_proxy_; 78 std::unique_ptr<BrowserPresentationConnectionProxy> browser_connection_proxy_;
78 MockMediaRouter mock_router_; 79 MockMediaRouter mock_router_;
79 content::TestBrowserThreadBundle thread_bundle_; 80 content::TestBrowserThreadBundle thread_bundle_;
80 }; 81 };
81 82
82 TEST_F(BrowserPresentationConnectionProxyTest, TestOnMessageTextMessage) { 83 TEST_F(BrowserPresentationConnectionProxyTest, TestOnMessageTextMessage) {
83 std::string message = "test message"; 84 std::string message = "test message";
84 blink::mojom::ConnectionMessagePtr session_message = 85 content::PresentationConnectionMessage connection_message(message);
85 blink::mojom::ConnectionMessage::New();
86 session_message->type = blink::mojom::PresentationMessageType::TEXT;
87 session_message->message = message;
88 86
89 base::MockCallback<base::Callback<void(bool)>> mock_on_message_callback; 87 base::MockCallback<base::Callback<void(bool)>> mock_on_message_callback;
90 EXPECT_CALL(*mock_router(), SendRouteMessage(kMediaRouteId, message, _)); 88 EXPECT_CALL(*mock_router(), SendRouteMessage(kMediaRouteId, message, _));
91 89
92 browser_connection_proxy()->OnMessage(std::move(session_message), 90 browser_connection_proxy()->OnMessage(std::move(connection_message),
93 mock_on_message_callback.Get()); 91 mock_on_message_callback.Get());
94 } 92 }
95 93
96 TEST_F(BrowserPresentationConnectionProxyTest, TestOnMessageBinaryMessage) { 94 TEST_F(BrowserPresentationConnectionProxyTest, TestOnMessageBinaryMessage) {
97 std::vector<uint8_t> expected_data; 95 std::vector<uint8_t> expected_data;
98 expected_data.push_back(42); 96 expected_data.push_back(42);
99 expected_data.push_back(36); 97 expected_data.push_back(36);
100 98
101 blink::mojom::ConnectionMessagePtr session_message = 99 content::PresentationConnectionMessage connection_message(expected_data);
102 blink::mojom::ConnectionMessage::New();
103 session_message->type = blink::mojom::PresentationMessageType::BINARY;
104 session_message->data = expected_data;
105 100
106 base::MockCallback<base::Callback<void(bool)>> mock_on_message_callback; 101 base::MockCallback<base::Callback<void(bool)>> mock_on_message_callback;
107 EXPECT_CALL(*mock_router(), SendRouteBinaryMessageInternal(_, _, _)) 102 EXPECT_CALL(*mock_router(), SendRouteBinaryMessageInternal(_, _, _))
108 .WillOnce(::testing::Invoke([&expected_data]( 103 .WillOnce(::testing::Invoke(
109 const MediaRoute::Id& route_id, std::vector<uint8_t>* data, 104 [&expected_data](
110 const BrowserPresentationConnectionProxy::OnMessageCallback& 105 const MediaRoute::Id& route_id, std::vector<uint8_t>* data,
111 callback) { EXPECT_EQ(expected_data, *data); })); 106 const BrowserPresentationConnectionProxy::OnMessageCallback&
107 callback) { EXPECT_EQ(*data, expected_data); }));
112 108
113 browser_connection_proxy()->OnMessage(std::move(session_message), 109 browser_connection_proxy()->OnMessage(std::move(connection_message),
114 mock_on_message_callback.Get()); 110 mock_on_message_callback.Get());
115 } 111 }
116 112
117 } // namespace media_router 113 } // namespace media_router
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698