DescriptionRevert of Move geolocation timer to frame-specific TaskRunnerTimer. (patchset #2 id:20001 of https://codereview.chromium.org/2642863002/ )
Reason for revert:
Seems to be the cause for failed test fast/dom/Geolocation/window-close-crash.html
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linux%20Trusty%20Leak/builds/1635
Original issue's description:
> Move geolocation timer to frame-specific TaskRunnerTimer.
>
> Move geolocation timer |m_timer| to frame-specific TaskRunnerTimer. This
> associates it with the geolocation frame's timer task queue. Note that the
> geolocation specification under specifies exactly what kind of timeout
> should be used, and we will use MiscPlatformAPI. See:
>
> https://www.w3.org/TR/geolocation-API/
>
> This m_timer is used to support, in part, the geolocation
> 'position options'.
>
> BUG=624694
>
> R=haraken,scheib
>
> Review-Url: https://codereview.chromium.org/2642863002
> Cr-Commit-Position: refs/heads/master@{#444794}
> Committed: https://chromium.googlesource.com/chromium/src/+/d620cdbebcf22fbc7a76c269cf289ab87f5c646d
TBR=haraken@chromium.org,scheib@chromium.org,dougt@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=624694
Review-Url: https://codereview.chromium.org/2640403002
Cr-Commit-Position: refs/heads/master@{#444839}
Committed: https://chromium.googlesource.com/chromium/src/+/dd4298c2034b2f5cf6e58239eb2057e7f91046b4
Patch Set 1 #
Messages
Total messages: 7 (3 generated)
|