Revert "Add a microtask version of the video.rAF for testing."

This reverts commit 6db10514. This did
not work well so it's being reverted.

R=tguilbert

Bug: 1012063
Change-Id: Ia5faf1026921dab735ead8acaa8f4fdd80dc6fc1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2135894
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Thomas Guilbert <tguilbert@chromium.org>
Auto-Submit: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Thomas Guilbert <tguilbert@chromium.org>
Cr-Commit-Position: refs/heads/master@{#756441}

media/base/media_switches.cc

@@ -390,12 +390,6 @@ const base::Feature kUseNewMediaCache{"use-new-media-cache",
 const base::Feature kUseMediaHistoryStore{"UseMediaHistoryStore",
                                           base::FEATURE_DISABLED_BY_DEFAULT};
 
-// Causes video.requestAniationFrame to use a microtask instead of running with
-// the rendering steps. TODO(crbug.com/1012063): Remove this once we figure out
-// which implementation to use.
-const base::Feature kUseMicrotaskForVideoRAF{"UseMicrotaskForVideoRAF",
-                                             base::FEATURE_DISABLED_BY_DEFAULT};
-
 // Use R16 texture for 9-16 bit channel instead of half-float conversion by CPU.
 const base::Feature kUseR16Texture{"use-r16-texture",
                                    base::FEATURE_DISABLED_BY_DEFAULT};

media/base/media_switches.h

@@ -164,7 +164,6 @@ MEDIA_EXPORT extern const base::Feature kUnifiedAutoplay;
 MEDIA_EXPORT extern const base::Feature kUseAndroidOverlayAggressively;
 MEDIA_EXPORT extern const base::Feature kUseFakeDeviceForMediaStream;
 MEDIA_EXPORT extern const base::Feature kUseMediaHistoryStore;
-MEDIA_EXPORT extern const base::Feature kUseMicrotaskForVideoRAF;
 MEDIA_EXPORT extern const base::Feature kUseNewMediaCache;
 MEDIA_EXPORT extern const base::Feature kUseR16Texture;
 MEDIA_EXPORT extern const base::Feature kVaapiH264AMDEncoder;

third_party/blink/renderer/modules/video_raf/DEPS

 include_rules = [
-    "+media/base/media_switches.h",
     "+media/base/video_frame.h",
 ]
\ No newline at end of file

third_party/blink/renderer/modules/video_raf/video_request_animation_frame_impl.cc

@@ -8,7 +8,6 @@
 #include <utility>
 
 #include "base/trace_event/trace_event.h"
-#include "media/base/media_switches.h"
 #include "third_party/blink/renderer/bindings/modules/v8/v8_video_frame_metadata.h"
 #include "third_party/blink/renderer/core/dom/document.h"
 #include "third_party/blink/renderer/core/dom/scripted_animation_controller.h"
@@ -17,7 +16,6 @@
 #include "third_party/blink/renderer/core/timing/performance.h"
 #include "third_party/blink/renderer/core/timing/time_clamper.h"
 #include "third_party/blink/renderer/modules/video_raf/video_frame_request_callback_collection.h"
-#include "third_party/blink/renderer/platform/bindings/microtask.h"
 #include "third_party/blink/renderer/platform/wtf/functional.h"
 
 namespace blink {
@@ -92,24 +90,12 @@ void VideoRequestAnimationFrameImpl::ScheduleCallbackExecution() {
     return;
 
   pending_execution_ = true;
-  if (base::FeatureList::IsEnabled(media::kUseMicrotaskForVideoRAF)) {
-    auto& time_converter =
-        GetSupplementable()->GetDocument().Loader()->GetTiming();
-    Microtask::EnqueueMicrotask(WTF::Bind(
-        &VideoRequestAnimationFrameImpl::OnRenderingSteps,
-        WrapWeakPersistent(this),
-        // TODO(crbug.com/1012063): Now is probably not the right value.
-        GetClampedTimeInMillis(
-            time_converter.MonotonicTimeToZeroBasedDocumentTime(
-                base::TimeTicks::Now()))));
-  } else {
   GetSupplementable()
       ->GetDocument()
       .GetScriptedAnimationController()
       .ScheduleVideoRafExecution(
           WTF::Bind(&VideoRequestAnimationFrameImpl::OnRenderingSteps,
                     WrapWeakPersistent(this)));
-  }
 }
 
 void VideoRequestAnimationFrameImpl::OnRequestAnimationFrame() {