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

Unified Diff: third_party/WebKit/Source/core/html/HTMLMediaElement.h

Issue 2510353004: Deprecating AutoplayExperimentHelper (Closed)
Patch Set: rebased Created 4 years 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/core/html/HTMLMediaElement.h
diff --git a/third_party/WebKit/Source/core/html/HTMLMediaElement.h b/third_party/WebKit/Source/core/html/HTMLMediaElement.h
index 2c9b6bc98bed2ee440b5bf5df6cb41d6ffe494d2..0d865c19052274c8e29ce880d874c11752443f3a 100644
--- a/third_party/WebKit/Source/core/html/HTMLMediaElement.h
+++ b/third_party/WebKit/Source/core/html/HTMLMediaElement.h
@@ -34,7 +34,6 @@
#include "core/dom/ActiveDOMObject.h"
#include "core/dom/ExceptionCode.h"
#include "core/events/GenericEventQueue.h"
-#include "core/html/AutoplayExperimentHelper.h"
#include "core/html/HTMLElement.h"
#include "core/html/track/TextTrack.h"
#include "platform/MIMETypeRegistry.h"
@@ -173,8 +172,7 @@ class CORE_EXPORT HTMLMediaElement : public HTMLElement,
TimeRanges* seekable() const;
bool ended() const;
bool autoplay() const;
- bool shouldAutoplay(
- const RecordMetricsBehavior = RecordMetricsBehavior::DoNotRecord);
+ bool shouldAutoplay();
bool loop() const;
void setLoop(bool);
ScriptPromise playForBindings(ScriptState*);
@@ -304,10 +302,6 @@ class CORE_EXPORT HTMLMediaElement : public HTMLElement,
void videoWillBeDrawnToCanvas() const;
- // Temporary callback for crbug.com/487345,402044
- void notifyPositionMayHaveChanged(const IntRect&);
- void updatePositionNotificationRegistration();
-
WebRemotePlaybackClient* remotePlaybackClient() {
return m_remotePlaybackClient;
}
@@ -334,8 +328,6 @@ class CORE_EXPORT HTMLMediaElement : public HTMLElement,
DisplayMode getDisplayMode() const { return m_displayMode; }
virtual void setDisplayMode(DisplayMode mode) { m_displayMode = mode; }
- void recordAutoplayMetric(AutoplayMetrics);
-
private:
void resetMediaPlayerAndMediaSource();
@@ -532,9 +524,6 @@ class CORE_EXPORT HTMLMediaElement : public HTMLElement,
void audioTracksTimerFired(TimerBase*);
- // TODO(liberato): remove once autoplay gesture override experiment concludes.
- void triggerAutoplayViewportCheckForTesting();
-
void scheduleResolvePlayPromises();
void scheduleRejectPlayPromises(ExceptionCode);
void scheduleNotifyPlaying();
@@ -717,18 +706,13 @@ class CORE_EXPORT HTMLMediaElement : public HTMLElement,
AudioSourceProviderImpl m_audioSourceProvider;
- class AutoplayHelperClientImpl;
-
friend class AutoplayUmaHelper; // for isAutoplayAllowedPerSettings
friend class AutoplayUmaHelperTest;
friend class Internals;
friend class TrackDisplayUpdateScope;
- friend class AutoplayExperimentHelper;
friend class MediaControlsTest;
friend class HTMLVideoElementTest;
- Member<AutoplayExperimentHelper::Client> m_autoplayHelperClient;
- Member<AutoplayExperimentHelper> m_autoplayHelper;
Member<AutoplayUmaHelper> m_autoplayUmaHelper;
WebRemotePlaybackClient* m_remotePlaybackClient;
« no previous file with comments | « third_party/WebKit/Source/core/html/BUILD.gn ('k') | third_party/WebKit/Source/core/html/HTMLMediaElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698