Address bar is shown on pop-out extension windows.

chrome/browser/extensions/window_open_apitest.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 "base/command_line.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/test/ui_test_utils.h"
@@ skipping 31 matching lines @@
         continue;
 
       // Check for TYPE_POPUP or TYPE_PANEL.
       ASSERT_TRUE((*iter)->is_type_popup() || (*iter)->is_type_panel());
     }
 
     break;
   }
 }
 
+IN_PROC_BROWSER_TEST_F(ExtensionApiTest, BrowserIsApp) {

[Erik does not do reviews, 2011/05/25 15:13:03]

Could you add a test where the app opens a popup to a random external URL as
well?  I don't have a strong opinion about what the behavior should be other
than it match the behavior prior to the regression you're fixing with this CL.

[prasadt, 2011/05/25 17:53:00]

Done.

+  ASSERT_TRUE(LoadExtension(
+      test_data_dir_.AppendASCII("window_open").AppendASCII("browser_is_app")));
+
+  WaitForTabsAndPopups(browser(), 0, 1);
+
+  BrowserList::const_iterator iter = BrowserList::begin();
+  if (browser() == *iter)
+    ++iter;
+
+  ASSERT_FALSE(browser()->is_app());
+  ASSERT_TRUE((*iter)->is_app());
+}
+
 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, PopupBlockingExtension) {
   host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(StartTestServer());
 
   ASSERT_TRUE(LoadExtension(
       test_data_dir_.AppendASCII("window_open").AppendASCII("popup_blocking")
       .AppendASCII("extension")));
 
   WaitForTabsAndPopups(browser(), 5, 3);
 }
@@ skipping 47 matching lines @@
     ExtensionApiTest::SetUpCommandLine(command_line);
     command_line->AppendSwitch(switches::kEnableExperimentalExtensionApis);
     command_line->AppendSwitch(switches::kEnablePanels);
   }
 };
 
 // TODO(jianli): Enable it when Panel::CreateNativePanel is implemented.
 IN_PROC_BROWSER_TEST_F(WindowOpenPanelTest, DISABLED_WindowOpenPanel) {
   ASSERT_TRUE(RunExtensionTest("window_open/panel")) << message_;
 }