Don't trigger Phishguard pings if we cannot compute URL reputation.

components/safe_browsing/password_protection/password_protection_request.h

 // Copyright 2017 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.
 
 #ifndef COMPONENTS_SAFE_BROWSING_PASSWORD_PROTECTION_PASSWORD_PROTECTION_REQUEST_H_
 #define COMPONENTS_SAFE_BROWSING_PASSWORD_PROTECTION_PASSWORD_PROTECTION_REQUEST_H_
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/task/cancelable_task_tracker.h"
@@ skipping 22 matching lines @@
 // (5) |   UI   | If verdict cached, quit request; else prepare request proto.
 // (6) |   UI   | Start a timeout task, and send network request.
 // (7) |   UI   | On receiving response, handle response and finish.
 //     |        | On request timeout, cancel request.
 //     |        | On deletion of |password_protection_service_|, cancel request.
 class PasswordProtectionRequest : public base::RefCountedThreadSafe<
                                       PasswordProtectionRequest,
                                       content::BrowserThread::DeleteOnUIThread>,
                                   public net::URLFetcherDelegate {
  public:
-  PasswordProtectionRequest(const GURL& main_frame_url,
+  PasswordProtectionRequest(content::WebContents* web_contents,
+                            const GURL& main_frame_url,
                             const GURL& password_form_action,
                             const GURL& password_form_frame_url,
                             const std::string& saved_domain,
                             LoginReputationClientRequest::TriggerType type,
                             PasswordProtectionService* pps,
                             int request_timeout_in_ms);
 
   base::WeakPtr<PasswordProtectionRequest> GetWeakPtr() {
     return weakptr_factory_.GetWeakPtr();
   }
@@ skipping 42 matching lines @@
   // Initiates network request to Safe Browsing backend.
   void SendRequest();
 
   // Start a timer to cancel the request if it takes too long.
   void StartTimeout();
 
   // |this| will be destroyed after calling this function.
   void Finish(PasswordProtectionService::RequestOutcome outcome,
               std::unique_ptr<LoginReputationClientResponse> response);
 
+  // WebContents of the password protection event.
+  content::WebContents* web_contents_;
+
   // Main frame URL of the login form.
   const GURL main_frame_url_;
 
   // The action URL of the password form.
   const GURL password_form_action_;
 
   // Frame url of the detected password form.
   const GURL password_form_frame_url_;
 
   // Domain on which a password is saved and gets reused.
   const std::string saved_domain_;
 
   // If this request is for unfamiliar login page or for a password reuse event.
   const LoginReputationClientRequest::TriggerType request_type_;
 
   // When request is sent.
   base::TimeTicks request_start_time_;
 
   // URLFetcher instance for sending request and receiving response.
   std::unique_ptr<net::URLFetcher> fetcher_;
 
   // The PasswordProtectionService instance owns |this|.
   // Can only be accessed on UI thread.
   PasswordProtectionService* password_protection_service_;
 
-  // Safe Browsing database manager used to look up CSD whitelist.
-  // Can only be accessed on IO thread.
-  scoped_refptr<SafeBrowsingDatabaseManager> database_manager_;
-
   // If we haven't receive response after this period of time, we cancel this
   // request.
   const int request_timeout_in_ms_;
 
   std::unique_ptr<LoginReputationClientRequest> request_proto_;
 
   // Needed for canceling tasks posted to different threads.
   base::CancelableTaskTracker tracker_;
 
   base::WeakPtrFactory<PasswordProtectionRequest> weakptr_factory_;
   DISALLOW_COPY_AND_ASSIGN(PasswordProtectionRequest);
 };
 
 }  // namespace safe_browsing
 
 #endif  // COMPONENTS_SAFE_BROWSING_PASSWORD_PROTECTION_PASSWORD_PROTECTION_REQUEST_H_