Wireup SafeBrowsingNavigationObserverManager to help PPAPI download attribution

chrome/browser/file_select_helper.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/file_select_helper.h"
 
 #include <stddef.h>
 
 #include <string>
 #include <utility>
@@ skipping 528 matching lines @@
         base::FilePath::StringType extension =
             default_file_path.ReplaceExtension(extension_in_list)
                 .FinalExtension();
         alternate_extensions.push_back(extension);
       }
     }
   }
 
   GURL requestor_url = params->requestor;
   sb_service->download_protection_service()->CheckPPAPIDownloadRequest(
-      requestor_url, default_file_path, alternate_extensions, profile_,
+      requestor_url, render_frame_host_->GetLastCommittedURL(),

[asanka, 2017/01/05 20:35:31]

render_frame_host_ could be nullptr :-(

[Jialiu Lin, 2017/01/05 22:10:13]

Added a check here. in case render_frame_host_ is nullptr

+      WebContents::FromRenderFrameHost(render_frame_host_),
+      default_file_path, alternate_extensions, profile_,
       base::Bind(&InterpretSafeBrowsingVerdict,
                  base::Bind(&FileSelectHelper::ProceedWithSafeBrowsingVerdict,
                             this, default_file_path, base::Passed(&params))));
 }
 
 void FileSelectHelper::ProceedWithSafeBrowsingVerdict(
     const base::FilePath& default_file_path,
     std::unique_ptr<content::FileChooserParams> params,
     bool allowed_by_safe_browsing) {
   if (!allowed_by_safe_browsing) {
@@ skipping 143 matching lines @@
 
 // static
 base::FilePath FileSelectHelper::GetSanitizedFileName(
     const base::FilePath& suggested_filename) {
   if (suggested_filename.empty())
     return base::FilePath();
   return net::GenerateFileName(
       GURL(), std::string(), std::string(), suggested_filename.AsUTF8Unsafe(),
       std::string(), l10n_util::GetStringUTF8(IDS_DEFAULT_DOWNLOAD_FILENAME));
 }