Revert 76199 - Trying changes to see the perfomance impact on Mac.

This change list is same as CL 6588039.

BUG=73915

Review URL: http://codereview.chromium.org/6602004

TBR=rtenneti@chromium.org
Review URL: http://codereview.chromium.org/6591035

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76200 0039d316-1c4b-4281-b951-d872f2087c98

11 files changed, 115 insertions, 253 deletions

diff --git a/chrome/browser/browser_main.cc b/chrome/browser/browser_main.cc
index d3f390a..2f494cc 100644
--- a/chrome/browser/browser_main.cc
+++ b/chrome/browser/browser_main.cc
@@ -45,7 +45,6 @@
 #include "chrome/browser/metrics/histogram_synchronizer.h"
 #include "chrome/browser/metrics/metrics_log.h"
 #include "chrome/browser/metrics/metrics_service.h"
-#include "chrome/browser/metrics/thread_watcher.h"
 #include "chrome/browser/net/blob_url_request_job_factory.h"
 #include "chrome/browser/net/chrome_dns_cert_provenance_checker.h"
 #include "chrome/browser/net/chrome_dns_cert_provenance_checker_factory.h"
@@ -703,9 +702,6 @@ void CreateChildThreads(BrowserProcessImpl* process) {
   process->process_launcher_thread();
   process->cache_thread();
   process->io_thread();
-  // Create watchdog thread after creating all other threads because it will
-  // watch the other threads and they must be running.
-  process->watchdog_thread();
 }
 
 // Returns the new local state object, guaranteed non-NULL.
@@ -1357,11 +1353,6 @@ int BrowserMain(const MainFunctionParams& parameters) {
   scoped_refptr<HistogramSynchronizer> histogram_synchronizer(
       new HistogramSynchronizer());
 
-  // Initialize thread watcher system. This is a singleton and is used by
-  // WatchDogThread to keep track of information about threads that are being
-  // watched.
-  scoped_ptr<ThreadWatcherList> thread_watcher_list(new ThreadWatcherList());
-
   // Initialize the prefs of the local state.
   browser::RegisterLocalState(local_state);
 
@@ -1395,14 +1386,6 @@ int BrowserMain(const MainFunctionParams& parameters) {
 
   CreateChildThreads(browser_process.get());
 
-  // Start watching all browser threads for responsiveness.
-  MessageLoop* message_loop = WatchDogThread::CurrentMessageLoop();
-  if (message_loop)
-    message_loop->PostDelayedTask(
-        FROM_HERE,
-        NewRunnableFunction(&WatchDogThread::StartWatchingAll),
-        base::TimeDelta::FromSeconds(10).InMilliseconds());
-
 #if defined(OS_CHROMEOS)
   // Now that the file thread exists we can record our stats.
   chromeos::BootTimesLoader::Get()->RecordChromeMainStats();
@@ -1920,9 +1903,6 @@ int BrowserMain(const MainFunctionParams& parameters) {
 
   process_singleton.Cleanup();
 
-  // Stop all tasks that might run on WatchDogThread.
-  ThreadWatcherList::StopWatchingAll();
-
   metrics->Stop();
 
   // browser_shutdown takes care of deleting browser_process, so we need to
diff --git a/chrome/browser/browser_process.h b/chrome/browser/browser_process.h
index 8a1ab65a..c1275db 100644
--- a/chrome/browser/browser_process.h
+++ b/chrome/browser/browser_process.h
@@ -39,7 +39,6 @@ class ResourceDispatcherHost;
 class SidebarManager;
 class TabCloseableStateWatcher;
 class ThumbnailGenerator;
-class WatchDogThread;
 
 namespace base {
 class Thread;
@@ -116,9 +115,6 @@ class BrowserProcess {
   virtual base::Thread* background_x11_thread() = 0;
 #endif
 
-  // Returns the thread that is used for health check of all browser threads.
-  virtual WatchDogThread* watchdog_thread() = 0;
-
   virtual policy::BrowserPolicyConnector* browser_policy_connector() = 0;
 
   virtual IconManager* icon_manager() = 0;
diff --git a/chrome/browser/browser_process_impl.cc b/chrome/browser/browser_process_impl.cc
index 9109974..f65c65b 100644
--- a/chrome/browser/browser_process_impl.cc
+++ b/chrome/browser/browser_process_impl.cc
@@ -32,7 +32,6 @@
 #include "chrome/browser/intranet_redirect_detector.h"
 #include "chrome/browser/io_thread.h"
 #include "chrome/browser/metrics/metrics_service.h"
-#include "chrome/browser/metrics/thread_watcher.h"
 #include "chrome/browser/net/chrome_net_log.h"
 #include "chrome/browser/net/predictor_api.h"
 #include "chrome/browser/net/sdch_dictionary_fetcher.h"
@@ -98,7 +97,6 @@ BrowserProcessImpl::BrowserProcessImpl(const CommandLine& command_line)
       created_db_thread_(false),
       created_process_launcher_thread_(false),
       created_cache_thread_(false),
-      created_watchdog_thread_(false),
       created_profile_manager_(false),
       created_local_state_(false),
       created_icon_manager_(false),
@@ -247,9 +245,6 @@ BrowserProcessImpl::~BrowserProcessImpl() {
   // on the db thread too.
   db_thread_.reset();
 
-  // Stop the watchdog thread after stopping other threads.
-  watchdog_thread_.reset();
-
   // At this point, no render process exist and the file, io, db, and
   // webkit threads in this process have all terminated, so it's safe
   // to access local state data such as cookies, database, or local storage.
@@ -395,14 +390,6 @@ base::Thread* BrowserProcessImpl::background_x11_thread() {
 }
 #endif
 
-WatchDogThread* BrowserProcessImpl::watchdog_thread() {
-  DCHECK(CalledOnValidThread());
-  if (!created_watchdog_thread_)
-    CreateWatchdogThread();
-  DCHECK(watchdog_thread_.get() != NULL);
-  return watchdog_thread_.get();
-}
-
 ProfileManager* BrowserProcessImpl::profile_manager() {
   DCHECK(CalledOnValidThread());
   if (!created_profile_manager_)
@@ -738,18 +725,6 @@ void BrowserProcessImpl::CreateCacheThread() {
   cache_thread_.swap(thread);
 }
 
-void BrowserProcessImpl::CreateWatchdogThread() {
-  DCHECK(!created_watchdog_thread_ && watchdog_thread_.get() == NULL);
-  created_watchdog_thread_ = true;
-
-  scoped_ptr<WatchDogThread> thread(new WatchDogThread());
-  base::Thread::Options options;
-  options.stack_size = 48*1024;
-  if (!thread->StartWithOptions(options))
-    return;
-  watchdog_thread_.swap(thread);
-}
-
 void BrowserProcessImpl::CreateProfileManager() {
   DCHECK(!created_profile_manager_ && profile_manager_.get() == NULL);
   created_profile_manager_ = true;
diff --git a/chrome/browser/browser_process_impl.h b/chrome/browser/browser_process_impl.h
index 162b82a..4926812 100644
--- a/chrome/browser/browser_process_impl.h
+++ b/chrome/browser/browser_process_impl.h
@@ -57,7 +57,6 @@ class BrowserProcessImpl : public BrowserProcess,
 #if defined(USE_X11)
   virtual base::Thread* background_x11_thread();
 #endif
-  virtual WatchDogThread* watchdog_thread();
   virtual ProfileManager* profile_manager();
   virtual PrefService* local_state();
   virtual DevToolsManager* devtools_manager();
@@ -124,7 +123,6 @@ class BrowserProcessImpl : public BrowserProcess,
   void CreateDBThread();
   void CreateProcessLauncherThread();
   void CreateCacheThread();
-  void CreateWatchdogThread();
   void CreateTemplateURLModel();
   void CreateProfileManager();
   void CreateWebDataService();
@@ -172,9 +170,6 @@ class BrowserProcessImpl : public BrowserProcess,
   bool created_cache_thread_;
   scoped_ptr<base::Thread> cache_thread_;
 
-  bool created_watchdog_thread_;
-  scoped_ptr<WatchDogThread> watchdog_thread_;
-
   bool created_profile_manager_;
   scoped_ptr<ProfileManager> profile_manager_;
 
diff --git a/chrome/browser/metrics/thread_watcher.cc b/chrome/browser/metrics/thread_watcher.cc
index 54f8bf9..ececcac 100644
--- a/chrome/browser/metrics/thread_watcher.cc
+++ b/chrome/browser/metrics/thread_watcher.cc
@@ -2,16 +2,14 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#if 0
+
 #include "base/threading/thread_restrictions.h"
 #include "build/build_config.h"
 #include "chrome/browser/metrics/metrics_service.h"
 #include "chrome/browser/metrics/thread_watcher.h"
 #include "chrome/common/notification_service.h"
 
-#if defined(OS_WIN)
-#include <Objbase.h>
-#endif
-
 // static
 const int ThreadWatcher::kPingCount = 3;
 
@@ -44,20 +42,19 @@ void ThreadWatcher::StartWatching(const BrowserThread::ID thread_id,
   DCHECK_GE(sleep_time.InMilliseconds(), 0);
   DCHECK_GE(unresponsive_time.InMilliseconds(), sleep_time.InMilliseconds());
 
-  // If we are not on WatchDogThread, then post a task to call StartWatching on
-  // WatchDogThread.
-  if (!WatchDogThread::CurrentlyOnWatchDogThread()) {
-    MessageLoop* message_loop = WatchDogThread::CurrentMessageLoop();
-    if (message_loop)
-      message_loop->PostTask(
-          FROM_HERE,
-          NewRunnableFunction(
-              &ThreadWatcher::StartWatching,
-              thread_id, thread_name, sleep_time, unresponsive_time));
+  // If we are not on WATCHDOG thread, then post a task to call StartWatching on
+  // WATCHDOG thread.
+  if (!BrowserThread::CurrentlyOn(BrowserThread::WATCHDOG)) {
+    BrowserThread::PostTask(
+        BrowserThread::WATCHDOG,
+        FROM_HERE,
+        NewRunnableFunction(
+            &ThreadWatcher::StartWatching,
+            thread_id, thread_name, sleep_time, unresponsive_time));
     return;
   }
 
-  DCHECK(WatchDogThread::CurrentlyOnWatchDogThread());
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::WATCHDOG));
 
   // Create a new thread watcher object for the given thread and activate it.
   ThreadWatcher* watcher =
@@ -67,7 +64,7 @@ void ThreadWatcher::StartWatching(const BrowserThread::ID thread_id,
 }
 
 void ThreadWatcher::ActivateThreadWatching() {
-  DCHECK(WatchDogThread::CurrentlyOnWatchDogThread());
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::WATCHDOG));
   if (active_) return;
   active_ = true;
   ping_count_ = kPingCount;
@@ -77,14 +74,14 @@ void ThreadWatcher::ActivateThreadWatching() {
 }
 
 void ThreadWatcher::DeActivateThreadWatching() {
-  DCHECK(WatchDogThread::CurrentlyOnWatchDogThread());
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::WATCHDOG));
   active_ = false;
   ping_count_ = 0;
   method_factory_.RevokeAll();
 }
 
 void ThreadWatcher::WakeUp() {
-  DCHECK(WatchDogThread::CurrentlyOnWatchDogThread());
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::WATCHDOG));
   // There is some user activity, PostPingMessage task of thread watcher if
   // needed.
   if (!active_) return;
@@ -98,7 +95,7 @@ void ThreadWatcher::WakeUp() {
 }
 
 void ThreadWatcher::PostPingMessage() {
-  DCHECK(WatchDogThread::CurrentlyOnWatchDogThread());
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::WATCHDOG));
   // If we have stopped watching or if the user is idle, then stop sending
   // ping messages.
   if (!active_ || ping_count_ <= 0)
@@ -129,7 +126,7 @@ void ThreadWatcher::PostPingMessage() {
 }
 
 void ThreadWatcher::OnPongMessage(uint64 ping_sequence_number) {
-  DCHECK(WatchDogThread::CurrentlyOnWatchDogThread());
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::WATCHDOG));
   // Record watched thread's response time.
   base::TimeDelta response_time = base::TimeTicks::Now() - ping_time_;
   histogram_->AddTime(response_time);
@@ -155,7 +152,7 @@ void ThreadWatcher::OnPongMessage(uint64 ping_sequence_number) {
 }
 
 bool ThreadWatcher::OnCheckResponsiveness(uint64 ping_sequence_number) {
-  DCHECK(WatchDogThread::CurrentlyOnWatchDogThread());
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::WATCHDOG));
   // If we have stopped watching then consider thread as responding.
   if (!active_)
     return true;
@@ -181,11 +178,7 @@ void ThreadWatcher::OnPingMessage(const BrowserThread::ID thread_id,
                                   Task* callback_task) {
   // This method is called on watched thread.
   DCHECK(BrowserThread::CurrentlyOn(thread_id));
-  MessageLoop* message_loop = WatchDogThread::CurrentMessageLoop();
-  if (message_loop)
-    message_loop->PostTask(FROM_HERE, callback_task);
-  else
-    delete callback_task;
+  BrowserThread::PostTask(BrowserThread::WATCHDOG, FROM_HERE, callback_task);
 }
 
 //------------------------------------------------------------------------------
@@ -198,15 +191,22 @@ ThreadWatcherList::ThreadWatcherList()
     : last_wakeup_time_(base::TimeTicks::Now()) {
   // Assert we are not running on WATCHDOG thread. Would be ideal to assert we
   // are on UI thread, but Unit tests are not running on UI thread.
-  DCHECK(!WatchDogThread::CurrentlyOnWatchDogThread());
-  CHECK(!global_);
+  DCHECK(!BrowserThread::CurrentlyOn(BrowserThread::WATCHDOG));
+  DCHECK(!global_);
   global_ = this;
   // Register Notifications observer.
+#if defined(OS_WIN)
   MetricsService::SetupNotifications(&registrar_, this);
+#endif
 }
 
 ThreadWatcherList::~ThreadWatcherList() {
   base::AutoLock auto_lock(lock_);
+  while (!registered_.empty()) {
+    RegistrationList::iterator it = registered_.begin();
+    delete it->second;
+    registered_.erase(it->first);
+  }
   DCHECK(this == global_);
   global_ = NULL;
 }
@@ -224,16 +224,15 @@ void ThreadWatcherList::StopWatchingAll() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   if (!global_)
     return;
-
-  // Remove all notifications for all watched threads.
-  RemoveNotifications();
-
-  // Delete all thread watcher objects on WatchDogThread.
-  MessageLoop* message_loop = WatchDogThread::CurrentMessageLoop();
-  if (message_loop)
-    message_loop->PostTask(
+  base::AutoLock auto_lock(global_->lock_);
+  for (RegistrationList::iterator it = global_->registered_.begin();
+       global_->registered_.end() != it;
+       ++it)
+    BrowserThread::PostTask(
+        BrowserThread::WATCHDOG,
         FROM_HERE,
-        NewRunnableMethod(global_, &ThreadWatcherList::DeleteAll));
+        NewRunnableMethod(
+            it->second, &ThreadWatcher::DeActivateThreadWatching));
 }
 
 // static
@@ -241,18 +240,10 @@ void ThreadWatcherList::RemoveNotifications() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   if (!global_)
     return;
+#if defined(OS_WIN)
   base::AutoLock auto_lock(global_->lock_);
   global_->registrar_.RemoveAll();
-}
-
-void ThreadWatcherList::DeleteAll() {
-  DCHECK(WatchDogThread::CurrentlyOnWatchDogThread());
-  base::AutoLock auto_lock(lock_);
-  while (!registered_.empty()) {
-    RegistrationList::iterator it = registered_.begin();
-    delete it->second;
-    registered_.erase(it->first);
-  }
+#endif
 }
 
 void ThreadWatcherList::Observe(NotificationType type,
@@ -269,17 +260,15 @@ void ThreadWatcherList::Observe(NotificationType type,
       last_wakeup_time_ = now;
     }
   }
-  if (need_to_awaken) {
-    MessageLoop* message_loop = WatchDogThread::CurrentMessageLoop();
-    if (message_loop)
-      message_loop->PostTask(
-          FROM_HERE,
-          NewRunnableMethod(this, &ThreadWatcherList::WakeUpAll));
-  }
+  if (need_to_awaken)
+    BrowserThread::PostTask(
+        BrowserThread::WATCHDOG,
+        FROM_HERE,
+        NewRunnableMethod(this, &ThreadWatcherList::WakeUpAll));
 }
 
 void ThreadWatcherList::WakeUpAll() {
-  DCHECK(WatchDogThread::CurrentlyOnWatchDogThread());
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::WATCHDOG));
   base::AutoLock auto_lock(lock_);
   for (RegistrationList::iterator it = global_->registered_.begin();
        global_->registered_.end() != it;
@@ -305,81 +294,42 @@ ThreadWatcher* ThreadWatcherList::PreLockedFind(
 //------------------------------------------------------------------------------
 // WatchDogThread methods and members.
 
-// static
-base::Lock WatchDogThread::lock_;
-// static
-WatchDogThread* WatchDogThread::watchdog_thread_ = NULL;
-
 // The WatchDogThread object must outlive any tasks posted to the IO thread
 // before the Quit task.
 DISABLE_RUNNABLE_METHOD_REFCOUNT(WatchDogThread);
 
-WatchDogThread::WatchDogThread() : Thread("WATCHDOG") {
+WatchDogThread::WatchDogThread()
+    : BrowserProcessSubThread(BrowserThread::WATCHDOG) {
 }
 
 WatchDogThread::~WatchDogThread() {
+  // Remove all notifications for all watched threads.
+  ThreadWatcherList::RemoveNotifications();
   // We cannot rely on our base class to stop the thread since we want our
   // CleanUp function to run.
   Stop();
 }
 
-void WatchDogThread::StartWatchingAll() {
-  const base::TimeDelta kSleepTime = base::TimeDelta::FromSeconds(5);
-  const base::TimeDelta kUnresponsiveTime = base::TimeDelta::FromSeconds(10);
-  if (BrowserThread::IsMessageLoopValid(BrowserThread::UI))
-    ThreadWatcher::StartWatching(BrowserThread::UI, "UI", kSleepTime,
-                                 kUnresponsiveTime);
-  if (BrowserThread::IsMessageLoopValid(BrowserThread::IO))
-    ThreadWatcher::StartWatching(BrowserThread::IO, "IO", kSleepTime,
-                                 kUnresponsiveTime);
-  if (BrowserThread::IsMessageLoopValid(BrowserThread::DB))
-    ThreadWatcher::StartWatching(BrowserThread::DB, "DB", kSleepTime,
-                                 kUnresponsiveTime);
-  if (BrowserThread::IsMessageLoopValid(BrowserThread::FILE))
-    ThreadWatcher::StartWatching(BrowserThread::FILE, "FILE", kSleepTime,
-                                 kUnresponsiveTime);
-  if (BrowserThread::IsMessageLoopValid(BrowserThread::CACHE))
-    ThreadWatcher::StartWatching(BrowserThread::CACHE, "CACHE", kSleepTime,
-                                 kUnresponsiveTime);
-}
-
-// static
-MessageLoop* WatchDogThread::CurrentMessageLoop() {
-  base::AutoLock lock(lock_);
-  MessageLoop* message_loop = watchdog_thread_ ?
-      watchdog_thread_->message_loop() : NULL;
-  return message_loop;
-}
-
-// static
-bool WatchDogThread::CurrentlyOnWatchDogThread() {
-  base::AutoLock lock(lock_);
-  return watchdog_thread_ &&
-    watchdog_thread_->message_loop() == MessageLoop::current();
-}
-
 void WatchDogThread::Init() {
   // This thread shouldn't be allowed to perform any blocking disk I/O.
   base::ThreadRestrictions::SetIOAllowed(false);
 
-#if defined(OS_WIN)
-  // Initializes the COM library on the current thread.
-  CoInitialize(NULL);
-#endif
-
-  base::AutoLock lock(lock_);
-  watchdog_thread_ = this;
-}
-
-void WatchDogThread::CleanUp() {
-  base::AutoLock lock(lock_);
-  watchdog_thread_ = NULL;
-}
+  BrowserProcessSubThread::Init();
 
-void WatchDogThread::CleanUpAfterMessageLoopDestruction() {
 #if defined(OS_WIN)
-  // Closes the COM library on the current thread. CoInitialize must
-  // be balanced by a corresponding call to CoUninitialize.
-  CoUninitialize();
+  const base::TimeDelta kSleepTime = base::TimeDelta::FromSeconds(5);
+  const base::TimeDelta kUnresponsiveTime = base::TimeDelta::FromSeconds(10);
+  ThreadWatcher::StartWatching(BrowserThread::UI, "UI", kSleepTime,
+                               kUnresponsiveTime);
+  ThreadWatcher::StartWatching(BrowserThread::IO, "IO", kSleepTime,
+                               kUnresponsiveTime);
+  ThreadWatcher::StartWatching(BrowserThread::DB, "DB", kSleepTime,
+                               kUnresponsiveTime);
+  ThreadWatcher::StartWatching(BrowserThread::FILE, "FILE", kSleepTime,
+                               kUnresponsiveTime);
+  ThreadWatcher::StartWatching(BrowserThread::CACHE, "CACHE", kSleepTime,
+                               kUnresponsiveTime);
 #endif
 }
+
+#endif  // 0
diff --git a/chrome/browser/metrics/thread_watcher.h b/chrome/browser/metrics/thread_watcher.h
index 2811fc0..f7765c8 100644
--- a/chrome/browser/metrics/thread_watcher.h
+++ b/chrome/browser/metrics/thread_watcher.h
@@ -8,13 +8,14 @@
 #define CHROME_BROWSER_METRICS_THREAD_WATCHER_H_
 #pragma once
 
+#if 0
+
 #include <map>
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/gtest_prod_util.h"
-#include "base/message_loop.h"
 #include "base/metrics/histogram.h"
 #include "base/ref_counted.h"
 #include "base/scoped_ptr.h"
@@ -22,6 +23,7 @@
 #include "base/task.h"
 #include "base/threading/thread.h"
 #include "base/time.h"
+#include "chrome/browser/browser_process_sub_thread.h"
 #include "chrome/browser/browser_thread.h"
 #include "chrome/common/notification_observer.h"
 #include "chrome/common/notification_registrar.h"
@@ -105,21 +107,21 @@ class ThreadWatcher {
   // OnPongMessage. It also posts a task (OnCheckResponsiveness) to check
   // responsiveness of monitored thread that would be called after waiting
   // unresponsive_time_.
-  // This method is accessible on WatchDogThread.
+  // This method is accessible on WATCHDOG thread.
   virtual void PostPingMessage();
 
   // This method handles a Pong Message from watched thread. It will track the
   // response time (pong time minus ping time) via histograms. It posts a
   // PostPingMessage task that would be called after waiting sleep_time_.  It
   // increments ping_sequence_number_ by 1.
-  // This method is accessible on WatchDogThread.
+  // This method is accessible on WATCHDOG thread.
   virtual void OnPongMessage(uint64 ping_sequence_number);
 
   // This method will determine if the watched thread is responsive or not. If
   // the latest ping_sequence_number_ is not same as the ping_sequence_number
   // that is passed in, then we can assume that watched thread has responded
   // with a pong message.
-  // This method is accessible on WatchDogThread.
+  // This method is accessible on WATCHDOG thread.
   virtual bool OnCheckResponsiveness(uint64 ping_sequence_number);
 
  private:
@@ -201,36 +203,28 @@ class ThreadWatcherList : public NotificationObserver {
   // Register() stores a pointer to the given ThreadWatcher in a global map.
   static void Register(ThreadWatcher* watcher);
 
-  // This method posts a task on WatchDogThread to RevokeAll tasks and to
+  // This method posts a task on WATCHDOG thread to RevokeAll tasks and to
   // deactive thread watching of other threads and tell NotificationService to
   // stop calling Observe.
-  // This method is accessible on UI thread.
   static void StopWatchingAll();
 
   // RemoveAll NotificationTypes that are being observed.
-  // This method is accessible on UI thread.
   static void RemoveNotifications();
 
  private:
   // Allow tests to access our innards for testing purposes.
   FRIEND_TEST(ThreadWatcherTest, Registration);
 
-  // Delete all thread watcher objects and remove them from global map.
-  // This method is accessible on WatchDogThread.
-  void DeleteAll();
-
   // This will ensure that the watching is actively taking place. It will wakeup
   // all thread watchers every 2 seconds. This is the implementation of
   // NotificationObserver. When a matching notification is posted to the
   // notification service, this method is called.
-  // This method is accessible on UI thread.
   virtual void Observe(NotificationType type,
                        const NotificationSource& source,
                        const NotificationDetails& details);
 
   // This will ensure that the watching is actively taking place, and awaken
   // all thread watchers that are registered.
-  // This method is accessible on WatchDogThread.
   virtual void WakeUpAll();
 
   // The Find() method can be used to test to see if a given ThreadWatcher was
@@ -256,41 +250,23 @@ class ThreadWatcherList : public NotificationObserver {
 };
 
 //------------------------------------------------------------------------------
-// Class for WatchDogThread and in its Init method, we start watching UI, IO,
+// Class for WATCHDOG thread and in its Init method, we start watching UI, IO,
 // DB, FILE, CACHED threads.
-class WatchDogThread : public base::Thread {
+class WatchDogThread : public BrowserProcessSubThread {
  public:
-  // Constructor.
   WatchDogThread();
-
-  // Destroys the thread and stops the thread.
   virtual ~WatchDogThread();
 
-  // Start watching all browser threads.
-  static void StartWatchingAll();
-
-  // Returns current message loop for the watchdog thread. This only returns
-  // non-null after a successful call to Start.  After Stop has been called,
-  // this will return NULL.
-  static MessageLoop* CurrentMessageLoop();
-
-  // Callable on any thread.  Returns whether you're currently on a
-  // watchdog_thread_.
-  static bool CurrentlyOnWatchDogThread();
-
  protected:
   virtual void Init();
-  virtual void CleanUp();
-  virtual void CleanUpAfterMessageLoopDestruction();
-
-  // This lock protects watchdog_thread_.
-  static base::Lock lock_;
-  static WatchDogThread* watchdog_thread_;
 
   DISALLOW_COPY_AND_ASSIGN(WatchDogThread);
 };
 
+
 DISABLE_RUNNABLE_METHOD_REFCOUNT(ThreadWatcher);
 DISABLE_RUNNABLE_METHOD_REFCOUNT(ThreadWatcherList);
 
+#endif  // 0
+
 #endif  // CHROME_BROWSER_METRICS_THREAD_WATCHER_H_
diff --git a/chrome/browser/metrics/thread_watcher_unittest.cc b/chrome/browser/metrics/thread_watcher_unittest.cc
index 0082d08..52a4288 100644
--- a/chrome/browser/metrics/thread_watcher_unittest.cc
+++ b/chrome/browser/metrics/thread_watcher_unittest.cc
@@ -2,6 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#if 0
+
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
@@ -11,7 +13,6 @@
 #include "base/synchronization/lock.h"
 #include "base/threading/platform_thread.h"
 #include "base/time.h"
-#include "build/build_config.h"
 #include "chrome/browser/metrics/thread_watcher.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "testing/platform_test.h"
@@ -145,7 +146,7 @@ class CustomThreadWatcher : public ThreadWatcher {
   }
 
   void VeryLongMethod(TimeDelta wait_time) {
-    DCHECK(!WatchDogThread::CurrentlyOnWatchDogThread());
+    DCHECK(!BrowserThread::CurrentlyOn(BrowserThread::WATCHDOG));
     TimeTicks end_time = TimeTicks::Now() + wait_time;
     {
       base::AutoLock auto_lock(custom_lock_);
@@ -160,7 +161,7 @@ class CustomThreadWatcher : public ThreadWatcher {
   }
 
   State WaitForStateChange(const TimeDelta& wait_time, State expected_state) {
-    DCHECK(!WatchDogThread::CurrentlyOnWatchDogThread());
+    DCHECK(!BrowserThread::CurrentlyOn(BrowserThread::WATCHDOG));
     UpdateWaitState(STARTED_WAITING);
 
     State exit_state;
@@ -195,7 +196,7 @@ class CustomThreadWatcher : public ThreadWatcher {
 
   CheckResponseState WaitForCheckResponse(const TimeDelta& wait_time,
                                           CheckResponseState expected_state) {
-    DCHECK(!WatchDogThread::CurrentlyOnWatchDogThread());
+    DCHECK(!BrowserThread::CurrentlyOn(BrowserThread::WATCHDOG));
     UpdateWaitState(STARTED_WAITING);
 
     CheckResponseState exit_state;
@@ -244,9 +245,13 @@ class ThreadWatcherTest : public ::testing::Test {
   CustomThreadWatcher* webkit_watcher_;
 
   ThreadWatcherTest() {
+  }
+
+ protected:
+  virtual void SetUp() {
     webkit_thread_.reset(new BrowserThread(BrowserThread::WEBKIT));
     io_thread_.reset(new BrowserThread(BrowserThread::IO));
-    watchdog_thread_.reset(new WatchDogThread());
+    watchdog_thread_.reset(new BrowserThread(BrowserThread::WATCHDOG));
     webkit_thread_->Start();
     io_thread_->Start();
     watchdog_thread_->Start();
@@ -263,7 +268,7 @@ class ThreadWatcherTest : public ::testing::Test {
         webkit_thread_id, webkit_thread_name, kSleepTime, kUnresponsiveTime);
   }
 
-  ~ThreadWatcherTest() {
+  virtual void TearDown() {
     // io_thread_->Stop();
     // webkit_thread_->Stop();
     // watchdog_thread_->Stop();
@@ -276,7 +281,7 @@ class ThreadWatcherTest : public ::testing::Test {
  private:
   scoped_ptr<BrowserThread> webkit_thread_;
   scoped_ptr<BrowserThread> io_thread_;
-  scoped_ptr<WatchDogThread> watchdog_thread_;
+  scoped_ptr<BrowserThread> watchdog_thread_;
   ThreadWatcherList* thread_watcher_list_;
 };
 
@@ -314,15 +319,14 @@ TEST_F(ThreadWatcherTest, Registration) {
 
 // Test ActivateThreadWatching and DeActivateThreadWatching of IO thread. This
 // method also checks that pong message was sent by the watched thread and pong
-// message was received by the WatchDogThread. It also checks that
+// message was received by the WATCHDOG thread. It also checks that
 // OnCheckResponsiveness has verified the ping-pong mechanism and the watched
 // thread is not hung.
 TEST_F(ThreadWatcherTest, ThreadResponding) {
   TimeTicks time_before_ping = TimeTicks::Now();
   // Activate watching IO thread.
-  MessageLoop* message_loop = WatchDogThread::CurrentMessageLoop();
-  ASSERT_TRUE(message_loop != NULL);
-  message_loop->PostTask(
+  BrowserThread::PostTask(
+      BrowserThread::WATCHDOG,
       FROM_HERE,
       NewRunnableMethod(io_watcher_, &ThreadWatcher::ActivateThreadWatching));
 
@@ -345,9 +349,8 @@ TEST_F(ThreadWatcherTest, ThreadResponding) {
   EXPECT_EQ(io_watcher_->failed_response_, static_cast<uint64>(0));
 
   // DeActivate thread watching for shutdown.
-  message_loop = WatchDogThread::CurrentMessageLoop();
-  ASSERT_TRUE(message_loop != NULL);
-  message_loop->PostTask(
+  BrowserThread::PostTask(
+      BrowserThread::WATCHDOG,
       FROM_HERE,
       NewRunnableMethod(io_watcher_, &ThreadWatcher::DeActivateThreadWatching));
 }
@@ -368,9 +371,8 @@ TEST_F(ThreadWatcherTest, ThreadNotResponding) {
           kUnresponsiveTime * 10));
 
   // Activate thread watching.
-  MessageLoop* message_loop = WatchDogThread::CurrentMessageLoop();
-  ASSERT_TRUE(message_loop != NULL);
-  message_loop->PostTask(
+  BrowserThread::PostTask(
+      BrowserThread::WATCHDOG,
       FROM_HERE,
       NewRunnableMethod(io_watcher_, &ThreadWatcher::ActivateThreadWatching));
 
@@ -381,9 +383,8 @@ TEST_F(ThreadWatcherTest, ThreadNotResponding) {
   EXPECT_GT(io_watcher_->failed_response_, static_cast<uint64>(0));
 
   // DeActivate thread watching for shutdown.
-  message_loop = WatchDogThread::CurrentMessageLoop();
-  ASSERT_TRUE(message_loop != NULL);
-  message_loop->PostTask(
+  BrowserThread::PostTask(
+      BrowserThread::WATCHDOG,
       FROM_HERE,
       NewRunnableMethod(io_watcher_, &ThreadWatcher::DeActivateThreadWatching));
 }
@@ -391,15 +392,14 @@ TEST_F(ThreadWatcherTest, ThreadNotResponding) {
 // Test watching of multiple threads with all threads not responding.
 TEST_F(ThreadWatcherTest, MultipleThreadsResponding) {
   // Check for WEBKIT thread to perform ping/pong messaging.
-  MessageLoop* message_loop = WatchDogThread::CurrentMessageLoop();
-  ASSERT_TRUE(message_loop != NULL);
-  message_loop->PostTask(
+  BrowserThread::PostTask(
+      BrowserThread::WATCHDOG,
       FROM_HERE,
       NewRunnableMethod(
           webkit_watcher_, &ThreadWatcher::ActivateThreadWatching));
-
   // Check for IO thread to perform ping/pong messaging.
-  message_loop->PostTask(
+  BrowserThread::PostTask(
+      BrowserThread::WATCHDOG,
       FROM_HERE,
       NewRunnableMethod(io_watcher_, &ThreadWatcher::ActivateThreadWatching));
 
@@ -422,14 +422,12 @@ TEST_F(ThreadWatcherTest, MultipleThreadsResponding) {
   EXPECT_EQ(io_watcher_->failed_response_, static_cast<uint64>(0));
 
   // DeActivate thread watching for shutdown.
-  message_loop = WatchDogThread::CurrentMessageLoop();
-  ASSERT_TRUE(message_loop != NULL);
-  message_loop->PostTask(
+  BrowserThread::PostTask(
+      BrowserThread::WATCHDOG,
       FROM_HERE,
-      NewRunnableMethod(
-          io_watcher_, &ThreadWatcher::DeActivateThreadWatching));
-
-  message_loop->PostTask(
+      NewRunnableMethod(io_watcher_, &ThreadWatcher::DeActivateThreadWatching));
+  BrowserThread::PostTask(
+      BrowserThread::WATCHDOG,
       FROM_HERE,
       NewRunnableMethod(
           webkit_watcher_, &ThreadWatcher::DeActivateThreadWatching));
@@ -448,15 +446,15 @@ TEST_F(ThreadWatcherTest, MultipleThreadsNotResponding) {
           kUnresponsiveTime * 10));
 
   // Activate watching of WEBKIT thread.
-  MessageLoop* message_loop = WatchDogThread::CurrentMessageLoop();
-  ASSERT_TRUE(message_loop != NULL);
-  message_loop->PostTask(
+  BrowserThread::PostTask(
+      BrowserThread::WATCHDOG,
       FROM_HERE,
       NewRunnableMethod(
           webkit_watcher_, &ThreadWatcher::ActivateThreadWatching));
 
   // Activate watching of IO thread.
-  message_loop->PostTask(
+  BrowserThread::PostTask(
+      BrowserThread::WATCHDOG,
       FROM_HERE,
       NewRunnableMethod(io_watcher_, &ThreadWatcher::ActivateThreadWatching));
 
@@ -473,13 +471,15 @@ TEST_F(ThreadWatcherTest, MultipleThreadsNotResponding) {
   EXPECT_GT(io_watcher_->failed_response_, static_cast<uint64>(0));
 
   // DeActivate thread watching for shutdown.
-  message_loop = WatchDogThread::CurrentMessageLoop();
-  ASSERT_TRUE(message_loop != NULL);
-  message_loop->PostTask(
+  BrowserThread::PostTask(
+      BrowserThread::WATCHDOG,
       FROM_HERE,
       NewRunnableMethod(io_watcher_, &ThreadWatcher::DeActivateThreadWatching));
-  message_loop->PostTask(
+  BrowserThread::PostTask(
+      BrowserThread::WATCHDOG,
       FROM_HERE,
       NewRunnableMethod(
           webkit_watcher_, &ThreadWatcher::DeActivateThreadWatching));
 }
+
+#endif  // 0
diff --git a/chrome/chrome_browser.gypi b/chrome/chrome_browser.gypi
index a7ad7c4..2ecd6bc 100644
--- a/chrome/chrome_browser.gypi
+++ b/chrome/chrome_browser.gypi
@@ -1356,8 +1356,6 @@
         'browser/metrics/metrics_response.h',
         'browser/metrics/metrics_service.cc',
         'browser/metrics/metrics_service.h',
-        'browser/metrics/thread_watcher.cc',
-        'browser/metrics/thread_watcher.h',
         'browser/metrics/user_metrics.cc',
         'browser/metrics/user_metrics.h',
         'browser/nacl_host/nacl_broker_host_win.cc',
diff --git a/chrome/chrome_tests.gypi b/chrome/chrome_tests.gypi
index 4c39a00..613ec71 100644
--- a/chrome/chrome_tests.gypi
+++ b/chrome/chrome_tests.gypi
@@ -1367,7 +1367,6 @@
         'browser/metrics/metrics_log_unittest.cc',
         'browser/metrics/metrics_response_unittest.cc',
         'browser/metrics/metrics_service_unittest.cc',
-        'browser/metrics/thread_watcher_unittest.cc',
         'browser/mock_plugin_exceptions_table_model.cc',
         'browser/mock_plugin_exceptions_table_model.h',
         'browser/net/connection_tester_unittest.cc',
diff --git a/chrome/test/testing_browser_process.cc b/chrome/test/testing_browser_process.cc
index 71561c3..f6af58f 100644
--- a/chrome/test/testing_browser_process.cc
+++ b/chrome/test/testing_browser_process.cc
@@ -58,10 +58,6 @@ base::Thread* TestingBrowserProcess::cache_thread() {
   return NULL;
 }
 
-WatchDogThread* TestingBrowserProcess::watchdog_thread() {
-  return NULL;
-}
-
 ProfileManager* TestingBrowserProcess::profile_manager() {
   return profile_manager_.get();
 }
diff --git a/chrome/test/testing_browser_process.h b/chrome/test/testing_browser_process.h
index 41e5b2c..d544f82 100644
--- a/chrome/test/testing_browser_process.h
+++ b/chrome/test/testing_browser_process.h
@@ -22,7 +22,6 @@
 class IOThread;
 class GoogleURLTracker;
 class PrefService;
-class WatchDogThread;
 
 namespace base {
 class WaitableEvent;
@@ -59,8 +58,6 @@ class TestingBrowserProcess : public BrowserProcess {
 
   virtual base::Thread* cache_thread();
 
-  virtual WatchDogThread* watchdog_thread();
-
   virtual ProfileManager* profile_manager();
 
   virtual PrefService* local_state();