Move geolocation timer to frame-specific TaskRunnerTimer.

third_party/WebKit/Source/modules/geolocation/GeoNotifier.cpp

 // Copyright 2014 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 "modules/geolocation/GeoNotifier.h"
 
+#include "core/dom/TaskRunnerHelper.h"
 #include "modules/geolocation/Geolocation.h"
 #include "modules/geolocation/PositionError.h"
 #include "modules/geolocation/PositionOptions.h"
 #include "platform/Histogram.h"
 #include "wtf/Assertions.h"
 
 namespace blink {
 
 GeoNotifier::GeoNotifier(Geolocation* geolocation,
                          PositionCallback* successCallback,
                          PositionErrorCallback* errorCallback,
                          const PositionOptions& options)
     : m_geolocation(geolocation),
       m_successCallback(successCallback),
       m_errorCallback(errorCallback),
       m_options(options),
-      m_timer(this, &GeoNotifier::timerFired),
+      m_timer(TaskRunnerHelper::get(TaskType::Timer, geolocation->frame()),

[haraken, 2017/01/18 23:30:56]

Do you have any spec that says this should use the timer task source?

Basically the timer task source is for JS timers (setTimeout). If this is an
unspeced task, we should use UnspecedTimer.

+              this,
+              &GeoNotifier::timerFired),
       m_useCachedPosition(false) {
   DCHECK(m_geolocation);
   DCHECK(m_successCallback);
 
   DEFINE_STATIC_LOCAL(CustomCountHistogram, timeoutHistogram,
                       ("Geolocation.Timeout", 0,
                        1000 * 60 * 10 /* 10 minute max */, 20 /* buckets */));
   timeoutHistogram.count(m_options.timeout());
 }
 
@@ skipping 65 matching lines @@
 
   DEFINE_STATIC_LOCAL(CustomCountHistogram, timeoutExpiredHistogram,
                       ("Geolocation.TimeoutExpired", 0,
                        1000 * 60 * 10 /* 10 minute max */, 20 /* buckets */));
   timeoutExpiredHistogram.count(m_options.timeout());
 
   m_geolocation->requestTimedOut(this);
 }
 
 }  // namespace blink