Remove disable-policy-key-verification command line flag

components/policy/core/common/cloud/cloud_policy_constants.cc

Index: components/policy/core/common/cloud/cloud_policy_constants.cc
diff --git a/components/policy/core/common/cloud/cloud_policy_constants.cc b/components/policy/core/common/cloud/cloud_policy_constants.cc
index 4e2bc47a31b9e0af1587ab533c54ab35c5be3577..f272a72c1a15b36ead0144d3f3e6d4aee1d2f39d 100644
--- a/components/policy/core/common/cloud/cloud_policy_constants.cc
+++ b/components/policy/core/common/cloud/cloud_policy_constants.cc
@@ -99,13 +99,8 @@ const uint8_t kPolicyVerificationKey[] = {
 const char kPolicyVerificationKeyHash[] = "1:356l7w";
 
 std::string GetPolicyVerificationKey() {

[pmarko, 2017/01/04 13:50:22]

Do you think that we should still expose this as a std::string returning
function (even though there's nothing dynamic inside anymore), or rather expose
the array and a separate size constant?
(or even as a std::array but I haven't seen a precedent for this)

[pastarmovj, 2017/01/04 14:37:27]

I think this function is still useful utility for getting the key in a more
usable form.

Dunno if std::array will do with the POD only constants rule in the style guide.

-  base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
-  if (command_line->HasSwitch(switches::kDisablePolicyKeyVerification)) {
-    return std::string();
-  } else {
-    return std::string(reinterpret_cast<const char*>(kPolicyVerificationKey),
-                       sizeof(kPolicyVerificationKey));
-  }
+  return std::string(reinterpret_cast<const char*>(kPolicyVerificationKey),
+                     sizeof(kPolicyVerificationKey));
 }
 
 }  // namespace policy