Make DownloadProtectionService not use BrowserList::GetLastActive() to load a link. Instead pass th…

chrome/browser/safe_browsing/download_protection_service.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/safe_browsing/download_protection_service.h"
 
 #include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/format_macros.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 847 matching lines @@
 void DownloadProtectionService::RequestFinished(
     CheckClientDownloadRequest* request) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   std::set<scoped_refptr<CheckClientDownloadRequest> >::iterator it =
       download_requests_.find(request);
   DCHECK(it != download_requests_.end());
   download_requests_.erase(*it);
 }
 
 void DownloadProtectionService::ShowDetailsForDownload(
-    const DownloadProtectionService::DownloadInfo& info) {
-  Browser* browser = BrowserList::GetLastActive();
-  DCHECK(browser && browser->is_type_tabbed());
-  content::OpenURLParams params(GURL(chrome::kDownloadScanningLearnMoreURL),
-                                content::Referrer(), NEW_FOREGROUND_TAB,
-                                content::PAGE_TRANSITION_TYPED, false);
-  browser->OpenURL(params);
+    const DownloadProtectionService::DownloadInfo& info,
+    content::PageNavigator* navigator) {
+  navigator->OpenURL(
+      content::OpenURLParams(GURL(chrome::kDownloadScanningLearnMoreURL),
+                             content::Referrer(),
+                             NEW_FOREGROUND_TAB,
+                             content::PAGE_TRANSITION_LINK,
+                             false));
 }
 
 namespace {
 // Escapes a certificate attribute so that it can be used in a whitelist
 // entry.  Currently, we only escape slashes, since they are used as a
 // separator between attributes.
 std::string EscapeCertAttribute(const std::string& attribute) {
   std::string escaped;
   for (size_t i = 0; i < attribute.size(); ++i) {
     if (attribute[i] == '%') {
@@ skipping 58 matching lines @@
 
   std::string issuer_fp = base::HexEncode(issuer.fingerprint().data,
                                           sizeof(issuer.fingerprint().data));
   for (std::set<std::string>::iterator it = paths_to_check.begin();
        it != paths_to_check.end(); ++it) {
     whitelist_strings->push_back("cert/" + issuer_fp + *it);
   }
 }
 
 }  // namespace safe_browsing