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

Unified Diff: chrome/browser/ui/send_feedback_experiment.h

Issue 12490012: Send Feedback Experiment (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: whitespace Created 7 years, 9 months 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: chrome/browser/ui/send_feedback_experiment.h
diff --git a/chrome/browser/ui/send_feedback_experiment.h b/chrome/browser/ui/send_feedback_experiment.h
new file mode 100644
index 0000000000000000000000000000000000000000..2cc7bc82465793f731254785d4e9550c39bf5a7f
--- /dev/null
+++ b/chrome/browser/ui/send_feedback_experiment.h
@@ -0,0 +1,30 @@
+// 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 som
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 al
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 the
Harry McCleave 2013/03/19 22:23:29 Done.
+} // namespace chrome
+
+#endif // CHROME_BROWSER_UI_SEND_FEEDBACK_EXPERIMENT_H_

Powered by Google App Engine
This is Rietveld 408576698