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

chrome/browser/password_manager/chrome_password_manager_client_unittest.cc

 // Copyright 2014 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/password_manager/chrome_password_manager_client.h"
 
 #include <stdint.h>
 
 #include <string>
 
@@ skipping 67 matching lines @@
   ~MockPasswordProtectionService() override {}
 
   MOCK_METHOD3(FillReferrerChain,
                void(const GURL&,
                     int,
                     safe_browsing::LoginReputationClientRequest::Frame*));
   MOCK_METHOD0(IsExtendedReporting, bool());
   MOCK_METHOD0(IsIncognito, bool());
   MOCK_METHOD2(IsPingingEnabled, bool(const base::Feature&, RequestOutcome*));
   MOCK_METHOD0(IsHistorySyncEnabled, bool());
-  MOCK_METHOD3(MaybeStartPasswordFieldOnFocusRequest,
-               void(const GURL&, const GURL&, const GURL&));
-  MOCK_METHOD2(MaybeStartProtectedPasswordEntryRequest,
-               void(const GURL&, const std::string&));
+  MOCK_METHOD4(MaybeStartPasswordFieldOnFocusRequest,
+               void(WebContents*, const GURL&, const GURL&, const GURL&));
+  MOCK_METHOD3(MaybeStartProtectedPasswordEntryRequest,
+               void(WebContents*, const GURL&, const std::string&));
 
  private:
   DISALLOW_COPY_AND_ASSIGN(MockPasswordProtectionService);
 };
 #endif
 
 // TODO(vabr): Get rid of the mocked client in the client's own test, see
 // http://crbug.com/474577.
 class MockChromePasswordManagerClient : public ChromePasswordManagerClient {
  public:
@@ skipping 510 matching lines @@
 
 #if defined(SAFE_BROWSING_DB_LOCAL)
 TEST_F(ChromePasswordManagerClientTest,
        VerifyMaybeStartPasswordFieldOnFocusRequestCalled) {
   std::unique_ptr<WebContents> test_web_contents(
       content::WebContentsTester::CreateTestWebContents(
           web_contents()->GetBrowserContext(), nullptr));
   std::unique_ptr<MockChromePasswordManagerClient> client(
       new MockChromePasswordManagerClient(test_web_contents.get()));
   EXPECT_CALL(*client->password_protection_service(),
-              MaybeStartPasswordFieldOnFocusRequest(_, _, _))
+              MaybeStartPasswordFieldOnFocusRequest(_, _, _, _))
       .Times(1);
   client->CheckSafeBrowsingReputation(GURL("http://foo.com/submit"),
                                       GURL("http://foo.com/iframe.html"));
 }
 
 TEST_F(ChromePasswordManagerClientTest,
        VerifyMaybeProtectedPasswordEntryRequestCalled) {
   std::unique_ptr<WebContents> test_web_contents(
       content::WebContentsTester::CreateTestWebContents(
           web_contents()->GetBrowserContext(), nullptr));
   std::unique_ptr<MockChromePasswordManagerClient> client(
       new MockChromePasswordManagerClient(test_web_contents.get()));
   EXPECT_CALL(*client->password_protection_service(),
-              MaybeStartProtectedPasswordEntryRequest(_, _))
+              MaybeStartProtectedPasswordEntryRequest(_, _, _))
       .Times(1);
   client->CheckProtectedPasswordEntry(std::string("saved_domain.com"));
 }
 
 #endif