Add ASSERT(isMainThread()) to WorkerThread::start() and ::terminate()

Source/core/workers/WorkerThread.cpp

 /*
  * 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 159 matching lines @@
 
 WorkerThread::~WorkerThread()
 {
     MutexLocker lock(threadSetMutex());
     ASSERT(workerThreads().contains(this));
     workerThreads().remove(this);
 }
 
 void WorkerThread::start(PassOwnPtr<WorkerThreadStartupData> startupData)
 {
+    ASSERT(isMainThread());
     if (m_started)
         return;
 
     m_started = true;
     backingThread().postTask(FROM_HERE, new Task(threadSafeBind(&WorkerThread::initialize, AllowCrossThreadAccess(this), startupData)));
 }
 
 void WorkerThread::interruptAndDispatchInspectorCommands()
 {
     MutexLocker locker(m_workerInspectorControllerMutex);
@@ skipping 91 matching lines @@
 
     // Notify the proxy that the WorkerGlobalScope has been disposed of.
     // This can free this thread object, hence it must not be touched afterwards.
     workerReportingProxy().workerThreadTerminated();
 
     m_terminationEvent->signal();
 }
 
 void WorkerThread::terminate()
 {
+    ASSERT(isMainThread());

[kinuko, 2015/06/12 05:48:03]

Should this rather be in terminateInternal() ?

[Takashi Toyoshima, 2015/06/15 05:01:03]

Done.

     // Prevent the deadlock between GC and an attempt to terminate a thread.
     SafePointScope safePointScope(ThreadState::HeapPointersOnStack);
     terminateInternal();
 }
 
 void WorkerThread::terminateAndWait()
 {
     terminate();
     m_terminationEvent->wait();
 }
@@ skipping 182 matching lines @@
     InspectorInstrumentation::didLeaveNestedRunLoop(m_workerGlobalScope.get());
 }
 
 void WorkerThread::setWorkerInspectorController(WorkerInspectorController* workerInspectorController)
 {
     MutexLocker locker(m_workerInspectorControllerMutex);
     m_workerInspectorController = workerInspectorController;
 }
 
 } // namespace blink