Address bar is shown on pop-out extension windows.

chrome/browser/extensions/extension_host.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 "chrome/browser/extensions/extension_host.h"
 
 #include <list>
 
 #include "base/memory/singleton.h"
 #include "base/message_loop.h"
@@ skipping 568 matching lines @@
   if (disposition == NEW_POPUP) {
     // Find a browser with a matching profile for creating a popup.
     // (If none is found, NULL argument to NavigateParams is valid.)
     Browser* browser = BrowserList::FindTabbedBrowser(
         contents->profile(),
         false);  // Match incognito exactly.
     TabContentsWrapper* wrapper = new TabContentsWrapper(contents);
     browser::NavigateParams params(browser, wrapper);
     if (!browser)
       params.profile = contents->profile();
+    params.extension_app_id = extension_id_;

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

I forget the semantics of ShowCreatedWindow in this context.  Will it only be
called for URLs that are part of the extension?  Or is it called for any URL
opened from within the extension?  Do you know if this change means that we've
changed behavior in this case?

In any event, if this param is the bit that causes the popup to open up without
an address bar, please add a comment to that effect so that others don't have to
go digging to track down the source of this behavior.

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

This is called for any URL opened with window.open() within the extension.
We didn't change behavior.  This is the same behavior as it was before the
regression happened.
Not sure of the value of the comment here, but added on anyway.

     params.disposition = NEW_POPUP;
     params.window_bounds = initial_pos;
     params.window_action = browser::NavigateParams::SHOW_WINDOW;
     params.user_gesture = user_gesture;
     browser::Navigate(&params);
     return;
   }
 
   // If the tab contents isn't a popup, it's a normal tab. We need to find a
   // home for it. This is typically a Browser, but it can also be some other
@@ skipping 211 matching lines @@
   return window_id;
 }
 
 void ExtensionHost::OnRunFileChooser(
     const ViewHostMsg_RunFileChooser_Params& params) {
   if (file_select_helper_.get() == NULL)
     file_select_helper_.reset(new FileSelectHelper(profile()));
   file_select_helper_->RunFileChooser(render_view_host_,
                                       GetAssociatedTabContents(), params);
 }