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

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

Powered by Google App Engine
This is Rietveld 408576698