Tidy AutoplayExperimentHelper.

third_party/WebKit/Source/core/html/AutoplayExperimentHelper.cpp

 // 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 "core/html/AutoplayExperimentHelper.h"
 
 #include "core/dom/Document.h"
 #include "core/frame/Settings.h"
 #include "core/html/HTMLMediaElement.h"
 #include "core/layout/LayoutBox.h"
@@ skipping 28 matching lines @@
 {
     m_mode = fromString(this->client().autoplayExperimentMode());
 
     DVLOG_IF(3, isExperimentEnabled()) << "autoplay experiment set to " << m_mode;
 }
 
 AutoplayExperimentHelper::~AutoplayExperimentHelper()
 {
 }
 
-void AutoplayExperimentHelper::dispose()
-{
-    // Do any cleanup that requires the client.
-    unregisterForPositionUpdatesIfNeeded();
-}
-
 void AutoplayExperimentHelper::becameReadyToPlay()
 {
     // Assuming that we're eligible to override the user gesture requirement,
     // either play if we meet the visibility checks, or install a listener
     // to wait for them to pass.  We do not actually start playback; our
     // caller must do that.
     autoplayMediaEncountered();
 
     if (isEligible()) {
         if (meetsVisibilityRequirements())
@@ skipping 342 matching lines @@
 
     // Don't bother to call autoplayMediaEncountered, since whoever initiates
     // playback has do it anyway, in case we don't allow autoplay.
 
     unregisterForPositionUpdatesIfNeeded();
     muteIfNeeded();
 
     // Do not actually start playback here.
 }
 
-AutoplayExperimentHelper::Client& AutoplayExperimentHelper::client() const
-{
-    return *m_client;
-}
-
 AutoplayExperimentHelper::Mode AutoplayExperimentHelper::fromString(const String& mode)
 {
     Mode value = ExperimentOff;
     if (mode.contains("-forvideo"))
         value |= ForVideo;
     if (mode.contains("-foraudio"))
         value |= ForAudio;
     if (mode.contains("-ifpagevisible"))
         value |= IfPageVisible;
     if (mode.contains("-ifviewport"))
@@ skipping 114 matching lines @@
 {
     return client().isHTMLVideoElement() && (enabled(IfViewport) || enabled(IfPartialViewport));
 }
 
 bool AutoplayExperimentHelper::isExperimentEnabled()
 {
     return m_mode != Mode::ExperimentOff;
 }
 
 } // namespace blink