Add finch control of user population in low reputation requests

chrome/browser/password_manager/chrome_password_manager_client_unittest.cc

Index: chrome/browser/password_manager/chrome_password_manager_client_unittest.cc
diff --git a/chrome/browser/password_manager/chrome_password_manager_client_unittest.cc b/chrome/browser/password_manager/chrome_password_manager_client_unittest.cc
index b308749f850ea39ed50343668029e370a19e9034..d3a8f26c8513f259146f59665be4f9a787f824fd 100644
--- a/chrome/browser/password_manager/chrome_password_manager_client_unittest.cc
+++ b/chrome/browser/password_manager/chrome_password_manager_client_unittest.cc
@@ -83,7 +83,8 @@ class MockPasswordProtectionService
                     safe_browsing::LoginReputationClientRequest::Frame*));
   MOCK_METHOD0(IsExtendedReporting, bool());
   MOCK_METHOD0(IsIncognito, bool());
-  MOCK_METHOD0(IsPingingEnabled, bool());
+  MOCK_METHOD1(IsPingingEnabled, bool(const base::Feature&));
+  MOCK_METHOD0(IsHistorySyncEnabled, bool());

[Nathan Parker, 2017/05/05 20:58:21]

Are these used in this file?  Or do you need them just to return their default
value (false)?

[Jialiu Lin, 2017/05/05 22:19:32]

They are pure virtual functions in PasswordProtectionService, need to declare
them as mock functions to create instance of MockPasswordProtectionService.

   MOCK_METHOD3(MaybeStartLowReputationRequest,
                void(const GURL&, const GURL&, const GURL&));