Remove static thread pointers from CC

cc/proxy.cc

 // Copyright 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 
 #include "cc/proxy.h"
 
 #include "cc/thread.h"
+#include "cc/thread_impl.h"
 
 namespace cc {
 
-namespace {
-#ifndef NDEBUG
-bool implThreadIsOverridden = false;
-bool s_isMainThreadBlocked = false;
-#endif
-Thread* s_mainThread = 0;
-Thread* s_implThread = 0;
+void Proxy::setMainThread(Thread* thread)
+{
+    m_mainThread.reset(thread);
 }
 
-void Proxy::setMainThread(Thread* thread)
+Thread* Proxy::mainThread() const
 {
-    s_mainThread = thread;
+    return m_mainThread.get();
 }
 
-Thread* Proxy::mainThread()
+bool Proxy::hasImplThread() const
 {
-    return s_mainThread;
-}
-
-bool Proxy::hasImplThread()
-{
-    return s_implThread;
+    return !!m_implThread.get();

[jamesr, 2012/10/31 04:42:13]

return m_implThread; will do the right thing

 }
 
 void Proxy::setImplThread(Thread* thread)
 {
-    s_implThread = thread;
+    m_implThread.reset(thread);
 }
 
-Thread* Proxy::implThread()
+Thread* Proxy::implThread() const
 {
-    return s_implThread;
+    return m_implThread.get();
 }
 
-Thread* Proxy::currentThread()
+Thread* Proxy::currentThread() const
 {
-    if (s_mainThread && s_mainThread->belongsToCurrentThread())
-        return s_mainThread;
-    if (s_implThread && s_implThread->belongsToCurrentThread())
-        return s_implThread;
+    if (mainThread() && mainThread()->belongsToCurrentThread())
+        return mainThread();
+    if (implThread() && implThread()->belongsToCurrentThread())
+        return implThread();
     return 0;
 }
 
-bool Proxy::isMainThread()
+bool Proxy::isMainThread() const
 {
 #ifndef NDEBUG
-    DCHECK(s_mainThread);
-    if (implThreadIsOverridden)
+    DCHECK(mainThread());
+    if (m_implThreadIsOverridden)
         return false;
-    return s_mainThread->belongsToCurrentThread();
+    return mainThread()->belongsToCurrentThread();
 #else
     return true;
 #endif
 }
 
-bool Proxy::isImplThread()
+bool Proxy::isImplThread() const
 {
 #ifndef NDEBUG
-    if (implThreadIsOverridden)
+    if (m_implThreadIsOverridden)
         return true;
-    return s_implThread && s_implThread->belongsToCurrentThread();
+    return implThread() && implThread()->belongsToCurrentThread();
 #else
     return true;
 #endif
 }
 
 #ifndef NDEBUG
 void Proxy::setCurrentThreadIsImplThread(bool isImplThread)
 {
-    implThreadIsOverridden = isImplThread;
+    m_implThreadIsOverridden = isImplThread;
 }
 #endif
 
-bool Proxy::isMainThreadBlocked()
+bool Proxy::isMainThreadBlocked() const
 {
 #ifndef NDEBUG
-    return s_isMainThreadBlocked;
+    return m_isMainThreadBlocked;
 #else
     return true;
 #endif
 }
 
 #ifndef NDEBUG
 void Proxy::setMainThreadBlocked(bool isMainThreadBlocked)
 {
-    s_isMainThreadBlocked = isMainThreadBlocked;
+    m_isMainThreadBlocked = isMainThreadBlocked;
 }
 #endif
 
-Proxy::Proxy()
+Proxy::Proxy(Thread* implThread)
+    : m_mainThread(ThreadImpl::createForCurrentThread())
+    , m_implThread(implThread)
+#ifndef NDEBUG
+    , m_implThreadIsOverridden(false)
+    , m_isMainThreadBlocked(false)
+#endif
 {
-    DCHECK(isMainThread());
 }
 
 Proxy::~Proxy()
 {
     DCHECK(isMainThread());
 }
 
 }