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

Side by Side Diff: android_webview/native/permission/media_access_permission_request_unittest.cc

Issue 2291893002: Let Contraints Controll Mute/Unmute Audio Local Playback For Desktop Sharing (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Resolve Browser Test Issuex Created 4 years, 2 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
« no previous file with comments | « no previous file | chrome/browser/media/webrtc/desktop_capture_access_handler.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "android_webview/native/permission/media_access_permission_request.h" 5 #include "android_webview/native/permission/media_access_permission_request.h"
6 #include "base/bind.h" 6 #include "base/bind.h"
7 #include "testing/gtest/include/gtest/gtest.h" 7 #include "testing/gtest/include/gtest/gtest.h"
8 8
9 namespace android_webview { 9 namespace android_webview {
10 10
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 content::MediaStreamDevice(content::MEDIA_DEVICE_VIDEO_CAPTURE, 46 content::MediaStreamDevice(content::MEDIA_DEVICE_VIDEO_CAPTURE,
47 first_video_device_id_, "v2")); 47 first_video_device_id_, "v2"));
48 video_devices.push_back( 48 video_devices.push_back(
49 content::MediaStreamDevice(content::MEDIA_DEVICE_VIDEO_CAPTURE, 49 content::MediaStreamDevice(content::MEDIA_DEVICE_VIDEO_CAPTURE,
50 video_device_id_, "v1")); 50 video_device_id_, "v1"));
51 51
52 GURL origin("https://www.google.com"); 52 GURL origin("https://www.google.com");
53 content::MediaStreamRequest request( 53 content::MediaStreamRequest request(
54 0, 0, 0, origin, false, content::MEDIA_GENERATE_STREAM, audio_id, 54 0, 0, 0, origin, false, content::MEDIA_GENERATE_STREAM, audio_id,
55 video_id, content::MEDIA_DEVICE_AUDIO_CAPTURE, 55 video_id, content::MEDIA_DEVICE_AUDIO_CAPTURE,
56 content::MEDIA_DEVICE_VIDEO_CAPTURE); 56 content::MEDIA_DEVICE_VIDEO_CAPTURE, false);
boliu 2016/10/05 17:31:25 nit: maybe add an inline comment to make this a bi
qiangchen 2016/10/05 18:13:57 Done.
57 57
58 std::unique_ptr<TestMediaAccessPermissionRequest> permission_request; 58 std::unique_ptr<TestMediaAccessPermissionRequest> permission_request;
59 permission_request.reset(new TestMediaAccessPermissionRequest( 59 permission_request.reset(new TestMediaAccessPermissionRequest(
60 request, 60 request,
61 base::Bind(&MediaAccessPermissionRequestTest::Callback, 61 base::Bind(&MediaAccessPermissionRequestTest::Callback,
62 base::Unretained(this)), 62 base::Unretained(this)),
63 audio_devices, video_devices)); 63 audio_devices, video_devices));
64 return permission_request; 64 return permission_request;
65 } 65 }
66 66
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 129
130 TEST_F(MediaAccessPermissionRequestTest, TestDenyPermissionRequest) { 130 TEST_F(MediaAccessPermissionRequestTest, TestDenyPermissionRequest) {
131 std::unique_ptr<TestMediaAccessPermissionRequest> request = 131 std::unique_ptr<TestMediaAccessPermissionRequest> request =
132 CreateRequest(std::string(), std::string()); 132 CreateRequest(std::string(), std::string());
133 request->NotifyRequestResult(false); 133 request->NotifyRequestResult(false);
134 EXPECT_TRUE(devices_.empty()); 134 EXPECT_TRUE(devices_.empty());
135 EXPECT_EQ(content::MEDIA_DEVICE_PERMISSION_DENIED, result_); 135 EXPECT_EQ(content::MEDIA_DEVICE_PERMISSION_DENIED, result_);
136 } 136 }
137 137
138 } // namespace android_webview 138 } // namespace android_webview
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/media/webrtc/desktop_capture_access_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698