[Blink-Worker] WorkerThread fires idleHandler only at the end of processing all tasks.

Source/core/workers/WorkerThread.h

 /*
  * Copyright (C) 2008 Apple 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 76 matching lines @@
         void didLeaveNestedLoop();
 
         WorkerGlobalScope* workerGlobalScope() const { return m_workerGlobalScope.get(); }
         bool terminated() const { return m_terminated; }
 
         // Number of active worker threads.
         static unsigned workerThreadCount();
 
         void interruptAndDispatchInspectorCommands();
         void setWorkerInspectorController(WorkerInspectorController*);
+        unsigned decrementAndReturnTaskCount();

[jochen (gone - plz use gerrit), 2014/09/02 10:14:14]

those should all be private

[Mayur Kankanwadi, 2014/09/03 11:26:22]

These functions are accessed from within the WorkerThreadTask, hence they need
to be public.

+        unsigned taskCount();
+        void queueUpIdleHandlerNow();
+        void decrementTaskCount();
+        bool isIdleHandlerTaskFiredOnce();
 
     protected:
         WorkerThread(WorkerLoaderProxy&, WorkerReportingProxy&, PassOwnPtrWillBeRawPtr<WorkerThreadStartupData>);
 
         // Factory method for creating a new worker context for the thread.
         virtual PassRefPtrWillBeRawPtr<WorkerGlobalScope> createWorkerGlobalScope(PassOwnPtrWillBeRawPtr<WorkerThreadStartupData>) = 0;
 
         virtual void postInitialize() { }
 
     private:
@@ skipping 17 matching lines @@
 
         RefPtrWillBePersistent<WorkerInspectorController> m_workerInspectorController;
         Mutex m_workerInspectorControllerMutex;
 
         Mutex m_threadCreationMutex;
         RefPtrWillBePersistent<WorkerGlobalScope> m_workerGlobalScope;
         OwnPtrWillBePersistent<WorkerThreadStartupData> m_startupData;
 
         // Used to signal thread shutdown.
         OwnPtr<blink::WebWaitableEvent> m_shutdownEvent;
+        Mutex m_taskCounterMutex;
+        unsigned m_tasksCount;
+        bool m_isIdleHandlerTaskFiredOnce;
+        bool m_isIdleHandlerTask;
 
         // FIXME: This has to be last because of crbug.com/401397 - the
         // WorkerThread might get deleted before it had a chance to properly
         // shut down. By deleting the WebThread first, we can guarantee that
         // no pending tasks on the thread might want to access any of the other
         // members during the WorkerThread's destruction.
         OwnPtr<blink::WebThread> m_thread;
     };
 
 } // namespace blink
 
 #endif // WorkerThread_h