Use the scheduling mechanism provided by the platform

Source/web/WebKit.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 32 matching lines @@
 #include "gin/public/v8_platform.h"
 #include "modules/InitModules.h"
 #include "platform/LayoutTestSupport.h"
 #include "platform/Logging.h"
 #include "platform/RuntimeEnabledFeatures.h"
 #include "platform/graphics/ImageDecodingStore.h"
 #include "platform/graphics/media/MediaPlayer.h"
 #include "platform/heap/Heap.h"
 #include "platform/heap/glue/MessageLoopInterruptor.h"
 #include "platform/heap/glue/PendingGCRunner.h"
-#include "platform/scheduler/Scheduler.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebPrerenderingSupport.h"
+#include "public/platform/WebScheduler.h"
 #include "public/platform/WebThread.h"
 #include "web/IndexedDBClientImpl.h"
 #include "web/WebMediaPlayerClientImpl.h"
 #include "wtf/Assertions.h"
 #include "wtf/CryptographicallyRandomNumber.h"
 #include "wtf/MainThread.h"
 #include "wtf/WTF.h"
 #include "wtf/text/AtomicString.h"
 #include "wtf/text/TextEncoding.h"
 #include <v8.h>
@@ skipping 75 matching lines @@
     ASSERT(!s_webKitInitialized);
     s_webKitInitialized = true;
 
     ASSERT(platform);
     Platform::initialize(platform);
 
     WTF::setRandomSource(cryptographicallyRandomValues);
     WTF::initialize(currentTimeFunction, monotonicallyIncreasingTimeFunction);
     WTF::initializeMainThread(callOnMainThreadFunction);
     Heap::init();
-    Scheduler::initializeOnMainThread();
 
     ThreadState::attachMainThread();
     // currentThread will always be non-null in production, but can be null in Chromium unit tests.
     if (WebThread* currentThread = platform->currentThread()) {
         ASSERT(!s_pendingGCRunner);
         s_pendingGCRunner = new PendingGCRunner;
         currentThread->addTaskObserver(s_pendingGCRunner);
 
         ASSERT(!s_messageLoopInterruptor);
         s_messageLoopInterruptor = new MessageLoopInterruptor(currentThread);
@@ skipping 57 matching lines @@
 
     V8PerIsolateData::destroy(isolate);
 
     shutdownWithoutV8();
 }
 
 void shutdownWithoutV8()
 {
     ASSERT(!s_endOfTaskRunner);
     CoreInitializer::shutdown();
-    Scheduler::shutdown();
+    if (WebScheduler* scheduler = Platform::current()->scheduler())

[eseidel, 2014/10/21 16:11:02]

Does this need to be before WTF and Heap?  Is this just draining the event
queue?

[Sami, 2014/10/21 18:49:34]

Yes, it needs to be right between CoreInitializer::shutdown() (which can still
post new tasks, e.g., from JS) and Heap::shutdown() (which makes any posted
tasks invalid -- we can't even delete them safely after that point). Shutdown
deletes all the pending tasks we have queued up but doesn't run them.

+        scheduler->shutdown();
     Heap::shutdown();
     WTF::shutdown();
     Platform::shutdown();
     WebPrerenderingSupport::shutdown();
 }
 
 void setLayoutTestMode(bool value)
 {
     LayoutTestSupport::setIsRunningLayoutTest(value);
 }
@@ skipping 21 matching lines @@
         channel->state = WTFLogChannelOn;
 #endif // !LOG_DISABLED
 }
 
 void resetPluginCache(bool reloadPages)
 {
     Page::refreshPlugins(reloadPages);
 }
 
 } // namespace blink