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

Unified Diff: Source/platform/scheduler/Scheduler.cpp

Issue 705003002: Change the return type of WTF::bind() to |OwnPtr<Function>| (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Reflect comments from yhirano Created 6 years, 1 month 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
« no previous file with comments | « Source/platform/scheduler/Scheduler.h ('k') | Source/web/WebEmbeddedWorkerImpl.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/platform/scheduler/Scheduler.cpp
diff --git a/Source/platform/scheduler/Scheduler.cpp b/Source/platform/scheduler/Scheduler.cpp
index b91644a2d14c26805c3a91432a5629b8ec797cf7..535f114114da747340502f20efd830cbf81440d0 100644
--- a/Source/platform/scheduler/Scheduler.cpp
+++ b/Source/platform/scheduler/Scheduler.cpp
@@ -14,7 +14,7 @@ namespace blink {
class IdleTaskRunner : public WebScheduler::IdleTask {
public:
- explicit IdleTaskRunner(const Scheduler::IdleTask& task)
+ explicit IdleTaskRunner(PassOwnPtr<Scheduler::IdleTask> task)
: m_task(task)
{
}
@@ -26,10 +26,10 @@ public:
// WebScheduler::IdleTask implementation.
void run(double deadlineSeconds) override
{
- m_task(deadlineSeconds);
+ (*m_task)(deadlineSeconds);
}
private:
- Scheduler::IdleTask m_task;
+ OwnPtr<Scheduler::IdleTask> m_task;
};
Scheduler* Scheduler::s_sharedScheduler = nullptr;
@@ -58,7 +58,7 @@ Scheduler::~Scheduler()
m_webScheduler->shutdown();
}
-void Scheduler::postIdleTask(const TraceLocation& location, const IdleTask& idleTask)
+void Scheduler::postIdleTask(const TraceLocation& location, PassOwnPtr<IdleTask> idleTask)
{
if (m_webScheduler)
m_webScheduler->postIdleTask(WebTraceLocation(location), new IdleTaskRunner(idleTask));
« no previous file with comments | « Source/platform/scheduler/Scheduler.h ('k') | Source/web/WebEmbeddedWorkerImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698