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

chrome/browser/platform_util_chromeos.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/platform_util.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/file_util.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/chromeos/extensions/file_manager_util.h"
 #include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_navigator.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "content/public/browser/browser_thread.h"
 #include "googleurl/src/gurl.h"
 
 using content::BrowserThread;
 
 class Profile;
 
 namespace {
 
 const char kGmailComposeUrl[] =
     "https://mail.google.com/mail/?extsrc=mailto&url=";
 
 void OpenItemOnFileThread(const FilePath& full_path) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
   base::Closure callback;
   if (file_util::DirectoryExists(full_path))
     callback = base::Bind(&file_manager_util::ViewFolder, full_path);
   else
     callback = base::Bind(&file_manager_util::ViewFile, full_path);
   BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, callback);
 }
 
 void OpenURL(const std::string& url) {
   // TODO(beng): improve this to locate context from call stack.
-  chrome::NavigateParams params(NULL, GURL(url), content::PAGE_TRANSITION_LINK);
+  Browser* browser = browser::FindOrCreateTabbedBrowser(
+      ProfileManager::GetDefaultProfileOrOffTheRecord());
+  chrome::NavigateParams params(
+      browser, GURL(url), content::PAGE_TRANSITION_LINK);
   params.disposition = NEW_FOREGROUND_TAB;
-  params.profile = ProfileManager::GetDefaultProfileOrOffTheRecord();
   chrome::Navigate(&params);
 }
 
 }  // namespace
 
 namespace platform_util {
 
 void ShowItemInFolder(const FilePath& full_path) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   file_manager_util::ShowFileInFolder(full_path);
@@ skipping 17 matching lines @@
   // As such we should keep this code here.
   if (url.SchemeIs("mailto")) {
     std::string string_url = kGmailComposeUrl;
     string_url.append(url.spec());
     BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
                             base::Bind(OpenURL, string_url));
   }
 }
 
 }  // namespace platform_util