Not opening File Manager as an app on Ctrl+O.

chrome/browser/ui/ash/chrome_shell_delegate.cc

 // Copyright (c) 2012 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/ui/ash/chrome_shell_delegate.h"
 
 #include "ash/launcher/launcher_types.h"
 #include "ash/system/tray/system_tray_delegate.h"
 #include "ash/wm/window_util.h"
 #include "base/command_line.h"
@@ skipping 179 matching lines @@
     return;
   // Get out of fullscreen when in fullscreen mode.
   if (ash::wm::IsWindowFullscreen(window)) {
     chrome::ToggleFullscreenMode(GetTargetBrowser());
     return;
   }
   ash::wm::ToggleMaximizedWindow(window);
 }
 
 void ChromeShellDelegate::OpenFileManager(bool as_dialog) {
 #if defined(OS_CHROMEOS)

[sky, 2012/11/06 16:46:58]

Why is any of this in an ifdef?

[SeRya, 2012/11/07 07:27:23]

Because file_manager_util is available in ChromeOS but not in Ash on Windows for
instance. May be it's wrong but it's a separate issue.

   if (as_dialog) {
     Browser* browser =
         browser::FindBrowserWithWindow(ash::wm::GetActiveWindow());
     // Open the select file dialog only if there is an active browser where the
-    // selected file is displayed. Otherwise open a file manager in a tab.
+    // selected file is displayed.
     if (browser) {
       browser->OpenFile();
       return;
     }
+  } else {
+    file_manager_util::OpenApplication();
   }
-  file_manager_util::OpenApplication();
 #endif
 }
 
 void ChromeShellDelegate::OpenCrosh() {
 #if defined(OS_CHROMEOS)
   Browser* browser = GetTargetBrowser();
   GURL crosh_url = TerminalExtensionHelper::GetCroshExtensionURL(
       browser->profile());
   if (!crosh_url.is_valid())
     return;
@@ skipping 259 matching lines @@
       ash::Shell::GetInstance()->ShowLauncher();
       break;
     default:
       NOTREACHED() << "Unexpected notification " << type;
   }
 #else
   // MSVC++ warns about switch statements without any cases.
   NOTREACHED() << "Unexpected notification " << type;
 #endif
 }