Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(177)

Side by Side Diff: chrome/browser/password_manager/chrome_password_manager_client_unittest.cc

Issue 2856033002: Add finch control of user population in low reputation requests (Closed)
Patch Set: rebase Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | chrome/browser/safe_browsing/chrome_password_protection_service.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/password_manager/chrome_password_manager_client.h" 5 #include "chrome/browser/password_manager/chrome_password_manager_client.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 8
9 #include <string> 9 #include <string>
10 10
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
76 nullptr) {} 76 nullptr) {}
77 77
78 ~MockPasswordProtectionService() override {} 78 ~MockPasswordProtectionService() override {}
79 79
80 MOCK_METHOD3(FillReferrerChain, 80 MOCK_METHOD3(FillReferrerChain,
81 void(const GURL&, 81 void(const GURL&,
82 int, 82 int,
83 safe_browsing::LoginReputationClientRequest::Frame*)); 83 safe_browsing::LoginReputationClientRequest::Frame*));
84 MOCK_METHOD0(IsExtendedReporting, bool()); 84 MOCK_METHOD0(IsExtendedReporting, bool());
85 MOCK_METHOD0(IsIncognito, bool()); 85 MOCK_METHOD0(IsIncognito, bool());
86 MOCK_METHOD0(IsPingingEnabled, bool()); 86 MOCK_METHOD1(IsPingingEnabled, bool(const base::Feature&));
87 MOCK_METHOD0(IsHistorySyncEnabled, bool());
87 MOCK_METHOD3(MaybeStartLowReputationRequest, 88 MOCK_METHOD3(MaybeStartLowReputationRequest,
88 void(const GURL&, const GURL&, const GURL&)); 89 void(const GURL&, const GURL&, const GURL&));
89 90
90 private: 91 private:
91 DISALLOW_COPY_AND_ASSIGN(MockPasswordProtectionService); 92 DISALLOW_COPY_AND_ASSIGN(MockPasswordProtectionService);
92 }; 93 };
93 #endif 94 #endif
94 95
95 // TODO(vabr): Get rid of the mocked client in the client's own test, see 96 // TODO(vabr): Get rid of the mocked client in the client's own test, see
96 // http://crbug.com/474577. 97 // http://crbug.com/474577.
(...skipping 518 matching lines...) Expand 10 before | Expand all | Expand 10 after
615 web_contents()->GetBrowserContext(), nullptr)); 616 web_contents()->GetBrowserContext(), nullptr));
616 std::unique_ptr<MockChromePasswordManagerClient> client( 617 std::unique_ptr<MockChromePasswordManagerClient> client(
617 new MockChromePasswordManagerClient(test_web_contents.get())); 618 new MockChromePasswordManagerClient(test_web_contents.get()));
618 EXPECT_CALL(*client->password_protection_service(), 619 EXPECT_CALL(*client->password_protection_service(),
619 MaybeStartLowReputationRequest(_, _, _)) 620 MaybeStartLowReputationRequest(_, _, _))
620 .Times(1); 621 .Times(1);
621 client->CheckSafeBrowsingReputation(GURL("http://foo.com/submit"), 622 client->CheckSafeBrowsingReputation(GURL("http://foo.com/submit"),
622 GURL("http://foo.com/iframe.html")); 623 GURL("http://foo.com/iframe.html"));
623 } 624 }
624 #endif 625 #endif
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/safe_browsing/chrome_password_protection_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698