Worker: Fix broken GC logic on Dedicated Worker while DOMTimer is set

third_party/WebKit/Source/core/workers/DedicatedWorkerThread.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 48 matching lines @@
 WorkerOrWorkletGlobalScope* DedicatedWorkerThread::createWorkerGlobalScope(std::unique_ptr<WorkerThreadStartupData> startupData)
 {
     return DedicatedWorkerGlobalScope::create(this, std::move(startupData), m_timeOrigin);
 }
 
 ConsoleMessageStorage* DedicatedWorkerThread::consoleMessageStorage()
 {
     return toWorkerGlobalScope(globalScope())->consoleMessageStorage();
 }
 
 void DedicatedWorkerThread::postInitialize()

[kinuko, 2016/08/05 14:55:28]

nit: can we rename this to... say, didInitializeOnWorkerThread ?  (postXxx is
confusing, would be nice to have threading suffix)

[nhiroki, 2016/08/08 09:19:13]

Agreed. Renamed.

 {
     // Notify the parent object of our current active state before the event
     // loop starts processing tasks.
-    m_workerObjectProxy.reportPendingActivity(false);
+    m_workerObjectProxy.reportPendingActivity(toWorkerGlobalScope(globalScope()));
 }
 
 } // namespace blink