Quick fix for broken compositor driven UI animations on Chrome OS

This CL disables AnimationDelegateViews from using the compositor driven
CompositorAnimationRunner in favour of the legacy base::RepeatingTimer
driven runner.

We are disabling this due to the ui::Compositor sometimes going away
during the browser window's lifetime on Chrome OS resulting in animations
not running anymore.

Bug: 969788
Change-Id: If2a029764dd471eb91aadd4030f576c3604a6636
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1715358
Commit-Queue: Allen Bauer <kylixrd@chromium.org>
Reviewed-by: Allen Bauer <kylixrd@chromium.org>
Auto-Submit: Alan Cutter <alancutter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#681357}

ui/views/animation/animation_delegate_views.cc

@@ -56,6 +56,12 @@ void AnimationDelegateViews::AnimationContainerShuttingDown(
 }
 
 void AnimationDelegateViews::UpdateAnimationRunner() {
+#if defined(OS_CHROMEOS)
+  // TODO(crbug.com/969788): Re-enable this function with better ui::Compositor
+  // switching support.
+  return;
+#endif  // defined(OS_CHROMEOS)
+
   if (!container_)
     return;
 

ui/views/animation/compositor_animation_runner_unittest.cc

@@ -15,7 +15,14 @@ namespace test {
 
 using CompositorAnimationRunnerTest = WidgetTest;
 
-TEST_F(CompositorAnimationRunnerTest, BasicCoverageTest) {
+// TODO(crbug.com/969788): Re-enable CompositorAnimationRunner with better
+// ui::Compositor switching support.
+#if defined(OS_CHROMEOS)
+#define MAYBE_BasicCoverageTest DISABLED_BasicCoverageTest
+#else
+#define MAYBE_BasicCoverageTest BasicCoverageTest
+#endif
+TEST_F(CompositorAnimationRunnerTest, MAYBE_BasicCoverageTest) {
   WidgetAutoclosePtr widget(CreateTopLevelPlatformWidget());
   widget->Show();