Fix GetChannel interface misuse in ThreadWatcher and ShutdownWatcher.

chrome/browser/metrics/thread_watcher.cc

 // Copyright (c) 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 "chrome/browser/metrics/thread_watcher.h"
 
 #include <math.h>  // ceil
 
 #include "base/debug/alias.h"
 #include "base/string_tokenizer.h"
@@ skipping 405 matching lines @@
 
 // static
 void ThreadWatcherList::ParseCommandLine(
     const CommandLine& command_line,
     uint32* unresponsive_threshold,
     std::set<std::string>* crash_on_hang_thread_names,
     uint32* live_threads_threshold) {
   // Determine |unresponsive_threshold| based on switches::kCrashOnHangSeconds.
   *unresponsive_threshold = kUnresponsiveCount;
 
-  if (chrome::VersionInfo::GetChannel() == chrome::VersionInfo::CHANNEL_BETA) {
-    // Increase the unresponsive_threshold in Beta channel to reduce the number
-    // of crashes due to ThreadWatcher.
+  // Increase the unresponsive_threshold on the Stable and Beta channels to
+  // reduce the number of crashes due to ThreadWatcher.
+  chrome::VersionInfo::Channel channel = chrome::VersionInfo::GetChannel();
+  if (channel == chrome::VersionInfo::CHANNEL_STABLE) {
+    *unresponsive_threshold *= 4;
+  } else if (channel == chrome::VersionInfo::CHANNEL_BETA) {
     *unresponsive_threshold *= 2;
-  } else {
-    // In Canary and Dev channels, for Windows XP (old systems), double the
-    // unresponsive_threshold to give OS a chance to schedule UI/IO threads a
-    // time slice to respond with a pong message (to get around limitations with
-    // the OS).
+  }
+
 #if defined(OS_WIN)
-    if (base::win::GetVersion() <= base::win::VERSION_XP)
-      *unresponsive_threshold *= 2;
+  // For Windows XP (old systems), double the unresponsive_threshold to give
+  // the OS a chance to schedule UI/IO threads a time slice to respond with a
+  // pong message (to get around limitations with the OS).
+  if (base::win::GetVersion() <= base::win::VERSION_XP)
+    *unresponsive_threshold *= 2;
 #endif
-  }
 
   std::string crash_on_hang_seconds =
       command_line.GetSwitchValueASCII(switches::kCrashOnHangSeconds);
   if (!crash_on_hang_seconds.empty()) {
     int crash_seconds = atoi(crash_on_hang_seconds.c_str());
     if (crash_seconds > 0) {
       *unresponsive_threshold = static_cast<uint32>(
           ceil(static_cast<float>(crash_seconds) / kUnresponsiveSeconds));
     }
   }
 
   std::string crash_on_hang_threads;
 
   // Default to crashing the browser if UI or IO threads are not responsive
   // except in stable channel.
-  if (chrome::VersionInfo::GetChannel() == chrome::VersionInfo::CHANNEL_STABLE)
+  if (channel == chrome::VersionInfo::CHANNEL_STABLE)
     crash_on_hang_threads = "";
   else
     crash_on_hang_threads = "UI,IO";
 
   if (command_line.HasSwitch(switches::kCrashOnHangThreads)) {
     crash_on_hang_threads =
         command_line.GetSwitchValueASCII(switches::kCrashOnHangThreads);
   }
   StringTokenizer tokens(crash_on_hang_threads, ",");
   while (tokens.GetNext())
@@ skipping 255 matching lines @@
   if (shutdown_watchdog_) {
     shutdown_watchdog_->Disarm();
     delete shutdown_watchdog_;
     shutdown_watchdog_ = NULL;
   }
 }
 
 void ShutdownWatcherHelper::Arm(const base::TimeDelta& duration) {
   DCHECK(!shutdown_watchdog_);
   base::TimeDelta actual_duration = duration;
+
   chrome::VersionInfo::Channel channel = chrome::VersionInfo::GetChannel();
   if (channel == chrome::VersionInfo::CHANNEL_STABLE) {
     actual_duration *= 50;
   } else if (channel == chrome::VersionInfo::CHANNEL_BETA ||
              channel == chrome::VersionInfo::CHANNEL_DEV) {
     actual_duration *= 25;
-  } else {
-    // In Canary, for Windows XP, give twice the time for shutdown.
+  }
+
 #if defined(OS_WIN)
-    if (base::win::GetVersion() <= base::win::VERSION_XP)
-      actual_duration *= 2;
+  // On Windows XP, give twice the time for shutdown.
+  if (base::win::GetVersion() <= base::win::VERSION_XP)
+    actual_duration *= 2;
 #endif
-  }
+
   shutdown_watchdog_ = new ShutdownWatchDogThread(actual_duration);
   shutdown_watchdog_->Arm();
 }