Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(761)

Unified Diff: third_party/WebKit/Source/platform/Timer.cpp

Issue 2550373005: Make WebTaskRunner ThreadSafeRefCounted (Closed)
Patch Set: mac fix Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/Timer.cpp
diff --git a/third_party/WebKit/Source/platform/Timer.cpp b/third_party/WebKit/Source/platform/Timer.cpp
index b02e8b6c8e52b447cd9e6b50c130d73445ec50f8..c0c09b8b5bb39d8101bfce48c1f7e7d508e112d7 100644
--- a/third_party/WebKit/Source/platform/Timer.cpp
+++ b/third_party/WebKit/Source/platform/Timer.cpp
@@ -40,10 +40,10 @@
namespace blink {
-TimerBase::TimerBase(WebTaskRunner* webTaskRunner)
+TimerBase::TimerBase(RefPtr<WebTaskRunner> webTaskRunner)
: m_nextFireTime(0),
m_repeatInterval(0),
- m_webTaskRunner(webTaskRunner->clone()),
+ m_webTaskRunner(std::move(webTaskRunner)),
#if DCHECK_IS_ON()
m_thread(currentThread()),
#endif
@@ -86,17 +86,17 @@ double TimerBase::nextFireInterval() const {
}
// static
-WebTaskRunner* TimerBase::getTimerTaskRunner() {
+RefPtr<WebTaskRunner> TimerBase::getTimerTaskRunner() {
return Platform::current()->currentThread()->scheduler()->timerTaskRunner();
}
// static
-WebTaskRunner* TimerBase::getUnthrottledTaskRunner() {
+RefPtr<WebTaskRunner> TimerBase::getUnthrottledTaskRunner() {
return Platform::current()->currentThread()->getWebTaskRunner();
}
-WebTaskRunner* TimerBase::timerTaskRunner() const {
- return m_webTaskRunner.get();
+RefPtr<WebTaskRunner> TimerBase::timerTaskRunner() const {
+ return m_webTaskRunner;
}
void TimerBase::setNextFireTime(double now, double delay) {

Powered by Google App Engine
This is Rietveld 408576698