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

Unified Diff: components/policy/core/common/cloud/cloud_policy_client.cc

Issue 2537643003: Drop verification_key_hash from CloudPolicyClient constructor (Closed)
Patch Set: git cl format Created 4 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: components/policy/core/common/cloud/cloud_policy_client.cc
diff --git a/components/policy/core/common/cloud/cloud_policy_client.cc b/components/policy/core/common/cloud/cloud_policy_client.cc
index 322ac7916bc8809ee538f625146096e2c5b9a352..79cca4049e9dd1f5ed1a12b963f96cceb9b1b309 100644
--- a/components/policy/core/common/cloud/cloud_policy_client.cc
+++ b/components/policy/core/common/cloud/cloud_policy_client.cc
@@ -55,13 +55,11 @@ void CloudPolicyClient::Observer::OnRobotAuthCodesFetched(
CloudPolicyClient::CloudPolicyClient(
const std::string& machine_id,
const std::string& machine_model,
- const std::string& verification_key_hash,
DeviceManagementService* service,
scoped_refptr<net::URLRequestContextGetter> request_context,
SigningService* signing_service)
: machine_id_(machine_id),
machine_model_(machine_model),
- verification_key_hash_(verification_key_hash),
service_(service), // Can be null for unit tests.
signing_service_(signing_service),
request_context_(request_context),
@@ -233,8 +231,7 @@ void CloudPolicyClient::FetchPolicy() {
if (public_key_version_valid_)
fetch_request->set_public_key_version(public_key_version_);
- if (!verification_key_hash_.empty())
- fetch_request->set_verification_key_hash(verification_key_hash_);
+ fetch_request->set_verification_key_hash(kPolicyVerificationKeyHash);
// These fields are included only in requests for chrome policy.
if (IsChromePolicy(type_to_fetch.first)) {

Powered by Google App Engine
This is Rietveld 408576698