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

Unified Diff: third_party/WebKit/Source/modules/webaudio/AbstractAudioContext.cpp

Issue 2046933003: WebAudio: record whether user gesture was available and provided. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2743
Patch Set: Created 4 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: third_party/WebKit/Source/modules/webaudio/AbstractAudioContext.cpp
diff --git a/third_party/WebKit/Source/modules/webaudio/AbstractAudioContext.cpp b/third_party/WebKit/Source/modules/webaudio/AbstractAudioContext.cpp
index fcfa28d9c3f55cbcff6d24dc9f2ce5767d40f202..e775c8573e862e6d0758018ceaa45e87a4fb1c43 100644
--- a/third_party/WebKit/Source/modules/webaudio/AbstractAudioContext.cpp
+++ b/third_party/WebKit/Source/modules/webaudio/AbstractAudioContext.cpp
@@ -32,6 +32,7 @@
#include "core/dom/Document.h"
#include "core/dom/ExceptionCode.h"
#include "core/dom/ExecutionContextTask.h"
+#include "core/frame/Settings.h"
#include "core/html/HTMLMediaElement.h"
#include "modules/mediastream/MediaStream.h"
#include "modules/webaudio/AnalyserNode.h"
@@ -64,13 +65,27 @@
#include "modules/webaudio/ScriptProcessorNode.h"
#include "modules/webaudio/StereoPannerNode.h"
#include "modules/webaudio/WaveShaperNode.h"
+#include "platform/Histogram.h"
#include "platform/ThreadSafeFunctional.h"
+#include "platform/UserGestureIndicator.h"
#include "platform/audio/IIRFilter.h"
#include "public/platform/Platform.h"
#include "wtf/text/WTFString.h"
namespace blink {
+namespace {
+
+enum UserGestureRecord {
+ UserGestureRequiredAndAvailable = 0,
+ UserGestureRequiredAndNotAvailable,
+ UserGestureNotRequiredAndAvailable,
+ UserGestureNotRequiredAndNotAvailable,
+ UserGestureRecordMax
+};
+
+} // anonymous namespace
+
AbstractAudioContext* AbstractAudioContext::create(Document& document, ExceptionState& exceptionState)
{
return AudioContext::create(document, exceptionState);
@@ -86,6 +101,7 @@ AbstractAudioContext::AbstractAudioContext(Document* document)
, m_destinationNode(nullptr)
, m_isCleared(false)
, m_isResolvingResumePromises(false)
+ , m_userGestureRequired(false)
, m_connectionCount(0)
, m_deferredTaskHandler(DeferredTaskHandler::create())
, m_contextState(Suspended)
@@ -95,6 +111,12 @@ AbstractAudioContext::AbstractAudioContext(Document* document)
, m_periodicWaveSawtooth(nullptr)
, m_periodicWaveTriangle(nullptr)
{
+ // TODO(mlamouri): we might want to use other ways of checking for this but
+ // in order to record metrics, re-using the HTMLMediaElement setting is
+ // probably the simplest solution.
+ if (document->settings() && document->settings()->mediaPlaybackRequiresUserGesture())
+ m_userGestureRequired = true;
+
m_destinationNode = DefaultAudioDestinationNode::create(this);
initialize();
@@ -107,6 +129,7 @@ AbstractAudioContext::AbstractAudioContext(Document* document, unsigned numberOf
, m_destinationNode(nullptr)
, m_isCleared(false)
, m_isResolvingResumePromises(false)
+ , m_userGestureRequired(false)
, m_connectionCount(0)
, m_deferredTaskHandler(DeferredTaskHandler::create())
, m_contextState(Suspended)
@@ -116,6 +139,11 @@ AbstractAudioContext::AbstractAudioContext(Document* document, unsigned numberOf
, m_periodicWaveSawtooth(nullptr)
, m_periodicWaveTriangle(nullptr)
{
+ // TODO(mlamouri): we might want to use other ways of checking for this but
+ // in order to record metrics, re-using the HTMLMediaElement setting is
+ // probably the simplest solution.
+ if (document->settings() && document->settings()->mediaPlaybackRequiresUserGesture())
+ m_userGestureRequired = true;
}
AbstractAudioContext::~AbstractAudioContext()
@@ -713,6 +741,25 @@ PeriodicWave* AbstractAudioContext::periodicWave(int type)
}
}
+void AbstractAudioContext::recordUserGestureState()
+{
+ DEFINE_STATIC_LOCAL(EnumerationHistogram, userGestureHistogram, ("WebAudio.UserGesture", UserGestureRecordMax));
+
+ if (!m_userGestureRequired) {
+ if (UserGestureIndicator::processingUserGesture())
+ userGestureHistogram.count(UserGestureNotRequiredAndAvailable);
+ else
+ userGestureHistogram.count(UserGestureNotRequiredAndNotAvailable);
+ return;
+ }
+ if (!UserGestureIndicator::processingUserGesture()) {
+ userGestureHistogram.count(UserGestureRequiredAndNotAvailable);
+ return;
+ }
+ userGestureHistogram.count(UserGestureRequiredAndAvailable);
+ m_userGestureRequired = false;
+}
+
String AbstractAudioContext::state() const
{
// These strings had better match the strings for AudioContextState in AudioContext.idl.
@@ -957,6 +1004,8 @@ void AbstractAudioContext::startRendering()
ASSERT(isMainThread());
ASSERT(m_destinationNode);
+ recordUserGestureState();
+
if (m_contextState == Suspended) {
destination()->audioDestinationHandler().startRendering();
setContextState(Running);

Powered by Google App Engine
This is Rietveld 408576698