Remove toolbar from extension windows in streamlined hosted apps mode.

chrome/browser/ui/browser.cc

Index: chrome/browser/ui/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index e56a0b649a7becb90cd34a10932025b0498bc5f9..04a6f0c3d43def8e5b57f3d19dc41fcfca330878 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -289,6 +289,7 @@ Browser::CreateParams Browser::CreateParams::CreateForDevTools(
     chrome::HostDesktopType host_desktop_type) {
   CreateParams params(TYPE_POPUP, profile, host_desktop_type);
   params.app_name = DevToolsWindow::kDevToolsApp;
+  params.app_type = APP_TYPE_CHILD;

[sky, 2013/12/12 17:03:35]

Why are you changing the app_type? I have to admit I don't understand the
subtleties between the two, but APP_TYPE_HOST seems to better match dev tools.

[calamity, 2013/12/13 02:02:54]

My bad. Done.

   return params;
 }
 
@@ -2170,8 +2171,9 @@ bool Browser::SupportsWindowFeatureImpl(WindowFeature feature,
     if (is_type_tabbed())
       features |= FEATURE_TOOLBAR;
 
-    if (!is_app() || CommandLine::ForCurrentProcess()->HasSwitch(
-                         switches::kEnableStreamlinedHostedApps))
+    if (!is_app() || (app_type() == APP_TYPE_HOST &&

[sky, 2013/12/12 17:03:35]

Wouldn't it be better to special case devtools rather than look at app_type?

[calamity, 2013/12/13 02:02:54]

Done.

+                      CommandLine::ForCurrentProcess()->HasSwitch(
+                          switches::kEnableStreamlinedHostedApps)))
       features |= FEATURE_LOCATIONBAR;
   }
   return !!(features & feature);