Move VTTRegion Timers to TaskRunnerTimer

third_party/WebKit/Source/core/html/track/vtt/VTTRegion.h

 /*
  * Copyright (C) 2013 Google Inc.  All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 84 matching lines @@
   void displayLastVTTCueBox();
   void willRemoveVTTCueBox(VTTCueBox*);
 
   DECLARE_TRACE();
 
  private:
   VTTRegion();
 
   void prepareRegionDisplayTree();
 
+  void prepareScrollTimer(Document&);
+
   // The timer is needed to continue processing when cue scrolling ended.
   void startTimer();
   void stopTimer();
   void scrollTimerFired(TimerBase*);
 
   enum RegionSetting {
     None,
     Id,
     Width,
     Height,
@@ skipping 28 matching lines @@
   Member<TextTrack> m_track;
 
   // Keep track of the current numeric value of the css "top" property.
   double m_currentTop;
 
   // The timer is used to display the next cue line after the current one has
   // been displayed. It's main use is for scrolling regions and it triggers as
   // soon as the animation for rolling out one line has finished, but
   // currently it is used also for non-scrolling regions to use a single
   // code path.
-  Timer<VTTRegion> m_scrollTimer;
+  std::unique_ptr<TaskRunnerTimer<VTTRegion>> m_scrollTimer;

[haraken, 2017/02/08 11:49:40]

Can we just use TaskRunnerTimer<VTTRegion> ?

[maksims (do not use this acc), 2017/02/08 13:02:05]

Otherwise I cannot provide TaskRunnerTimer with appropriate task runner, which
is taken from document's frame later on, early in ctor

 };
 
 }  // namespace blink
 
 #endif  // VTTRegion_h