Extension in background unable to open window if there is no browser running.

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 580 matching lines @@
     return;
 
   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)

[Matt Perry, 2011/05/19 00:50:32]

Does it make sense to set the profile even if browser is non-NULL?

[prasadt, 2011/05/19 00:58:18]

I'm not really sure and tried to make the fix affecting as little as possible. 
Current code is using the browser's profile if there is a browser instance.

[Matt Perry, 2011/05/19 01:01:32]

Fair enough.

+      params.profile = contents->profile();
     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 216 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_,
                                       associated_tab_contents(), params);
 }