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

Unified Diff: content/browser/renderer_host/media/render_frame_audio_output_service.cc

Issue 2319493002: Add mojo interface for audio rendering. (Closed)
Patch Set: ++docs 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
Index: content/browser/renderer_host/media/render_frame_audio_output_service.cc
diff --git a/content/browser/renderer_host/media/render_frame_audio_output_service.cc b/content/browser/renderer_host/media/render_frame_audio_output_service.cc
new file mode 100644
index 0000000000000000000000000000000000000000..5a16635bc34109aa6ab0607e08d2d05434dca64c
--- /dev/null
+++ b/content/browser/renderer_host/media/render_frame_audio_output_service.cc
@@ -0,0 +1,129 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "content/browser/renderer_host/media/render_frame_audio_output_service.h"
+
+#include <utility>
+
+#include "base/metrics/histogram_macros.h"
+#include "base/threading/thread_checker.h"
+#include "content/browser/bad_message.h"
+#include "content/browser/media/capture/audio_mirroring_manager.h"
+#include "content/browser/renderer_host/media/audio_output_impl.h"
+#include "content/browser/renderer_host/media/audio_output_service_context.h"
+#include "content/browser/renderer_host/media/media_stream_manager.h"
+#include "content/common/media/audio_output.mojom.h"
+#include "media/audio/audio_manager.h"
+#include "media/base/audio_parameters.h"
+#include "mojo/public/cpp/bindings/binding.h"
+
+namespace content {
+
+namespace {
+
+void UMALogDeviceAuthorizationTime(base::TimeTicks auth_start_time) {
+ UMA_HISTOGRAM_CUSTOM_TIMES("Media.Audio.OutputDeviceAuthorizationTime",
+ base::TimeTicks::Now() - auth_start_time,
+ base::TimeDelta::FromMilliseconds(1),
+ base::TimeDelta::FromMilliseconds(5000), 50);
+}
+
+} // namespace
+
+RenderFrameAudioOutputService::RenderFrameAudioOutputService(
+ AudioOutputServiceContext* context,
+ int render_frame_id,
+ mojom::RendererAudioOutputServiceRequest request)
+ : binding_(this, std::move(request)),
+ context_(context),
+ render_frame_id_(render_frame_id) {
+ DCHECK(thread_checker_.CalledOnValidThread());
+ CHECK(context_);
+ binding_.set_connection_error_handler(
+ base::Bind(&AudioOutputServiceContext::OnServiceFinished,
+ base::Unretained(context), base::Unretained(this)));
+}
+
+RenderFrameAudioOutputService::~RenderFrameAudioOutputService() {
+ DCHECK(thread_checker_.CalledOnValidThread());
+}
+
+void RenderFrameAudioOutputService::RequestDeviceAuthorization(
+ mojom::AudioOutputRequest audio_output_request,
+ int64_t session_id,
+ const std::string& device_id,
+ const url::Origin& origin,
+ const RequestDeviceAuthorizationCallback& callback) {
+ DCHECK(thread_checker_.CalledOnValidThread());
+ const base::TimeTicks auth_start_time = base::TimeTicks::Now();
+
+ if (!base::IsValueInRangeForNumericType<int>(session_id)) {
+ UMALogDeviceAuthorizationTime(auth_start_time);
+ binding_.Close();
+ bad_message::ReceivedBadMessage(context_->GetRenderProcessId(),
+ bad_message::RFAOS_OUT_OF_RANGE_INTEGER);
+ context_->OnServiceFinished(this);
+ return;
+ }
+
+ context_->RequestDeviceAuthorization(
+ render_frame_id_, static_cast<int>(session_id), device_id, origin,
+ base::Bind(&RenderFrameAudioOutputService::AuthorizationCompleted,
+ base::Unretained(this), origin, auth_start_time,
+ base::Passed(std::move(audio_output_request)), callback));
+}
+
+void RenderFrameAudioOutputService::AuthorizationCompleted(
+ const url::Origin& origin,
+ base::TimeTicks auth_start_time,
+ mojom::AudioOutputRequest request,
+ const RequestDeviceAuthorizationCallback& callback,
+ media::OutputDeviceStatus status,
+ bool should_send_id,
+ const media::AudioParameters& params,
+ const std::string& raw_device_id) {
+ DCHECK(thread_checker_.CalledOnValidThread());
+ UMALogDeviceAuthorizationTime(auth_start_time);
+
+ if (status != media::OUTPUT_DEVICE_STATUS_OK) {
+ callback.Run(media::OutputDeviceStatus(status),
+ media::AudioParameters::UnavailableDeviceParams(),
+ std::string());
+ return;
+ }
+
+ if (request.is_pending()) {
+ audio_outputs_.push_back(
+ base::WrapUnique<mojom::AudioOutput>(new AudioOutputImpl(
+ std::move(request),
+ base::BindOnce(&AudioOutputServiceContext::CreateDelegate,
+ base::Unretained(context_), raw_device_id,
+ render_frame_id_),
+ base::BindOnce(&RenderFrameAudioOutputService::RemoveOutput,
+ base::Unretained(this)))));
+ }
+
+ callback.Run(media::OutputDeviceStatus(status), params,
+ should_send_id ? MediaStreamManager::GetHMACForMediaDeviceID(
+ context_->GetSalt(), origin, raw_device_id)
+ : std::string());
+}
+
+void RenderFrameAudioOutputService::RemoveOutput(
+ mojom::AudioOutput* audio_output) {
+ auto it = std::find_if(
+ audio_outputs_.begin(), audio_outputs_.end(),
+ [audio_output](const std::unique_ptr<mojom::AudioOutput>& other) {
+ return other.get() == audio_output;
+ });
+
+ // It is possible that the output is already gone, in case destruction
+ // triggered a connection error.
+ if (it != audio_outputs_.end()) {
+ std::swap(*it, audio_outputs_.back());
+ audio_outputs_.pop_back();
+ }
+}
+
+} // namespace content

Powered by Google App Engine
This is Rietveld 408576698