<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[215167] trunk</title>
</head>
<body>

<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }
#msg dl a { font-weight: bold}
#msg dl a:link    { color:#fc3; }
#msg dl a:active  { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/215167">215167</a></dd>
<dt>Author</dt> <dd>cdumez@apple.com</dd>
<dt>Date</dt> <dd>2017-04-09 16:42:02 -0700 (Sun, 09 Apr 2017)</dd>
</dl>

<h3>Log Message</h3>
<pre>Drop Timer::startRepeating() overload taking a double
https://bugs.webkit.org/show_bug.cgi?id=170656

Reviewed by Yusuke Suzuki.

Drop Timer::startRepeating() overload taking a double as people should use Seconds type now.

Source/WebCore:

* Modules/mediastream/CanvasCaptureMediaStreamTrack.cpp:
(WebCore::CanvasCaptureMediaStreamTrack::Source::startProducingData):
* editing/FrameSelection.cpp:
(WebCore::FrameSelection::updateAppearance):
* html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::startProgressEventTimer):
(WebCore::HTMLMediaElement::scheduleTimeupdateEvent):
* html/HTMLMediaElement.h:
* html/MediaController.cpp:
(MediaController::MediaController):
(MediaController::scheduleTimeupdateEvent):
* html/MediaController.h:
* html/MediaElementSession.cpp:
* inspector/InspectorOverlay.cpp:
(WebCore::InspectorOverlay::showPaintRect):
* loader/ProgressTracker.cpp:
* page/AutoscrollController.cpp:
* page/Frame.cpp:
* page/PageOverlay.cpp:
(WebCore::PageOverlay::startFadeAnimation):
* page/SuspendableTimer.h:
* page/WheelEventTestTrigger.cpp:
(WebCore::WheelEventTestTrigger::setTestCallbackAndStartNotificationTimer):
* platform/Theme.h:
(WebCore::Theme::caretBlinkInterval):
* platform/Timer.h:
* platform/cocoa/ScrollController.mm:
(WebCore::ScrollController::startSnapRubberbandTimer):
(WebCore::ScrollController::startScrollSnapTimer):
* platform/graphics/ca/TileGrid.cpp:
(WebCore::TileGrid::scheduleCohortRemoval):
* platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
(WebCore::MediaPlayerPrivateGStreamer::setDownloadBuffering):
* platform/mediastream/mac/MockRealtimeAudioSourceMac.mm:
(WebCore::MockRealtimeAudioSourceMac::reconfigure):
* platform/mock/MockRealtimeAudioSource.cpp:
(WebCore::MockRealtimeAudioSource::startProducingData):
* platform/mock/MockRealtimeAudioSource.h:
(WebCore::MockRealtimeAudioSource::renderInterval):
* platform/mock/MockRealtimeVideoSource.cpp:
(WebCore::MockRealtimeVideoSource::startProducingData):
(WebCore::MockRealtimeVideoSource::applyFrameRate):
* rendering/RenderMarquee.cpp:
(WebCore::RenderMarquee::start):
(WebCore::RenderMarquee::updateMarqueeStyle):
* rendering/RenderTheme.h:
(WebCore::RenderTheme::caretBlinkInterval):
* rendering/RenderThemeGtk.cpp:
(WebCore::RenderThemeGtk::caretBlinkInterval):
* rendering/RenderThemeGtk.h:
* xml/XMLHttpRequestProgressEventThrottle.cpp:
(WebCore::XMLHttpRequestProgressEventThrottle::dispatchThrottledProgressEvent):
* xml/XMLHttpRequestProgressEventThrottle.h:

Source/WebKit/mac:

* Plugins/WebNetscapePluginView.mm:
(PluginTimer::start):

Source/WebKit/win:

* FullscreenVideoController.cpp:

Source/WebKit2:

* Shared/WebMemorySampler.cpp:
(WebKit::WebMemorySampler::initializeTimers):
* UIProcess/Cocoa/ViewGestureController.cpp:
* UIProcess/HighPerformanceGraphicsUsageSampler.cpp:
* UIProcess/PerActivityStateCPUUsageSampler.cpp:
* WebProcess/Plugins/Netscape/NetscapePlugin.cpp:
(WebKit::NetscapePlugin::Timer::start):
* WebProcess/Plugins/Netscape/mac/NetscapePluginMac.mm:
* WebProcess/WebPage/CoordinatedGraphics/CompositingCoordinator.cpp:
(WebKit::CompositingCoordinator::scheduleReleaseInactiveAtlases):

Tools:

* TestWebKitAPI/Tests/WTF/RunLoop.cpp:
(TestWebKitAPI::TEST):</pre>

<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>
<li><a href="#trunkSourceWebCoreModulesmediastreamCanvasCaptureMediaStreamTrackcpp">trunk/Source/WebCore/Modules/mediastream/CanvasCaptureMediaStreamTrack.cpp</a></li>
<li><a href="#trunkSourceWebCoreeditingFrameSelectioncpp">trunk/Source/WebCore/editing/FrameSelection.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlHTMLMediaElementcpp">trunk/Source/WebCore/html/HTMLMediaElement.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlHTMLMediaElementh">trunk/Source/WebCore/html/HTMLMediaElement.h</a></li>
<li><a href="#trunkSourceWebCorehtmlMediaControllercpp">trunk/Source/WebCore/html/MediaController.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlMediaControllerh">trunk/Source/WebCore/html/MediaController.h</a></li>
<li><a href="#trunkSourceWebCorehtmlMediaElementSessioncpp">trunk/Source/WebCore/html/MediaElementSession.cpp</a></li>
<li><a href="#trunkSourceWebCoreinspectorInspectorOverlaycpp">trunk/Source/WebCore/inspector/InspectorOverlay.cpp</a></li>
<li><a href="#trunkSourceWebCoreloaderProgressTrackercpp">trunk/Source/WebCore/loader/ProgressTracker.cpp</a></li>
<li><a href="#trunkSourceWebCorepageAutoscrollControllercpp">trunk/Source/WebCore/page/AutoscrollController.cpp</a></li>
<li><a href="#trunkSourceWebCorepageFramecpp">trunk/Source/WebCore/page/Frame.cpp</a></li>
<li><a href="#trunkSourceWebCorepagePageOverlaycpp">trunk/Source/WebCore/page/PageOverlay.cpp</a></li>
<li><a href="#trunkSourceWebCorepageSuspendableTimerh">trunk/Source/WebCore/page/SuspendableTimer.h</a></li>
<li><a href="#trunkSourceWebCorepageWheelEventTestTriggercpp">trunk/Source/WebCore/page/WheelEventTestTrigger.cpp</a></li>
<li><a href="#trunkSourceWebCoreplatformThemeh">trunk/Source/WebCore/platform/Theme.h</a></li>
<li><a href="#trunkSourceWebCoreplatformTimerh">trunk/Source/WebCore/platform/Timer.h</a></li>
<li><a href="#trunkSourceWebCoreplatformcocoaScrollControllermm">trunk/Source/WebCore/platform/cocoa/ScrollController.mm</a></li>
<li><a href="#trunkSourceWebCoreplatformgraphicscaTileGridcpp">trunk/Source/WebCore/platform/graphics/ca/TileGrid.cpp</a></li>
<li><a href="#trunkSourceWebCoreplatformgraphicsgstreamerMediaPlayerPrivateGStreamercpp">trunk/Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp</a></li>
<li><a href="#trunkSourceWebCoreplatformmediastreammacMockRealtimeAudioSourceMacmm">trunk/Source/WebCore/platform/mediastream/mac/MockRealtimeAudioSourceMac.mm</a></li>
<li><a href="#trunkSourceWebCoreplatformmockMockRealtimeAudioSourcecpp">trunk/Source/WebCore/platform/mock/MockRealtimeAudioSource.cpp</a></li>
<li><a href="#trunkSourceWebCoreplatformmockMockRealtimeAudioSourceh">trunk/Source/WebCore/platform/mock/MockRealtimeAudioSource.h</a></li>
<li><a href="#trunkSourceWebCoreplatformmockMockRealtimeVideoSourcecpp">trunk/Source/WebCore/platform/mock/MockRealtimeVideoSource.cpp</a></li>
<li><a href="#trunkSourceWebCorerenderingRenderMarqueecpp">trunk/Source/WebCore/rendering/RenderMarquee.cpp</a></li>
<li><a href="#trunkSourceWebCorerenderingRenderThemeh">trunk/Source/WebCore/rendering/RenderTheme.h</a></li>
<li><a href="#trunkSourceWebCorerenderingRenderThemeGtkcpp">trunk/Source/WebCore/rendering/RenderThemeGtk.cpp</a></li>
<li><a href="#trunkSourceWebCorerenderingRenderThemeGtkh">trunk/Source/WebCore/rendering/RenderThemeGtk.h</a></li>
<li><a href="#trunkSourceWebCorexmlXMLHttpRequestProgressEventThrottlecpp">trunk/Source/WebCore/xml/XMLHttpRequestProgressEventThrottle.cpp</a></li>
<li><a href="#trunkSourceWebCorexmlXMLHttpRequestProgressEventThrottleh">trunk/Source/WebCore/xml/XMLHttpRequestProgressEventThrottle.h</a></li>
<li><a href="#trunkSourceWebKitmacChangeLog">trunk/Source/WebKit/mac/ChangeLog</a></li>
<li><a href="#trunkSourceWebKitmacPluginsWebNetscapePluginViewmm">trunk/Source/WebKit/mac/Plugins/WebNetscapePluginView.mm</a></li>
<li><a href="#trunkSourceWebKitwinChangeLog">trunk/Source/WebKit/win/ChangeLog</a></li>
<li><a href="#trunkSourceWebKitwinFullscreenVideoControllercpp">trunk/Source/WebKit/win/FullscreenVideoController.cpp</a></li>
<li><a href="#trunkSourceWebKit2ChangeLog">trunk/Source/WebKit2/ChangeLog</a></li>
<li><a href="#trunkSourceWebKit2SharedWebMemorySamplercpp">trunk/Source/WebKit2/Shared/WebMemorySampler.cpp</a></li>
<li><a href="#trunkSourceWebKit2UIProcessCocoaViewGestureControllercpp">trunk/Source/WebKit2/UIProcess/Cocoa/ViewGestureController.cpp</a></li>
<li><a href="#trunkSourceWebKit2UIProcessHighPerformanceGraphicsUsageSamplercpp">trunk/Source/WebKit2/UIProcess/HighPerformanceGraphicsUsageSampler.cpp</a></li>
<li><a href="#trunkSourceWebKit2UIProcessPerActivityStateCPUUsageSamplercpp">trunk/Source/WebKit2/UIProcess/PerActivityStateCPUUsageSampler.cpp</a></li>
<li><a href="#trunkSourceWebKit2WebProcessPluginsNetscapeNetscapePlugincpp">trunk/Source/WebKit2/WebProcess/Plugins/Netscape/NetscapePlugin.cpp</a></li>
<li><a href="#trunkSourceWebKit2WebProcessPluginsNetscapemacNetscapePluginMacmm">trunk/Source/WebKit2/WebProcess/Plugins/Netscape/mac/NetscapePluginMac.mm</a></li>
<li><a href="#trunkSourceWebKit2WebProcessWebPageCoordinatedGraphicsCompositingCoordinatorcpp">trunk/Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/CompositingCoordinator.cpp</a></li>
<li><a href="#trunkToolsChangeLog">trunk/Tools/ChangeLog</a></li>
<li><a href="#trunkToolsTestWebKitAPITestsWTFRunLoopcpp">trunk/Tools/TestWebKitAPI/Tests/WTF/RunLoop.cpp</a></li>
</ul>

</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/ChangeLog        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -1,3 +1,66 @@
</span><ins>+2017-04-09  Chris Dumez  &lt;cdumez@apple.com&gt;
+
+        Drop Timer::startRepeating() overload taking a double
+        https://bugs.webkit.org/show_bug.cgi?id=170656
+
+        Reviewed by Yusuke Suzuki.
+
+        Drop Timer::startRepeating() overload taking a double as people should use Seconds type now.
+
+        * Modules/mediastream/CanvasCaptureMediaStreamTrack.cpp:
+        (WebCore::CanvasCaptureMediaStreamTrack::Source::startProducingData):
+        * editing/FrameSelection.cpp:
+        (WebCore::FrameSelection::updateAppearance):
+        * html/HTMLMediaElement.cpp:
+        (WebCore::HTMLMediaElement::startProgressEventTimer):
+        (WebCore::HTMLMediaElement::scheduleTimeupdateEvent):
+        * html/HTMLMediaElement.h:
+        * html/MediaController.cpp:
+        (MediaController::MediaController):
+        (MediaController::scheduleTimeupdateEvent):
+        * html/MediaController.h:
+        * html/MediaElementSession.cpp:
+        * inspector/InspectorOverlay.cpp:
+        (WebCore::InspectorOverlay::showPaintRect):
+        * loader/ProgressTracker.cpp:
+        * page/AutoscrollController.cpp:
+        * page/Frame.cpp:
+        * page/PageOverlay.cpp:
+        (WebCore::PageOverlay::startFadeAnimation):
+        * page/SuspendableTimer.h:
+        * page/WheelEventTestTrigger.cpp:
+        (WebCore::WheelEventTestTrigger::setTestCallbackAndStartNotificationTimer):
+        * platform/Theme.h:
+        (WebCore::Theme::caretBlinkInterval):
+        * platform/Timer.h:
+        * platform/cocoa/ScrollController.mm:
+        (WebCore::ScrollController::startSnapRubberbandTimer):
+        (WebCore::ScrollController::startScrollSnapTimer):
+        * platform/graphics/ca/TileGrid.cpp:
+        (WebCore::TileGrid::scheduleCohortRemoval):
+        * platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
+        (WebCore::MediaPlayerPrivateGStreamer::setDownloadBuffering):
+        * platform/mediastream/mac/MockRealtimeAudioSourceMac.mm:
+        (WebCore::MockRealtimeAudioSourceMac::reconfigure):
+        * platform/mock/MockRealtimeAudioSource.cpp:
+        (WebCore::MockRealtimeAudioSource::startProducingData):
+        * platform/mock/MockRealtimeAudioSource.h:
+        (WebCore::MockRealtimeAudioSource::renderInterval):
+        * platform/mock/MockRealtimeVideoSource.cpp:
+        (WebCore::MockRealtimeVideoSource::startProducingData):
+        (WebCore::MockRealtimeVideoSource::applyFrameRate):
+        * rendering/RenderMarquee.cpp:
+        (WebCore::RenderMarquee::start):
+        (WebCore::RenderMarquee::updateMarqueeStyle):
+        * rendering/RenderTheme.h:
+        (WebCore::RenderTheme::caretBlinkInterval):
+        * rendering/RenderThemeGtk.cpp:
+        (WebCore::RenderThemeGtk::caretBlinkInterval):
+        * rendering/RenderThemeGtk.h:
+        * xml/XMLHttpRequestProgressEventThrottle.cpp:
+        (WebCore::XMLHttpRequestProgressEventThrottle::dispatchThrottledProgressEvent):
+        * xml/XMLHttpRequestProgressEventThrottle.h:
+
</ins><span class="cx"> 2017-04-09  Fujii Hironori  &lt;Hironori.Fujii@sony.com&gt;
</span><span class="cx"> 
</span><span class="cx">         generate-bindings-all.pl shouldn't use Perl threads
</span></span></pre></div>
<a id="trunkSourceWebCoreModulesmediastreamCanvasCaptureMediaStreamTrackcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/Modules/mediastream/CanvasCaptureMediaStreamTrack.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/Modules/mediastream/CanvasCaptureMediaStreamTrack.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/Modules/mediastream/CanvasCaptureMediaStreamTrack.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -88,7 +88,7 @@
</span><span class="cx">         return;
</span><span class="cx"> 
</span><span class="cx">     if (m_frameRequestRate.value())
</span><del>-        m_requestFrameTimer.startRepeating(1. / m_frameRequestRate.value());
</del><ins>+        m_requestFrameTimer.startRepeating(1_s / m_frameRequestRate.value());
</ins><span class="cx"> }
</span><span class="cx"> 
</span><span class="cx"> void CanvasCaptureMediaStreamTrack::Source::stopProducingData()
</span></span></pre></div>
<a id="trunkSourceWebCoreeditingFrameSelectioncpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/editing/FrameSelection.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/editing/FrameSelection.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/editing/FrameSelection.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -2057,7 +2057,7 @@
</span><span class="cx">     // Start blinking with a black caret. Be sure not to restart if we're
</span><span class="cx">     // already blinking in the right location.
</span><span class="cx">     if (shouldBlink &amp;&amp; !m_caretBlinkTimer.isActive()) {
</span><del>-        if (double blinkInterval = m_frame-&gt;page()-&gt;theme().caretBlinkInterval())
</del><ins>+        if (Seconds blinkInterval = m_frame-&gt;page()-&gt;theme().caretBlinkInterval())
</ins><span class="cx">             m_caretBlinkTimer.startRepeating(blinkInterval);
</span><span class="cx"> 
</span><span class="cx">         if (!m_caretPaint) {
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlHTMLMediaElementcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/HTMLMediaElement.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/HTMLMediaElement.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/html/HTMLMediaElement.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -2040,7 +2040,7 @@
</span><span class="cx"> 
</span><span class="cx">     m_previousProgressTime = monotonicallyIncreasingTime();
</span><span class="cx">     // 350ms is not magic, it is in the spec!
</span><del>-    m_progressEventTimer.startRepeating(0.350);
</del><ins>+    m_progressEventTimer.startRepeating(350_ms);
</ins><span class="cx"> }
</span><span class="cx"> 
</span><span class="cx"> void HTMLMediaElement::waitForSourceChange()
</span><span class="lines">@@ -3521,7 +3521,7 @@
</span><span class="cx"> 
</span><span class="cx"> // The spec says to fire periodic timeupdate events (those sent while playing) every
</span><span class="cx"> // &quot;15 to 250ms&quot;, we choose the slowest frequency
</span><del>-static const double maxTimeupdateEventFrequency = 0.25;
</del><ins>+static const Seconds maxTimeupdateEventFrequency { 250_ms };
</ins><span class="cx"> 
</span><span class="cx"> void HTMLMediaElement::startPlaybackProgressTimer()
</span><span class="cx"> {
</span><span class="lines">@@ -3564,8 +3564,8 @@
</span><span class="cx"> 
</span><span class="cx"> void HTMLMediaElement::scheduleTimeupdateEvent(bool periodicEvent)
</span><span class="cx"> {
</span><del>-    double now = monotonicallyIncreasingTime();
-    double timedelta = now - m_clockTimeAtLastUpdateEvent;
</del><ins>+    MonotonicTime now = MonotonicTime::now();
+    Seconds timedelta = now - m_clockTimeAtLastUpdateEvent;
</ins><span class="cx"> 
</span><span class="cx">     // throttle the periodic events
</span><span class="cx">     if (periodicEvent &amp;&amp; timedelta &lt; maxTimeupdateEventFrequency)
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlHTMLMediaElementh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/HTMLMediaElement.h (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/HTMLMediaElement.h        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/html/HTMLMediaElement.h        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -887,7 +887,7 @@
</span><span class="cx">     double m_playbackStartedTime { 0 };
</span><span class="cx"> 
</span><span class="cx">     // The last time a timeupdate event was sent (based on monotonic clock).
</span><del>-    double m_clockTimeAtLastUpdateEvent { 0 };
</del><ins>+    MonotonicTime m_clockTimeAtLastUpdateEvent;
</ins><span class="cx"> 
</span><span class="cx">     // The last time a timeupdate event was sent in movie time.
</span><span class="cx">     MediaTime m_lastTimeUpdateEventMovieTime;
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlMediaControllercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/MediaController.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/MediaController.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/html/MediaController.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -58,7 +58,6 @@
</span><span class="cx">     , m_clock(Clock::create())
</span><span class="cx">     , m_scriptExecutionContext(context)
</span><span class="cx">     , m_timeupdateTimer(*this, &amp;MediaController::scheduleTimeupdateEvent)
</span><del>-    , m_previousTimeupdateTime(0)
</del><span class="cx"> {
</span><span class="cx"> }
</span><span class="cx"> 
</span><span class="lines">@@ -664,7 +663,7 @@
</span><span class="cx"> 
</span><span class="cx"> // The spec says to fire periodic timeupdate events (those sent while playing) every
</span><span class="cx"> // &quot;15 to 250ms&quot;, we choose the slowest frequency
</span><del>-static const double maxTimeupdateEventFrequency = 0.25;
</del><ins>+static const Seconds maxTimeupdateEventFrequency { 250_ms };
</ins><span class="cx"> 
</span><span class="cx"> void MediaController::startTimeupdateTimer()
</span><span class="cx"> {
</span><span class="lines">@@ -676,8 +675,8 @@
</span><span class="cx"> 
</span><span class="cx"> void MediaController::scheduleTimeupdateEvent()
</span><span class="cx"> {
</span><del>-    double now = monotonicallyIncreasingTime();
-    double timedelta = now - m_previousTimeupdateTime;
</del><ins>+    MonotonicTime now = MonotonicTime::now();
+    Seconds timedelta = now - m_previousTimeupdateTime;
</ins><span class="cx"> 
</span><span class="cx">     if (timedelta &lt; maxTimeupdateEventFrequency)
</span><span class="cx">         return;
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlMediaControllerh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/MediaController.h (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/MediaController.h        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/html/MediaController.h        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -145,7 +145,7 @@
</span><span class="cx">     std::unique_ptr&lt;Clock&gt; m_clock;
</span><span class="cx">     ScriptExecutionContext&amp; m_scriptExecutionContext;
</span><span class="cx">     Timer m_timeupdateTimer;
</span><del>-    double m_previousTimeupdateTime;
</del><ins>+    MonotonicTime m_previousTimeupdateTime;
</ins><span class="cx">     bool m_resetCurrentTimeInNextPlay { false };
</span><span class="cx"> };
</span><span class="cx"> 
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlMediaElementSessioncpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/MediaElementSession.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/MediaElementSession.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/html/MediaElementSession.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -57,7 +57,7 @@
</span><span class="cx"> 
</span><span class="cx"> namespace WebCore {
</span><span class="cx"> 
</span><del>-static const double elementMainContentCheckInterval = .250;
</del><ins>+static const Seconds elementMainContentCheckInterval { 250_ms };
</ins><span class="cx"> 
</span><span class="cx"> static bool isElementRectMostlyInMainFrame(const HTMLMediaElement&amp;);
</span><span class="cx"> static bool isElementLargeEnoughForMainContent(const HTMLMediaElement&amp;, MediaSessionMainContentPurpose);
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorInspectorOverlaycpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/InspectorOverlay.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/InspectorOverlay.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/inspector/InspectorOverlay.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -513,8 +513,8 @@
</span><span class="cx">     m_paintRects.append(TimeRectPair(removeTime, rootRect));
</span><span class="cx"> 
</span><span class="cx">     if (!m_paintRectUpdateTimer.isActive()) {
</span><del>-        const double paintRectsUpdateIntervalSeconds = 0.032;
-        m_paintRectUpdateTimer.startRepeating(paintRectsUpdateIntervalSeconds);
</del><ins>+        const Seconds paintRectsUpdateInterval { 32_ms };
+        m_paintRectUpdateTimer.startRepeating(paintRectsUpdateInterval);
</ins><span class="cx">     }
</span><span class="cx"> 
</span><span class="cx">     drawPaintRects();
</span></span></pre></div>
<a id="trunkSourceWebCoreloaderProgressTrackercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/loader/ProgressTracker.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/loader/ProgressTracker.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/loader/ProgressTracker.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -52,7 +52,7 @@
</span><span class="cx"> static const int progressItemDefaultEstimatedLength = 1024 * 16;
</span><span class="cx"> 
</span><span class="cx"> // Check if the load is progressing this often.
</span><del>-static const double progressHeartbeatInterval = 0.1;
</del><ins>+static const Seconds progressHeartbeatInterval { 100_ms };
</ins><span class="cx"> 
</span><span class="cx"> // How many heartbeats must pass without progress before deciding the load is currently stalled.
</span><span class="cx"> static const unsigned loadStalledHeartbeatCount = 4;
</span></span></pre></div>
<a id="trunkSourceWebCorepageAutoscrollControllercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/AutoscrollController.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/AutoscrollController.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/page/AutoscrollController.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -44,8 +44,8 @@
</span><span class="cx"> // Delay time in second for start autoscroll if pointer is in border edge of scrollable element.
</span><span class="cx"> static const double autoscrollDelay = 0.2;
</span><span class="cx"> 
</span><del>-// When the autoscroll or the panScroll is triggered when do the scroll every 0.05s to make it smooth
-static const double autoscrollInterval = 0.05;
</del><ins>+// When the autoscroll or the panScroll is triggered when do the scroll every 50ms to make it smooth.
+static const Seconds autoscrollInterval { 50_ms };
</ins><span class="cx"> 
</span><span class="cx"> #if ENABLE(PAN_SCROLLING)
</span><span class="cx"> static Frame* getMainFrame(Frame* frame)
</span></span></pre></div>
<a id="trunkSourceWebCorepageFramecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/Frame.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/Frame.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/page/Frame.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -121,7 +121,7 @@
</span><span class="cx"> using namespace HTMLNames;
</span><span class="cx"> 
</span><span class="cx"> #if PLATFORM(IOS)
</span><del>-const unsigned scrollFrequency = 1000 / 60;
</del><ins>+static const Seconds scrollFrequency { 1000 / 60. };
</ins><span class="cx"> #endif
</span><span class="cx"> 
</span><span class="cx"> DEFINE_DEBUG_ONLY_GLOBAL(WTF::RefCountedLeakCounter, frameCounter, (&quot;Frame&quot;));
</span></span></pre></div>
<a id="trunkSourceWebCorepagePageOverlaycpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/PageOverlay.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/PageOverlay.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/page/PageOverlay.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -250,7 +250,7 @@
</span><span class="cx"> void PageOverlay::startFadeAnimation()
</span><span class="cx"> {
</span><span class="cx">     m_fadeAnimationStartTime = currentTime();
</span><del>-    m_fadeAnimationTimer.startRepeating(1 / fadeAnimationFrameRate);
</del><ins>+    m_fadeAnimationTimer.startRepeating(1_s / fadeAnimationFrameRate);
</ins><span class="cx"> }
</span><span class="cx"> 
</span><span class="cx"> void PageOverlay::fadeAnimationTimerFired()
</span></span></pre></div>
<a id="trunkSourceWebCorepageSuspendableTimerh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/SuspendableTimer.h (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/SuspendableTimer.h        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/page/SuspendableTimer.h        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -45,7 +45,6 @@
</span><span class="cx">     bool isActive() const { return TimerBase::isActive() || (m_suspended &amp;&amp; m_savedIsActive); }
</span><span class="cx">     bool isSuspended() const { return m_suspended; }
</span><span class="cx"> 
</span><del>-    void startRepeating(double repeatInterval) { startRepeating(Seconds { repeatInterval }); }
</del><span class="cx">     void startOneShot(double interval) { startOneShot(Seconds { interval }); }
</span><span class="cx"> 
</span><span class="cx">     Seconds repeatInterval() const;
</span></span></pre></div>
<a id="trunkSourceWebCorepageWheelEventTestTriggercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/WheelEventTestTrigger.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/WheelEventTestTrigger.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/page/WheelEventTestTrigger.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -60,7 +60,7 @@
</span><span class="cx">     }
</span><span class="cx">     
</span><span class="cx">     if (!m_testTriggerTimer.isActive())
</span><del>-        m_testTriggerTimer.startRepeating(1.0 / 60.0);
</del><ins>+        m_testTriggerTimer.startRepeating(1_s / 60.);
</ins><span class="cx"> }
</span><span class="cx"> 
</span><span class="cx"> void WheelEventTestTrigger::deferTestsForReason(ScrollableAreaIdentifier identifier, DeferTestTriggerReason reason)
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformThemeh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/Theme.h (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/Theme.h        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/platform/Theme.h        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -76,7 +76,7 @@
</span><span class="cx">     virtual FontCascade systemFont(ThemeFont, FontDescription&amp;) const { return FontCascade(); }
</span><span class="cx">     
</span><span class="cx">     // How fast the caret blinks in text fields.
</span><del>-    virtual double caretBlinkInterval() const { return 0.5; }
</del><ins>+    virtual Seconds caretBlinkInterval() const { return 500_ms; }
</ins><span class="cx"> 
</span><span class="cx">     // Notification when the theme has changed
</span><span class="cx">     virtual void themeChanged() { }
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformTimerh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/Timer.h (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/Timer.h        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/platform/Timer.h        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -54,7 +54,6 @@
</span><span class="cx"> 
</span><span class="cx">     WEBCORE_EXPORT void start(Seconds nextFireInterval, Seconds repeatInterval);
</span><span class="cx"> 
</span><del>-    void startRepeating(double repeatInterval) { startRepeating(Seconds { repeatInterval }); }
</del><span class="cx">     void startRepeating(Seconds repeatInterval) { start(repeatInterval, repeatInterval); }
</span><span class="cx"> 
</span><span class="cx">     void startOneShot(double interval) { startOneShot(Seconds { interval }); }
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformcocoaScrollControllermm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/cocoa/ScrollController.mm (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/cocoa/ScrollController.mm        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/platform/cocoa/ScrollController.mm        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -386,7 +386,7 @@
</span><span class="cx"> void ScrollController::startSnapRubberbandTimer()
</span><span class="cx"> {
</span><span class="cx">     m_client.startSnapRubberbandTimer();
</span><del>-    m_snapRubberbandTimer.startRepeating(1.0 / 60.0);
</del><ins>+    m_snapRubberbandTimer.startRepeating(1_s / 60.);
</ins><span class="cx"> 
</span><span class="cx">     m_client.deferTestsForReason(reinterpret_cast&lt;WheelEventTestTrigger::ScrollableAreaIdentifier&gt;(this), WheelEventTestTrigger::RubberbandInProgress);
</span><span class="cx"> }
</span><span class="lines">@@ -562,7 +562,7 @@
</span><span class="cx"> 
</span><span class="cx">     startDeferringTestsDueToScrollSnapping();
</span><span class="cx">     m_client.startScrollSnapTimer();
</span><del>-    m_scrollSnapTimer.startRepeating(1.0 / 60.0);
</del><ins>+    m_scrollSnapTimer.startRepeating(1_s / 60.);
</ins><span class="cx"> }
</span><span class="cx"> 
</span><span class="cx"> void ScrollController::stopScrollSnapTimer()
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformgraphicscaTileGridcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/graphics/ca/TileGrid.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/graphics/ca/TileGrid.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/platform/graphics/ca/TileGrid.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -523,7 +523,7 @@
</span><span class="cx"> 
</span><span class="cx"> void TileGrid::scheduleCohortRemoval()
</span><span class="cx"> {
</span><del>-    const double cohortRemovalTimerSeconds = 1;
</del><ins>+    const Seconds cohortRemovalTimerSeconds { 1_s };
</ins><span class="cx"> 
</span><span class="cx">     // Start the timer, or reschedule the timer from now if it's already active.
</span><span class="cx">     if (!m_cohortRemovalTimer.isActive())
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformgraphicsgstreamerMediaPlayerPrivateGStreamercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -1949,7 +1949,7 @@
</span><span class="cx">     if (shouldDownload) {
</span><span class="cx">         GST_DEBUG(&quot;Enabling on-disk buffering&quot;);
</span><span class="cx">         g_object_set(m_pipeline.get(), &quot;flags&quot;, flags | flagDownload, nullptr);
</span><del>-        m_fillTimer.startRepeating(0.2);
</del><ins>+        m_fillTimer.startRepeating(200_ms);
</ins><span class="cx">     } else {
</span><span class="cx">         GST_DEBUG(&quot;Disabling on-disk buffering&quot;);
</span><span class="cx">         g_object_set(m_pipeline.get(), &quot;flags&quot;, flags &amp; ~flagDownload, nullptr);
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformmediastreammacMockRealtimeAudioSourceMacmm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/mediastream/mac/MockRealtimeAudioSourceMac.mm (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/mediastream/mac/MockRealtimeAudioSourceMac.mm        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/platform/mediastream/mac/MockRealtimeAudioSourceMac.mm        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -125,7 +125,7 @@
</span><span class="cx"> 
</span><span class="cx"> void MockRealtimeAudioSourceMac::reconfigure()
</span><span class="cx"> {
</span><del>-    m_maximiumFrameCount = WTF::roundUpToPowerOfTwo(renderInterval() / 1000. * sampleRate() * 2);
</del><ins>+    m_maximiumFrameCount = WTF::roundUpToPowerOfTwo(renderInterval().seconds() * sampleRate() * 2);
</ins><span class="cx">     ASSERT(m_maximiumFrameCount);
</span><span class="cx"> 
</span><span class="cx">     const int bytesPerFloat = sizeof(Float32);
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformmockMockRealtimeAudioSourcecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/mock/MockRealtimeAudioSource.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/mock/MockRealtimeAudioSource.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/platform/mock/MockRealtimeAudioSource.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -109,7 +109,7 @@
</span><span class="cx">     MockRealtimeMediaSource::startProducingData();
</span><span class="cx"> 
</span><span class="cx">     m_startTime = monotonicallyIncreasingTime();
</span><del>-    m_timer.startRepeating(std::chrono::milliseconds(renderInterval()));
</del><ins>+    m_timer.startRepeating(renderInterval());
</ins><span class="cx"> }
</span><span class="cx"> 
</span><span class="cx"> void MockRealtimeAudioSource::stopProducingData()
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformmockMockRealtimeAudioSourceh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/mock/MockRealtimeAudioSource.h (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/mock/MockRealtimeAudioSource.h        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/platform/mock/MockRealtimeAudioSource.h        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -58,7 +58,7 @@
</span><span class="cx">     virtual void render(double) { }
</span><span class="cx"> 
</span><span class="cx">     double elapsedTime();
</span><del>-    static int renderInterval() { return 60; }
</del><ins>+    static Seconds renderInterval() { return 60_ms; }
</ins><span class="cx"> 
</span><span class="cx"> private:
</span><span class="cx"> 
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformmockMockRealtimeVideoSourcecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/mock/MockRealtimeVideoSource.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/mock/MockRealtimeVideoSource.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/platform/mock/MockRealtimeVideoSource.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -101,7 +101,7 @@
</span><span class="cx">     }
</span><span class="cx"> 
</span><span class="cx">     m_startTime = monotonicallyIncreasingTime();
</span><del>-    m_timer.startRepeating(std::chrono::milliseconds(lround(1000 / frameRate())));
</del><ins>+    m_timer.startRepeating(1_ms * lround(1000 / frameRate()));
</ins><span class="cx"> }
</span><span class="cx"> 
</span><span class="cx"> void MockRealtimeVideoSource::stopProducingData()
</span><span class="lines">@@ -155,7 +155,7 @@
</span><span class="cx"> bool MockRealtimeVideoSource::applyFrameRate(double rate)
</span><span class="cx"> {
</span><span class="cx">     if (m_timer.isActive())
</span><del>-        m_timer.startRepeating(std::chrono::milliseconds(lround(1000 / rate)));
</del><ins>+        m_timer.startRepeating(1_ms * lround(1000 / rate));
</ins><span class="cx"> 
</span><span class="cx">     updateSampleBuffer();
</span><span class="cx">     return true;
</span></span></pre></div>
<a id="trunkSourceWebCorerenderingRenderMarqueecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/RenderMarquee.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/rendering/RenderMarquee.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/rendering/RenderMarquee.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -168,7 +168,7 @@
</span><span class="cx">         m_stopped = false;
</span><span class="cx">     }
</span><span class="cx"> 
</span><del>-    m_timer.startRepeating(speed() * 0.001);
</del><ins>+    m_timer.startRepeating(1_ms * speed());
</ins><span class="cx"> }
</span><span class="cx"> 
</span><span class="cx"> void RenderMarquee::suspend()
</span><span class="lines">@@ -230,7 +230,7 @@
</span><span class="cx">     if (speed() != marqueeSpeed()) {
</span><span class="cx">         m_speed = marqueeSpeed();
</span><span class="cx">         if (m_timer.isActive())
</span><del>-            m_timer.startRepeating(speed() * 0.001);
</del><ins>+            m_timer.startRepeating(1_ms * speed());
</ins><span class="cx">     }
</span><span class="cx">     
</span><span class="cx">     // Check the loop count to see if we should now stop.
</span></span></pre></div>
<a id="trunkSourceWebCorerenderingRenderThemeh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/RenderTheme.h (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/rendering/RenderTheme.h        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/rendering/RenderTheme.h        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -178,7 +178,7 @@
</span><span class="cx"> #endif
</span><span class="cx">     virtual void platformColorsDidChange();
</span><span class="cx"> 
</span><del>-    virtual double caretBlinkInterval() const { return 0.5; }
</del><ins>+    virtual Seconds caretBlinkInterval() const { return 500_ms; }
</ins><span class="cx"> 
</span><span class="cx">     // System fonts and colors for CSS.
</span><span class="cx">     void systemFont(CSSValueID, FontCascadeDescription&amp;) const;
</span></span></pre></div>
<a id="trunkSourceWebCorerenderingRenderThemeGtkcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/RenderThemeGtk.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/rendering/RenderThemeGtk.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/rendering/RenderThemeGtk.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -1802,7 +1802,7 @@
</span><span class="cx"> }
</span><span class="cx"> #endif // GTK_CHECK_VERSION(3, 20, 0)
</span><span class="cx"> 
</span><del>-double RenderThemeGtk::caretBlinkInterval() const
</del><ins>+Seconds RenderThemeGtk::caretBlinkInterval() const
</ins><span class="cx"> {
</span><span class="cx">     GtkSettings* settings = gtk_settings_get_default();
</span><span class="cx"> 
</span><span class="lines">@@ -1812,9 +1812,9 @@
</span><span class="cx">     g_object_get(settings, &quot;gtk-cursor-blink&quot;, &amp;shouldBlink, &quot;gtk-cursor-blink-time&quot;, &amp;time, nullptr);
</span><span class="cx"> 
</span><span class="cx">     if (!shouldBlink)
</span><del>-        return 0;
</del><ins>+        return 0_s;
</ins><span class="cx"> 
</span><del>-    return time / 2000.;
</del><ins>+    return 2_ms * time;
</ins><span class="cx"> }
</span><span class="cx"> 
</span><span class="cx"> enum StyleColorType { StyleColorBackground, StyleColorForeground };
</span></span></pre></div>
<a id="trunkSourceWebCorerenderingRenderThemeGtkh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/RenderThemeGtk.h (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/rendering/RenderThemeGtk.h        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/rendering/RenderThemeGtk.h        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -81,7 +81,7 @@
</span><span class="cx">     Color platformInactiveListBoxSelectionBackgroundColor() const override;
</span><span class="cx">     Color platformInactiveListBoxSelectionForegroundColor() const override;
</span><span class="cx"> 
</span><del>-    double caretBlinkInterval() const override;
</del><ins>+    Seconds caretBlinkInterval() const override;
</ins><span class="cx"> 
</span><span class="cx">     void platformColorsDidChange() override;
</span><span class="cx"> 
</span></span></pre></div>
<a id="trunkSourceWebCorexmlXMLHttpRequestProgressEventThrottlecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/xml/XMLHttpRequestProgressEventThrottle.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/xml/XMLHttpRequestProgressEventThrottle.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/xml/XMLHttpRequestProgressEventThrottle.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -33,7 +33,7 @@
</span><span class="cx"> 
</span><span class="cx"> namespace WebCore {
</span><span class="cx"> 
</span><del>-const double XMLHttpRequestProgressEventThrottle::minimumProgressEventDispatchingIntervalInSeconds = .05; // 50 ms per specification.
</del><ins>+const Seconds XMLHttpRequestProgressEventThrottle::minimumProgressEventDispatchingInterval { 50_ms }; // 50 ms per specification.
</ins><span class="cx"> 
</span><span class="cx"> XMLHttpRequestProgressEventThrottle::XMLHttpRequestProgressEventThrottle(EventTarget* target)
</span><span class="cx">     : m_target(target)
</span><span class="lines">@@ -73,12 +73,12 @@
</span><span class="cx">         ASSERT(!m_hasThrottledProgressEvent);
</span><span class="cx"> 
</span><span class="cx">         dispatchEvent(XMLHttpRequestProgressEvent::create(eventNames().progressEvent, lengthComputable, loaded, total));
</span><del>-        startRepeating(minimumProgressEventDispatchingIntervalInSeconds);
</del><ins>+        startRepeating(minimumProgressEventDispatchingInterval);
</ins><span class="cx">         m_hasThrottledProgressEvent = false;
</span><span class="cx">         return;
</span><span class="cx">     }
</span><span class="cx"> 
</span><del>-    // The timer is already active so minimumProgressEventDispatchingIntervalInSeconds is the least frequent event.
</del><ins>+    // The timer is already active so minimumProgressEventDispatchingInterval is the least frequent event.
</ins><span class="cx">     m_hasThrottledProgressEvent = true;
</span><span class="cx"> }
</span><span class="cx"> 
</span></span></pre></div>
<a id="trunkSourceWebCorexmlXMLHttpRequestProgressEventThrottleh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/xml/XMLHttpRequestProgressEventThrottle.h (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/xml/XMLHttpRequestProgressEventThrottle.h        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebCore/xml/XMLHttpRequestProgressEventThrottle.h        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -55,7 +55,7 @@
</span><span class="cx">     void resume();
</span><span class="cx"> 
</span><span class="cx"> private:
</span><del>-    static const double minimumProgressEventDispatchingIntervalInSeconds;
</del><ins>+    static const Seconds minimumProgressEventDispatchingInterval;
</ins><span class="cx"> 
</span><span class="cx">     void fired() override;
</span><span class="cx">     void dispatchDeferredEvents();
</span></span></pre></div>
<a id="trunkSourceWebKitmacChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/mac/ChangeLog (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/mac/ChangeLog        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebKit/mac/ChangeLog        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -1,5 +1,17 @@
</span><span class="cx"> 2017-04-09  Chris Dumez  &lt;cdumez@apple.com&gt;
</span><span class="cx"> 
</span><ins>+        Drop Timer::startRepeating() overload taking a double
+        https://bugs.webkit.org/show_bug.cgi?id=170656
+
+        Reviewed by Yusuke Suzuki.
+
+        Drop Timer::startRepeating() overload taking a double as people should use Seconds type now.
+
+        * Plugins/WebNetscapePluginView.mm:
+        (PluginTimer::start):
+
+2017-04-09  Chris Dumez  &lt;cdumez@apple.com&gt;
+
</ins><span class="cx">         Start dropping Timer API dealing with double
</span><span class="cx">         https://bugs.webkit.org/show_bug.cgi?id=170649
</span><span class="cx"> 
</span></span></pre></div>
<a id="trunkSourceWebKitmacPluginsWebNetscapePluginViewmm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/mac/Plugins/WebNetscapePluginView.mm (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/mac/Plugins/WebNetscapePluginView.mm        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebKit/mac/Plugins/WebNetscapePluginView.mm        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -136,7 +136,7 @@
</span><span class="cx">             timeInterval = std::max(timeInterval, ThrottledTimerInterval);
</span><span class="cx">         
</span><span class="cx">         if (m_repeat)
</span><del>-            startRepeating(timeInterval);
</del><ins>+            startRepeating(Seconds { timeInterval });
</ins><span class="cx">         else
</span><span class="cx">             startOneShot(timeInterval);
</span><span class="cx">     }
</span></span></pre></div>
<a id="trunkSourceWebKitwinChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/win/ChangeLog (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/win/ChangeLog        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebKit/win/ChangeLog        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -1,5 +1,16 @@
</span><span class="cx"> 2017-04-09  Chris Dumez  &lt;cdumez@apple.com&gt;
</span><span class="cx"> 
</span><ins>+        Drop Timer::startRepeating() overload taking a double
+        https://bugs.webkit.org/show_bug.cgi?id=170656
+
+        Reviewed by Yusuke Suzuki.
+
+        Drop Timer::startRepeating() overload taking a double as people should use Seconds type now.
+
+        * FullscreenVideoController.cpp:
+
+2017-04-09  Chris Dumez  &lt;cdumez@apple.com&gt;
+
</ins><span class="cx">         Start dropping Timer API dealing with double
</span><span class="cx">         https://bugs.webkit.org/show_bug.cgi?id=170649
</span><span class="cx"> 
</span></span></pre></div>
<a id="trunkSourceWebKitwinFullscreenVideoControllercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/win/FullscreenVideoController.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/win/FullscreenVideoController.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebKit/win/FullscreenVideoController.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -52,7 +52,7 @@
</span><span class="cx"> using namespace std;
</span><span class="cx"> using namespace WebCore;
</span><span class="cx"> 
</span><del>-static const float timerInterval = 0.033;
</del><ins>+static const Seconds timerInterval { 33_ms };
</ins><span class="cx"> 
</span><span class="cx"> // HUD Size
</span><span class="cx"> static const int windowHeight = 59;
</span></span></pre></div>
<a id="trunkSourceWebKit2ChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/ChangeLog (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/ChangeLog        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebKit2/ChangeLog        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -1,3 +1,23 @@
</span><ins>+2017-04-09  Chris Dumez  &lt;cdumez@apple.com&gt;
+
+        Drop Timer::startRepeating() overload taking a double
+        https://bugs.webkit.org/show_bug.cgi?id=170656
+
+        Reviewed by Yusuke Suzuki.
+
+        Drop Timer::startRepeating() overload taking a double as people should use Seconds type now.
+
+        * Shared/WebMemorySampler.cpp:
+        (WebKit::WebMemorySampler::initializeTimers):
+        * UIProcess/Cocoa/ViewGestureController.cpp:
+        * UIProcess/HighPerformanceGraphicsUsageSampler.cpp:
+        * UIProcess/PerActivityStateCPUUsageSampler.cpp:
+        * WebProcess/Plugins/Netscape/NetscapePlugin.cpp:
+        (WebKit::NetscapePlugin::Timer::start):
+        * WebProcess/Plugins/Netscape/mac/NetscapePluginMac.mm:
+        * WebProcess/WebPage/CoordinatedGraphics/CompositingCoordinator.cpp:
+        (WebKit::CompositingCoordinator::scheduleReleaseInactiveAtlases):
+
</ins><span class="cx"> 2017-04-09  Jon Lee  &lt;jonlee@apple.com&gt;
</span><span class="cx"> 
</span><span class="cx">         [WK2][iOS] Add CoreMedia entitlement for WebProcess
</span></span></pre></div>
<a id="trunkSourceWebKit2SharedWebMemorySamplercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/Shared/WebMemorySampler.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/Shared/WebMemorySampler.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebKit2/Shared/WebMemorySampler.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -88,7 +88,7 @@
</span><span class="cx"> 
</span><span class="cx"> void WebMemorySampler::initializeTimers(double interval)
</span><span class="cx"> {
</span><del>-    m_sampleTimer.startRepeating(1);
</del><ins>+    m_sampleTimer.startRepeating(1_s);
</ins><span class="cx">     printf(&quot;Started memory sampler for process %s %d&quot;, processName().utf8().data(), getpid());
</span><span class="cx">     if (interval &gt; 0) {
</span><span class="cx">         m_stopTimer.startOneShot(interval);
</span></span></pre></div>
<a id="trunkSourceWebKit2UIProcessCocoaViewGestureControllercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/UIProcess/Cocoa/ViewGestureController.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/UIProcess/Cocoa/ViewGestureController.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebKit2/UIProcess/Cocoa/ViewGestureController.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -41,7 +41,7 @@
</span><span class="cx"> namespace WebKit {
</span><span class="cx"> 
</span><span class="cx"> static const std::chrono::seconds swipeSnapshotRemovalWatchdogAfterFirstVisuallyNonEmptyLayoutDuration = 3s;
</span><del>-static const std::chrono::milliseconds swipeSnapshotRemovalActiveLoadMonitoringInterval = 250ms;
</del><ins>+static const Seconds swipeSnapshotRemovalActiveLoadMonitoringInterval = 250_ms;
</ins><span class="cx"> 
</span><span class="cx"> #if PLATFORM(MAC)
</span><span class="cx"> static const std::chrono::seconds swipeSnapshotRemovalWatchdogDuration = 5s;
</span></span></pre></div>
<a id="trunkSourceWebKit2UIProcessHighPerformanceGraphicsUsageSamplercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/UIProcess/HighPerformanceGraphicsUsageSampler.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/UIProcess/HighPerformanceGraphicsUsageSampler.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebKit2/UIProcess/HighPerformanceGraphicsUsageSampler.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -33,7 +33,7 @@
</span><span class="cx"> 
</span><span class="cx"> using namespace WebCore;
</span><span class="cx"> 
</span><del>-static const std::chrono::minutes samplingInterval { 10 };
</del><ins>+static const Seconds samplingInterval { 10_min };
</ins><span class="cx"> 
</span><span class="cx"> HighPerformanceGraphicsUsageSampler::HighPerformanceGraphicsUsageSampler(WebProcessPool&amp; webProcessPool)
</span><span class="cx">     : m_webProcessPool(webProcessPool)
</span></span></pre></div>
<a id="trunkSourceWebKit2UIProcessPerActivityStateCPUUsageSamplercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/UIProcess/PerActivityStateCPUUsageSampler.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/UIProcess/PerActivityStateCPUUsageSampler.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebKit2/UIProcess/PerActivityStateCPUUsageSampler.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -38,7 +38,7 @@
</span><span class="cx"> 
</span><span class="cx"> using namespace WebCore;
</span><span class="cx"> 
</span><del>-static const std::chrono::minutes loggingInterval { 60 };
</del><ins>+static const Seconds loggingInterval { 60_min };
</ins><span class="cx"> 
</span><span class="cx"> PerActivityStateCPUUsageSampler::PerActivityStateCPUUsageSampler(WebProcessPool&amp; processPool)
</span><span class="cx">     : m_processPool(processPool)
</span></span></pre></div>
<a id="trunkSourceWebKit2WebProcessPluginsNetscapeNetscapePlugincpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebProcess/Plugins/Netscape/NetscapePlugin.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/Plugins/Netscape/NetscapePlugin.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebKit2/WebProcess/Plugins/Netscape/NetscapePlugin.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -326,7 +326,7 @@
</span><span class="cx"> 
</span><span class="cx"> void NetscapePlugin::Timer::start()
</span><span class="cx"> {
</span><del>-    double timeInterval = m_interval / 1000.0;
</del><ins>+    Seconds timeInterval = 1_ms * m_interval;
</ins><span class="cx"> 
</span><span class="cx">     if (m_repeat)
</span><span class="cx">         m_timer.startRepeating(timeInterval);
</span></span></pre></div>
<a id="trunkSourceWebKit2WebProcessPluginsNetscapemacNetscapePluginMacmm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebProcess/Plugins/Netscape/mac/NetscapePluginMac.mm (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/Plugins/Netscape/mac/NetscapePluginMac.mm        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebKit2/WebProcess/Plugins/Netscape/mac/NetscapePluginMac.mm        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -45,7 +45,7 @@
</span><span class="cx"> namespace WebKit {
</span><span class="cx"> 
</span><span class="cx"> #ifndef NP_NO_CARBON
</span><del>-static const double nullEventIntervalActive = 0.02;
</del><ins>+static const Seconds nullEventIntervalActive { 20_ms };
</ins><span class="cx"> 
</span><span class="cx"> static unsigned buttonStateFromLastMouseEvent;
</span><span class="cx"> 
</span></span></pre></div>
<a id="trunkSourceWebKit2WebProcessWebPageCoordinatedGraphicsCompositingCoordinatorcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/CompositingCoordinator.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/CompositingCoordinator.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/CompositingCoordinator.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -404,12 +404,12 @@
</span><span class="cx">     return m_updateAtlases.last()-&gt;paintOnAvailableBuffer(size, atlasID, offset, client);
</span><span class="cx"> }
</span><span class="cx"> 
</span><del>-const double ReleaseInactiveAtlasesTimerInterval = 0.5;
</del><ins>+const Seconds releaseInactiveAtlasesTimerInterval { 500_ms };
</ins><span class="cx"> 
</span><span class="cx"> void CompositingCoordinator::scheduleReleaseInactiveAtlases()
</span><span class="cx"> {
</span><span class="cx">     if (!m_releaseInactiveAtlasesTimer.isActive())
</span><del>-        m_releaseInactiveAtlasesTimer.startRepeating(ReleaseInactiveAtlasesTimerInterval);
</del><ins>+        m_releaseInactiveAtlasesTimer.startRepeating(releaseInactiveAtlasesTimerInterval);
</ins><span class="cx"> }
</span><span class="cx"> 
</span><span class="cx"> void CompositingCoordinator::releaseInactiveAtlasesTimerFired()
</span><span class="lines">@@ -426,7 +426,7 @@
</span><span class="cx">         UpdateAtlas* atlas = m_updateAtlases[i].get();
</span><span class="cx">         bool inUse = atlas-&gt;isInUse();
</span><span class="cx">         if (!inUse)
</span><del>-            atlas-&gt;addTimeInactive(ReleaseInactiveAtlasesTimerInterval);
</del><ins>+            atlas-&gt;addTimeInactive(releaseInactiveAtlasesTimerInterval.value());
</ins><span class="cx">         bool usableForRootContentsLayer = !atlas-&gt;supportsAlpha();
</span><span class="cx">         if (atlas-&gt;isInactive() || (!inUse &amp;&amp; policy == ReleaseUnused)) {
</span><span class="cx">             if (!foundActiveAtlasForRootContentsLayer &amp;&amp; !atlasToKeepAnyway &amp;&amp; usableForRootContentsLayer)
</span></span></pre></div>
<a id="trunkToolsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Tools/ChangeLog (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/ChangeLog        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Tools/ChangeLog        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -1,3 +1,15 @@
</span><ins>+2017-04-09  Chris Dumez  &lt;cdumez@apple.com&gt;
+
+        Drop Timer::startRepeating() overload taking a double
+        https://bugs.webkit.org/show_bug.cgi?id=170656
+
+        Reviewed by Yusuke Suzuki.
+
+        Drop Timer::startRepeating() overload taking a double as people should use Seconds type now.
+
+        * TestWebKitAPI/Tests/WTF/RunLoop.cpp:
+        (TestWebKitAPI::TEST):
+
</ins><span class="cx"> 2017-04-09  Youenn Fablet  &lt;youenn@apple.com&gt;
</span><span class="cx"> 
</span><span class="cx">         Resync WPT tests up to 23cd07d4685d81024b7440e042f8bbdb33e7ddec
</span></span></pre></div>
<a id="trunkToolsTestWebKitAPITestsWTFRunLoopcpp"></a>
<div class="modfile"><h4>Modified: trunk/Tools/TestWebKitAPI/Tests/WTF/RunLoop.cpp (215166 => 215167)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/TestWebKitAPI/Tests/WTF/RunLoop.cpp        2017-04-09 21:48:06 UTC (rev 215166)
+++ trunk/Tools/TestWebKitAPI/Tests/WTF/RunLoop.cpp        2017-04-09 23:42:02 UTC (rev 215167)
</span><span class="lines">@@ -129,7 +129,7 @@
</span><span class="cx"> 
</span><span class="cx">     {
</span><span class="cx">         DerivedTimer timer(testFinished);
</span><del>-        timer.startRepeating(0.01);
</del><ins>+        timer.startRepeating(10_ms);
</ins><span class="cx">         Util::run(&amp;testFinished);
</span><span class="cx">     }
</span><span class="cx"> }
</span></span></pre>
</div>
</div>

</body>
</html>