Replace kInternalAnimation with kInternalDefault

kInternalAnimation was introduced by replacing kUnspecedTimer, which is
now kInternalDefault [1]. As we defined the task type guideline [2],
the default behavior task runners should use kInternalDefault.

[1] https://chromium-review.googlesource.com/c/chromium/src/+/1009448
[2] https://bit.ly/2vMAsQ4,

BUG: 836410
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Ib5788929dce6331fc8deee8d93df9b351f6d83ca
Reviewed-on: https://chromium-review.googlesource.com/1055353Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#558203}

third_party/blink/public/platform/task_type.h

@@ -162,9 +162,6 @@ enum class TaskType : unsigned {
   // Tasks related to the inspector.
   kInternalInspector = 33,
 
-  // Tasks related to animation like blinking caret or CSS animation.
-  kInternalAnimation = 34,
-
   // Tasks related to workers. Tasks with this type are mainly posted by:
   // * //third_party/blink/renderer/core/workers
   kInternalWorker = 36,

third_party/blink/renderer/core/animation/document_timeline.h

@@ -154,7 +154,7 @@ class CORE_EXPORT DocumentTimeline : public AnimationTimeline {
     DocumentTimelineTiming(DocumentTimeline* timeline)
         : timeline_(timeline),
           timer_(timeline->GetDocument()->GetTaskRunner(
-                     TaskType::kInternalAnimation),
+                     TaskType::kInternalDefault),
                  this,
                  &DocumentTimelineTiming::TimerFired) {
       DCHECK(timeline_);

third_party/blink/renderer/core/animation/pending_animations.h

@@ -60,7 +60,7 @@ class CORE_EXPORT PendingAnimations final
     : public GarbageCollectedFinalized<PendingAnimations> {
  public:
   explicit PendingAnimations(Document& document)
-      : timer_(document.GetTaskRunner(TaskType::kInternalAnimation),
+      : timer_(document.GetTaskRunner(TaskType::kInternalDefault),
                this,
                &PendingAnimations::TimerFired),
         compositor_group_(1) {}

third_party/blink/renderer/core/editing/frame_caret.cc

@@ -51,7 +51,7 @@ FrameCaret::FrameCaret(LocalFrame& frame,
       display_item_client_(new CaretDisplayItemClient()),
       caret_visibility_(CaretVisibility::kHidden),
       caret_blink_timer_(new TaskRunnerTimer<FrameCaret>(
-          frame.GetTaskRunner(TaskType::kInternalAnimation),
+          frame.GetTaskRunner(TaskType::kInternalDefault),
           this,
           &FrameCaret::CaretBlinkTimerFired)),
       should_paint_caret_(true),

third_party/blink/renderer/core/html/forms/spin_button_element.cc

@@ -50,7 +50,7 @@ inline SpinButtonElement::SpinButtonElement(Document& document,
       capturing_(false),
       up_down_state_(kIndeterminate),
       press_starting_state_(kIndeterminate),
-      repeating_timer_(document.GetTaskRunner(TaskType::kInternalAnimation),
+      repeating_timer_(document.GetTaskRunner(TaskType::kInternalDefault),
                        this,
                        &SpinButtonElement::RepeatingTimerFired) {}
 

third_party/blink/renderer/core/layout/layout_progress.cc

@@ -36,7 +36,7 @@ LayoutProgress::LayoutProgress(HTMLProgressElement* element)
       animation_duration_(0),
       animating_(false),
       animation_timer_(
-          element->GetDocument().GetTaskRunner(TaskType::kInternalAnimation),
+          element->GetDocument().GetTaskRunner(TaskType::kInternalDefault),
           this,
           &LayoutProgress::AnimationTimerFired) {}
 

third_party/blink/renderer/core/svg/animation/smil_time_container.cc

@@ -49,11 +49,11 @@ SMILTimeContainer::SMILTimeContainer(SVGSVGElement& owner)
       paused_(false),
       document_order_indexes_dirty_(false),
       wakeup_timer_(
-          owner.GetDocument().GetTaskRunner(TaskType::kInternalAnimation),
+          owner.GetDocument().GetTaskRunner(TaskType::kInternalDefault),
           this,
           &SMILTimeContainer::WakeupTimerFired),
       animation_policy_once_timer_(
-          owner.GetDocument().GetTaskRunner(TaskType::kInternalAnimation),
+          owner.GetDocument().GetTaskRunner(TaskType::kInternalDefault),
           this,
           &SMILTimeContainer::AnimationPolicyTimerFired),
       owner_svg_element_(&owner) {}

third_party/blink/renderer/platform/scheduler/child/worker_scheduler.cc

@@ -70,7 +70,6 @@ scoped_refptr<base::SingleThreadTaskRunner> WorkerScheduler::GetTaskRunner(
     case TaskType::kInternalIPC:
     case TaskType::kInternalUserInteraction:
     case TaskType::kInternalInspector:
-    case TaskType::kInternalAnimation:
     case TaskType::kInternalWorker:
       // UnthrottledTaskRunner is generally discouraged in future.
       // TODO(nhiroki): Identify which tasks can be throttled / suspendable and

third_party/blink/renderer/platform/scheduler/main_thread/frame_scheduler_impl.cc

@@ -299,7 +299,6 @@ scoped_refptr<base::SingleThreadTaskRunner> FrameSchedulerImpl::GetTaskRunner(
     case TaskType::kInternalMedia:
     case TaskType::kInternalMediaRealTime:
     case TaskType::kInternalUserInteraction:
-    case TaskType::kInternalAnimation:
       return TaskRunnerImpl::Create(PausableTaskQueue(), type);
     case TaskType::kUnthrottled:
     case TaskType::kInternalTest:

third_party/blink/renderer/platform/scheduler/main_thread/main_thread_scheduler_impl.cc

@@ -178,8 +178,6 @@ const char* TaskTypeToString(TaskType task_type) {
       return "InternalUserInteraction";
     case TaskType::kInternalInspector:
       return "InternalInspector";
-    case TaskType::kInternalAnimation:
-      return "InternalAnimation";
     case TaskType::kInternalWorker:
       return "InternalWorker";
     case TaskType::kCount: