WebUI TaskManager: make it default on touch UI

WebUI TaskManager: make it default on touch UI

BUG=chromium-os:13885
TEST=manually on chromium-os

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=93363

[mazda, 2011-07-19 10:05:07]

http://codereview.chromium.org/7436001/diff/1/chrome/browser/ui/views/frame/b...
File chrome/browser/ui/views/frame/browser_view.cc (right):

http://codereview.chromium.org/7436001/diff/1/chrome/browser/ui/views/frame/b...
chrome/browser/ui/views/frame/browser_view.cc:1055: #if !defined(TOUCH_UI)
How about enclosing the block with one if-else clause?

#if defined(TOUCH_UI)
   TaskManagerDialog::Show();
#else
   if (CommandLine::ForCurrentProcess()
         ->HasSwitch(switches::kEnableWebUITaskManager)) {
     TaskManagerDialog::Show();
   } else {
     browser::ShowTaskManager();
   }
#endif  // defined(TOUCH_UI)

[yoshiki, 2011-07-20 07:38:59]

http://codereview.chromium.org/7436001/diff/1/chrome/browser/ui/views/frame/b...
File chrome/browser/ui/views/frame/browser_view.cc (right):

http://codereview.chromium.org/7436001/diff/1/chrome/browser/ui/views/frame/b...
chrome/browser/ui/views/frame/browser_view.cc:1055: #if !defined(TOUCH_UI)
On 2011/07/19 10:05:07, mazda wrote:
> How about enclosing the block with one if-else clause?
> 
> #if defined(TOUCH_UI)
>    TaskManagerDialog::Show();
> #else
>    if (CommandLine::ForCurrentProcess()
>          ->HasSwitch(switches::kEnableWebUITaskManager)) {
>      TaskManagerDialog::Show();
>    } else {
>      browser::ShowTaskManager();
>    }
> #endif  // defined(TOUCH_UI)

Done.

[mazda, 2011-07-20 12:04:54]

LGTM