Send Feedback Experiment

chrome/browser/ui/send_feedback_experiment.h

+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_UI_SEND_FEEDBACK_EXPERIMENT_H_
+#define CHROME_BROWSER_UI_SEND_FEEDBACK_EXPERIMENT_H_
+
+// This file exists to facilitate a finch experiment which
+// modifies the description and/or location of the
+// Report Feedback menu option, and corresponding option
+// on the chrome://help page. [crbug.com/169339.]

[SteveT, 2013/03/19 14:38:58]

nit: Hmm, I think we can include more words on some of the lines above before
hitting the 80 char limit.

nit: "[crbug.com/169339.]" -> "(crbug.com/169339)"

[Harry McCleave, 2013/03/19 22:23:29]

Done.

+namespace chrome {

[SteveT, 2013/03/19 14:38:58]

Hmm, I *think* nested name spaces like this are allowed, but I think we might
want a space between these two lines.

[Harry McCleave, 2013/03/19 22:23:29]

Done.

+namespace send_feedback_experiment {
+
+// Returns true if the alternative (Send Feedback) text should be
+// used in place of the original (Report an Issue...).
+bool UseAlternateText();
+
+// Returns True if the alternative menu location for the Send Feedback/Report
+// an Issue... option should be used.
+bool UseAlternateLocation();
+
+// Return the ID for the desired string to be used for a menu item
+// (See UseAlternateText).
+int GetMenuLabelID();
+
+}  // namespace send_feedback_experiment

[SteveT, 2013/03/19 14:38:58]

nit: I believe there should be a break between these two lines.

[Harry McCleave, 2013/03/19 22:23:29]

Done.

+}  // namespace chrome
+
+#endif  // CHROME_BROWSER_UI_SEND_FEEDBACK_EXPERIMENT_H_