Allow command-line arguments to override EF public key

chrome/browser/origin_trials/origin_trial_controller.h

Index: chrome/browser/origin_trials/origin_trial_controller.h
diff --git a/chrome/browser/origin_trials/origin_trial_controller.h b/chrome/browser/origin_trials/origin_trial_controller.h
new file mode 100644
index 0000000000000000000000000000000000000000..695b0e54795c4f9d10c77a890b51e7b00f89b8fe
--- /dev/null
+++ b/chrome/browser/origin_trials/origin_trial_controller.h
@@ -0,0 +1,24 @@
+// Copyright (c) 2016 The Chromium Authors. All rights reserved.

[sky, 2016/03/03 20:55:30]

style guide says no (c) anymore.

[iclelland, 2016/03/07 14:48:19]

Thanks, I missed that.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_ORIGIN_TRIALS_ORIGIN_TRIAL_CONTROLLER_H_
+#define CHROME_BROWSER_ORIGIN_TRIALS_ORIGIN_TRIAL_CONTROLLER_H_
+
+namespace base {
+class CommandLine;
+}
+
+class OriginTrialController {

[sky, 2016/03/03 20:55:30]

There is no point in using a class for this. If you really want the class, then
use DISALLOW_IMPLICIT_CONSTRUCTORS.

[iclelland, 2016/03/07 14:48:19]

The class is really just there for namespacing, not to hold any state.
Done.

+ public:
+  static const char kFieldTrialName[];
+  static const char kPublicKeyFieldName[];
+
+  // Updates the renderer command line to pass any relevant origin trial
+  // configuration to the renderer at startup. Currently this can include an
+  // updated public key to use for validating tokens. This method is
+  // thread-safe.
+  static void UpdateCommandLineFromFieldTrials(base::CommandLine* command_line);
+};
+
+#endif  // CHROME_BROWSER_ORIGIN_TRIALS_ORIGIN_TRIAL_CONTROLLER_H_