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

Unified Diff: content/renderer/media/user_media_client_impl.h

Issue 2941563002: Enable new getUserMedia audio constraints algorithm behind a flag. (Closed)
Patch Set: Created 3 years, 6 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
Index: content/renderer/media/user_media_client_impl.h
diff --git a/content/renderer/media/user_media_client_impl.h b/content/renderer/media/user_media_client_impl.h
index 0e4f587aef068442872418923581d8887bebd493..b7e92eb79f03546368403c542f3d612987cf3713 100644
--- a/content/renderer/media/user_media_client_impl.h
+++ b/content/renderer/media/user_media_client_impl.h
@@ -37,10 +37,11 @@ class TaskRunner;
}
namespace content {
-class PeerConnectionDependencyFactory;
+class AudioCaptureSettings;
class MediaStreamAudioSource;
class MediaStreamDispatcher;
class MediaStreamVideoSource;
+class PeerConnectionDependencyFactory;
class VideoCaptureSettings;
// UserMediaClientImpl is a delegate for the Media Stream GetUserMedia API.
@@ -121,10 +122,13 @@ class CONTENT_EXPORT UserMediaClientImpl
// Returns no value if there is no request being processed. Use only for
// testing.
- // TODO(guidou): Remove this method once spec-compliant constraints algorithm
- // for audio is implemented. http://crbug.com/543997
+ // TODO(guidou): Remove this function. http://crbug.com/706408
base::Optional<bool> AutomaticOutputDeviceSelectionEnabledForCurrentRequest();
+ // Intended to be used only for testing.
+ const AudioCaptureSettings& AudioCaptureSettingsForCurrentRequest() const;
+ const VideoCaptureSettings& VideoCaptureSettingsForCurrentRequest() const;
+
private:
class UserMediaRequestInfo;
typedef std::vector<blink::WebMediaStreamSource> LocalStreamSources;
@@ -225,21 +229,25 @@ class CONTENT_EXPORT UserMediaClientImpl
const ::mojom::MediaDevicesDispatcherHostPtr& GetMediaDevicesDispatcher();
- void SelectAudioInputDevice(
+ // TODO(guidou): Remove these functions. http://crbug.com/706408
+ void LegacySetupAudioInput();
+ void LegacySelectAudioInputDevice(
const blink::WebUserMediaRequest& user_media_request,
const EnumerationResult& device_enumeration);
- void SetupVideoInput(const blink::WebUserMediaRequest& user_media_request);
+ void SetupAudioInput(const blink::WebUserMediaRequest& user_media_request);
+ void SelectAudioSettings(const blink::WebUserMediaRequest& user_media_request,
+ std::vector<::mojom::AudioInputDeviceCapabilitiesPtr>
+ audio_input_capabilities);
+ void SetupVideoInput(const blink::WebUserMediaRequest& user_media_request);
void SelectVideoDeviceSettings(
const blink::WebUserMediaRequest& user_media_request,
std::vector<::mojom::VideoInputDeviceCapabilitiesPtr>
video_input_capabilities);
-
void FinalizeSelectVideoDeviceSettings(
const blink::WebUserMediaRequest& user_media_request,
const VideoCaptureSettings& settings);
-
void FinalizeSelectVideoContentSettings(
const blink::WebUserMediaRequest& user_media_request,
const VideoCaptureSettings& settings);

Powered by Google App Engine
This is Rietveld 408576698