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

Side by Side Diff: content/renderer/media/rtc_peer_connection_handler_unittest.cc

Issue 686523002: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 6 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 <string> 5 #include <string>
6 #include <vector> 6 #include <vector>
7 7
8 #include "base/memory/scoped_ptr.h" 8 #include "base/memory/scoped_ptr.h"
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 #include "base/strings/utf_string_conversions.h" 10 #include "base/strings/utf_string_conversions.h"
(...skipping 181 matching lines...) Expand 10 before | Expand all | Expand 10 after
192 RTCPeerConnectionHandler::native_peer_connection()); 192 RTCPeerConnectionHandler::native_peer_connection());
193 } 193 }
194 }; 194 };
195 195
196 class RTCPeerConnectionHandlerTest : public ::testing::Test { 196 class RTCPeerConnectionHandlerTest : public ::testing::Test {
197 public: 197 public:
198 RTCPeerConnectionHandlerTest() : mock_peer_connection_(NULL) { 198 RTCPeerConnectionHandlerTest() : mock_peer_connection_(NULL) {
199 child_process_.reset(new ChildProcess()); 199 child_process_.reset(new ChildProcess());
200 } 200 }
201 201
202 virtual void SetUp() { 202 void SetUp() override {
203 mock_client_.reset(new NiceMock<MockWebRTCPeerConnectionHandlerClient>()); 203 mock_client_.reset(new NiceMock<MockWebRTCPeerConnectionHandlerClient>());
204 mock_dependency_factory_.reset(new MockPeerConnectionDependencyFactory()); 204 mock_dependency_factory_.reset(new MockPeerConnectionDependencyFactory());
205 pc_handler_.reset( 205 pc_handler_.reset(
206 new RTCPeerConnectionHandlerUnderTest(mock_client_.get(), 206 new RTCPeerConnectionHandlerUnderTest(mock_client_.get(),
207 mock_dependency_factory_.get())); 207 mock_dependency_factory_.get()));
208 mock_tracker_.reset(new NiceMock<MockPeerConnectionTracker>()); 208 mock_tracker_.reset(new NiceMock<MockPeerConnectionTracker>());
209 blink::WebRTCConfiguration config; 209 blink::WebRTCConfiguration config;
210 blink::WebMediaConstraints constraints; 210 blink::WebMediaConstraints constraints;
211 EXPECT_TRUE(pc_handler_->InitializeForTest(config, constraints, 211 EXPECT_TRUE(pc_handler_->InitializeForTest(config, constraints,
212 mock_tracker_.get())); 212 mock_tracker_.get()));
213 213
214 mock_peer_connection_ = pc_handler_->native_peer_connection(); 214 mock_peer_connection_ = pc_handler_->native_peer_connection();
215 ASSERT_TRUE(mock_peer_connection_); 215 ASSERT_TRUE(mock_peer_connection_);
216 } 216 }
217 217
218 virtual void TearDown() { 218 void TearDown() override {
219 pc_handler_.reset(); 219 pc_handler_.reset();
220 mock_tracker_.reset(); 220 mock_tracker_.reset();
221 mock_dependency_factory_.reset(); 221 mock_dependency_factory_.reset();
222 mock_client_.reset(); 222 mock_client_.reset();
223 blink::WebHeap::collectAllGarbageForTesting(); 223 blink::WebHeap::collectAllGarbageForTesting();
224 } 224 }
225 225
226 // Creates a WebKit local MediaStream. 226 // Creates a WebKit local MediaStream.
227 blink::WebMediaStream CreateLocalMediaStream( 227 blink::WebMediaStream CreateLocalMediaStream(
228 const std::string& stream_label) { 228 const std::string& stream_label) {
(...skipping 637 matching lines...) Expand 10 before | Expand all | Expand 10 after
866 EXPECT_CALL(*mock_tracker_.get(), 866 EXPECT_CALL(*mock_tracker_.get(),
867 TrackCreateDTMFSender(pc_handler_.get(), 867 TrackCreateDTMFSender(pc_handler_.get(),
868 testing::Ref(tracks[0]))); 868 testing::Ref(tracks[0])));
869 869
870 scoped_ptr<blink::WebRTCDTMFSenderHandler> sender( 870 scoped_ptr<blink::WebRTCDTMFSenderHandler> sender(
871 pc_handler_->createDTMFSender(tracks[0])); 871 pc_handler_->createDTMFSender(tracks[0]));
872 EXPECT_TRUE(sender.get()); 872 EXPECT_TRUE(sender.get());
873 } 873 }
874 874
875 } // namespace content 875 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/media/render_media_log_unittest.cc ('k') | content/renderer/media/rtc_video_decoder_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698