Make chrome::Navigate take in a Browser* that is never NULL. That function doesn't have a context, …

chrome/browser/extensions/extension_host.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/extensions/extension_host.h"
 
 #include <list>
 
 #include "base/bind.h"
 #include "base/memory/singleton.h"
@@ skipping 569 matching lines @@
     WebContents* associated_contents = GetAssociatedWebContents();
     if (associated_contents &&
         associated_contents->GetBrowserContext() ==
             new_contents->GetBrowserContext()) {
       associated_contents->AddNewContents(
           new_contents, disposition, initial_pos, user_gesture);
       return;
     }
   }
 
-  // Find a browser with a profile that matches the new tab. If none is found,
-  // NULL argument to NavigateParams is valid.
   Profile* profile =
       Profile::FromBrowserContext(new_contents->GetBrowserContext());
-  Browser* browser = browser::FindTabbedBrowser(
-      profile, false);  // Match incognito exactly.
+  Browser* browser = browser::FindOrCreateTabbedBrowser(profile);
   TabContents* tab_contents = new TabContents(new_contents);
   chrome::NavigateParams params(browser, tab_contents);
 
   // The extension_app_id parameter ends up as app_name in the Browser
   // which causes the Browser to return true for is_app().  This affects
   // among other things, whether the location bar gets displayed.
   // TODO(mpcomplete): This seems wrong. What if the extension content is hosted
   // in a tab?
   if (disposition == NEW_POPUP)
     params.extension_app_id = extension_id_;
 
-  if (!browser)
-    params.profile = profile;
   params.disposition = disposition;
   params.window_bounds = initial_pos;
   params.window_action = chrome::NavigateParams::SHOW_WINDOW;
   params.user_gesture = user_gesture;
   chrome::Navigate(&params);
 }
 
 void ExtensionHost::RenderViewReady() {
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_EXTENSION_HOST_CREATED,
       content::Source<Profile>(profile_),
       content::Details<ExtensionHost>(this));
 }