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

Unified Diff: media/capture/video/video_capture_device_unittest.cc

Issue 2699073002: Make MockImageCaptureClient RefCountedThreadSafe (Closed)
Patch Set: rebase Created 3 years, 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/capture/video/video_capture_device_unittest.cc
diff --git a/media/capture/video/video_capture_device_unittest.cc b/media/capture/video/video_capture_device_unittest.cc
index 059a129d3ed955521da414c409f4f8d720820005..477c911a4f411aee8b70a9835f2049b7a610ec6e 100644
--- a/media/capture/video/video_capture_device_unittest.cc
+++ b/media/capture/video/video_capture_device_unittest.cc
@@ -158,7 +158,8 @@ class MockVideoCaptureClient : public VideoCaptureDevice::Client {
base::Callback<void(const VideoCaptureFormat&)> frame_cb_;
};
-class MockImageCaptureClient : public base::RefCounted<MockImageCaptureClient> {
+class MockImageCaptureClient
+ : public base::RefCountedThreadSafe<MockImageCaptureClient> {
public:
// GMock doesn't support move-only arguments, so we use this forward method.
void DoOnPhotoTaken(mojom::BlobPtr blob) {
@@ -197,7 +198,7 @@ class MockImageCaptureClient : public base::RefCounted<MockImageCaptureClient> {
const mojom::PhotoCapabilities* capabilities() { return capabilities_.get(); }
private:
- friend class base::RefCounted<MockImageCaptureClient>;
+ friend class base::RefCountedThreadSafe<MockImageCaptureClient>;
virtual ~MockImageCaptureClient() {}
mojom::PhotoCapabilitiesPtr capabilities_;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698