Allow command-line arguments to override EF public key

chrome/common/origin_trials/origin_trial_key_manager.h

Index: chrome/common/origin_trials/origin_trial_key_manager.h
diff --git a/chrome/common/origin_trials/origin_trial_key_manager.h b/chrome/common/origin_trials/origin_trial_key_manager.h
index cd6b2453b15ec4ef3a48722509766a0db0bcd013..f9daf477349c4dc4b37a825daa05560336ad6857 100644
--- a/chrome/common/origin_trials/origin_trial_key_manager.h
+++ b/chrome/common/origin_trials/origin_trial_key_manager.h
@@ -5,13 +5,25 @@
 #ifndef CHROME_COMMON_ORIGIN_TRIALS_ORIGIN_TRIAL_KEY_MANAGER_H_
 #define CHROME_COMMON_ORIGIN_TRIALS_ORIGIN_TRIAL_KEY_MANAGER_H_
 
+#include <string>
+
+#include "base/macros.h"
 #include "base/strings/string_piece.h"
 
 // This class is instantiated on the main/ui thread, but its methods can be
 // accessed from any thread.
 class OriginTrialKeyManager {
  public:
-  base::StringPiece GetPublicKey();
+  OriginTrialKeyManager();
+  ~OriginTrialKeyManager();
+
+  bool SetPublicKeyFromASCIIString(const std::string& ascii_public_key);

[sky, 2016/04/14 16:19:51]

Is there a reason to go with the setter instead of making this class immutable
and creating a new one when you want to change the value? Less threading issues
that way.

[iclelland, 2016/04/14 19:29:07]

That's a good point; I'll change that, and swap out the manager when the key
changes instead.

There are more values that could be set in the future (disabled features,
revoked tokens, etc), but I should be able to gather them all at once, and make
them constructor arguments instead.

+  base::StringPiece GetPublicKey() const;
+
+ private:
+  std::string public_key_;
+
+  DISALLOW_COPY_AND_ASSIGN(OriginTrialKeyManager);
 };
 
 #endif  // CHROME_COMMON_ORIGIN_TRIALS_ORIGIN_TRIAL_KEY_MANAGER_H_