Revert 255322 "ThreadWatcher: fixes Start/StopWatchingAll."

> ThreadWatcher: fixes Start/StopWatchingAll.
> 
> Start is deferred in relation to Stop, and when they're both called
> within the Start interval, the end result was started.
> 
> BUG=347887
> TEST=unit_tests --gtest_filter=ThreadWatcherList*
> 
> Review URL: https://codereview.chromium.org/183063007

BUG=347887,349987
TBR=bulach@chromium.org

Review URL: https://codereview.chromium.org/188643006

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

1 files changed, 0 insertions, 21 deletions

diff --git a/chrome/browser/metrics/thread_watcher.cc b/chrome/browser/metrics/thread_watcher.cc
index 8246b3e..1737593 100644
--- a/chrome/browser/metrics/thread_watcher.cc
+++ b/chrome/browser/metrics/thread_watcher.cc
@@ -409,8 +409,6 @@ bool ThreadWatcher::IsVeryUnresponsive() {
 // static
 ThreadWatcherList* ThreadWatcherList::g_thread_watcher_list_ = NULL;
 // static
-bool ThreadWatcherList::g_stopped_ = false;
-// static
 const int ThreadWatcherList::kSleepSeconds = 1;
 // static
 const int ThreadWatcherList::kUnresponsiveSeconds = 2;
@@ -445,10 +443,6 @@ void ThreadWatcherList::StartWatchingAll(const CommandLine& command_line) {
   ThreadWatcherObserver::SetupNotifications(
       base::TimeDelta::FromSeconds(kSleepSeconds * unresponsive_threshold));
 
-  WatchDogThread::PostTask(
-      FROM_HERE,
-      base::Bind(&ThreadWatcherList::SetStopped, false));
-
   WatchDogThread::PostDelayedTask(
       FROM_HERE,
       base::Bind(&ThreadWatcherList::InitializeAndStartWatching,
@@ -641,12 +635,6 @@ void ThreadWatcherList::InitializeAndStartWatching(
     const CrashOnHangThreadMap& crash_on_hang_threads) {
   DCHECK(WatchDogThread::CurrentlyOnWatchDogThread());
 
-  // This method is deferred in relationship to its StopWatchingAll()
-  // counterpart. If a previous initialization has already happened, or if
-  // stop has been called, there's nothing left to do here.
-  if (g_thread_watcher_list_ || g_stopped_)
-    return;
-
   ThreadWatcherList* thread_watcher_list = new ThreadWatcherList();
   CHECK(thread_watcher_list);
 
@@ -712,9 +700,6 @@ void ThreadWatcherList::DeleteAll() {
   }
 
   DCHECK(WatchDogThread::CurrentlyOnWatchDogThread());
-
-  SetStopped(true);
-
   if (!g_thread_watcher_list_)
     return;
 
@@ -740,12 +725,6 @@ ThreadWatcher* ThreadWatcherList::Find(const BrowserThread::ID& thread_id) {
   return it->second;
 }
 
-// static
-void ThreadWatcherList::SetStopped(bool stopped) {
-  DCHECK(WatchDogThread::CurrentlyOnWatchDogThread());
-  g_stopped_ = stopped;
-}
-
 // ThreadWatcherObserver methods and members.
 //
 // static