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

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

Issue 10071038: RefCounted types should not have public destructors, content/browser part 2 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 8 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 | Annotate | Revision Log
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 "base/bind.h" 5 #include "base/bind.h"
6 #include "content/renderer/media/capture_video_decoder.h" 6 #include "content/renderer/media/capture_video_decoder.h"
7 #include "content/renderer/media/video_capture_impl.h" 7 #include "content/renderer/media/video_capture_impl.h"
8 #include "content/renderer/media/video_capture_impl_manager.h" 8 #include "content/renderer/media/video_capture_impl_manager.h"
9 #include "media/base/filters.h" 9 #include "media/base/filters.h"
10 #include "media/base/limits.h" 10 #include "media/base/limits.h"
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 decoder->OnStopped(vc_impl); 46 decoder->OnStopped(vc_impl);
47 } 47 }
48 48
49 class MockVideoCaptureImpl : public VideoCaptureImpl { 49 class MockVideoCaptureImpl : public VideoCaptureImpl {
50 public: 50 public:
51 MockVideoCaptureImpl(const media::VideoCaptureSessionId id, 51 MockVideoCaptureImpl(const media::VideoCaptureSessionId id,
52 scoped_refptr<base::MessageLoopProxy> ml_proxy, 52 scoped_refptr<base::MessageLoopProxy> ml_proxy,
53 VideoCaptureMessageFilter* filter) 53 VideoCaptureMessageFilter* filter)
54 : VideoCaptureImpl(id, ml_proxy, filter) { 54 : VideoCaptureImpl(id, ml_proxy, filter) {
55 } 55 }
56 virtual ~MockVideoCaptureImpl() {}
57 56
58 MOCK_METHOD2(StartCapture, 57 MOCK_METHOD2(StartCapture,
59 void(media::VideoCapture::EventHandler* handler, 58 void(media::VideoCapture::EventHandler* handler,
60 const VideoCaptureCapability& capability)); 59 const VideoCaptureCapability& capability));
61 MOCK_METHOD1(StopCapture, void(media::VideoCapture::EventHandler* handler)); 60 MOCK_METHOD1(StopCapture, void(media::VideoCapture::EventHandler* handler));
62 MOCK_METHOD1(FeedBuffer, void(scoped_refptr<VideoFrameBuffer> buffer)); 61 MOCK_METHOD1(FeedBuffer, void(scoped_refptr<VideoFrameBuffer> buffer));
63 62
64 private: 63 private:
65 DISALLOW_COPY_AND_ASSIGN(MockVideoCaptureImpl); 64 DISALLOW_COPY_AND_ASSIGN(MockVideoCaptureImpl);
66 }; 65 };
67 66
68 class MockVideoCaptureImplManager : public VideoCaptureImplManager { 67 class MockVideoCaptureImplManager : public VideoCaptureImplManager {
69 public: 68 public:
70 MockVideoCaptureImplManager() {} 69 MockVideoCaptureImplManager() {}
71 virtual ~MockVideoCaptureImplManager() {}
72 70
73 MOCK_METHOD2(AddDevice, 71 MOCK_METHOD2(AddDevice,
74 media::VideoCapture*(media::VideoCaptureSessionId id, 72 media::VideoCapture*(media::VideoCaptureSessionId id,
75 media::VideoCapture::EventHandler* handler)); 73 media::VideoCapture::EventHandler* handler));
76 MOCK_METHOD2(RemoveDevice, 74 MOCK_METHOD2(RemoveDevice,
77 void(media::VideoCaptureSessionId id, 75 void(media::VideoCaptureSessionId id,
78 media::VideoCapture::EventHandler* handler)); 76 media::VideoCapture::EventHandler* handler));
79 77
80 private: 78 private:
79 virtual ~MockVideoCaptureImplManager() {}
80
81 DISALLOW_COPY_AND_ASSIGN(MockVideoCaptureImplManager); 81 DISALLOW_COPY_AND_ASSIGN(MockVideoCaptureImplManager);
82 }; 82 };
83 83
84 class CaptureVideoDecoderTest : public ::testing::Test { 84 class CaptureVideoDecoderTest : public ::testing::Test {
85 protected: 85 protected:
86 CaptureVideoDecoderTest() { 86 CaptureVideoDecoderTest() {
87 message_loop_.reset(new MessageLoop(MessageLoop::TYPE_IO)); 87 message_loop_.reset(new MessageLoop(MessageLoop::TYPE_IO));
88 message_loop_proxy_ = 88 message_loop_proxy_ =
89 base::MessageLoopProxy::current().get(); 89 base::MessageLoopProxy::current().get();
90 vc_manager_ = new MockVideoCaptureImplManager(); 90 vc_manager_ = new MockVideoCaptureImplManager();
(...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after
210 params.session_id = kVideoStreamId; 210 params.session_id = kVideoStreamId;
211 211
212 decoder_->OnDeviceInfoReceived(vc_impl_.get(), params); 212 decoder_->OnDeviceInfoReceived(vc_impl_.get(), params);
213 message_loop_->RunAllPending(); 213 message_loop_->RunAllPending();
214 214
215 EXPECT_EQ(expected_size.width(), decoder_->natural_size().width()); 215 EXPECT_EQ(expected_size.width(), decoder_->natural_size().width());
216 EXPECT_EQ(expected_size.height(), decoder_->natural_size().height()); 216 EXPECT_EQ(expected_size.height(), decoder_->natural_size().height());
217 217
218 Stop(); 218 Stop();
219 } 219 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698