Turn off Task Manager should not break the build on Linux

chrome/browser/ui/browser_commands.cc

Index: chrome/browser/ui/browser_commands.cc
diff --git a/chrome/browser/ui/browser_commands.cc b/chrome/browser/ui/browser_commands.cc
index de82e31324840c45892150c346eb4fc338e64f30..6b9b16d9fedea1309f06c082a11b9e56358a8c3e 100644
--- a/chrome/browser/ui/browser_commands.cc
+++ b/chrome/browser/ui/browser_commands.cc
@@ -1009,6 +1009,10 @@ void ToggleDevToolsWindow(Browser* browser, DevToolsToggleAction action) {
 }
 
 bool CanOpenTaskManager() {
+#if !defined(ENABLE_TASK_MANAGER)
+  return false;
+#endif  // defined(ENABLE_TASK_MANAGER)
+
 #if defined(OS_WIN)
   // In metro we can't display the task manager, as it is a native window.
   return !win8::IsSingleWindowMetroMode();
@@ -1018,8 +1022,10 @@ bool CanOpenTaskManager() {
 }
 
 void OpenTaskManager(Browser* browser) {
+#if defined(ENABLE_TASK_MANAGER)
   content::RecordAction(UserMetricsAction("TaskManager"));
   chrome::ShowTaskManager(browser);
+#endif  // defined(ENABLE_TASK_MANAGER)

[miket_OOO, 2014/03/24 23:23:26]

I can't tell for sure from here, but is this better as an #else NOT_REACHED()?
You should be able to inspect the code and determine that if
CanOpenTaskManager() is hard-coded false, then no code should ever call
OpenTaskManager().

 }
 
 void OpenFeedbackDialog(Browser* browser) {