Placing SRT Prompt on Feedback form behind Finch feature.

chrome/browser/extensions/api/feedback_private/feedback_private_api.h

Index: chrome/browser/extensions/api/feedback_private/feedback_private_api.h
diff --git a/chrome/browser/extensions/api/feedback_private/feedback_private_api.h b/chrome/browser/extensions/api/feedback_private/feedback_private_api.h
index b3c7d6104ef402f376d3f2dadeb9358c08a88afe..7111b933a75fe40b8bcc19314b470146c2a89058 100644
--- a/chrome/browser/extensions/api/feedback_private/feedback_private_api.h
+++ b/chrome/browser/extensions/api/feedback_private/feedback_private_api.h
@@ -5,6 +5,7 @@
 #ifndef CHROME_BROWSER_EXTENSIONS_API_FEEDBACK_PRIVATE_FEEDBACK_PRIVATE_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_FEEDBACK_PRIVATE_FEEDBACK_PRIVATE_API_H_
 
+#include "base/feature_list.h"
 #include "chrome/browser/extensions/chrome_extension_function.h"
 #include "chrome/common/extensions/api/feedback_private.h"
 #include "extensions/browser/browser_context_keyed_api_factory.h"
@@ -13,6 +14,9 @@
 
 namespace extensions {
 
+// Allows enabling/disabling SRT Prompt as a Variations feature.
+extern const base::Feature kSrtPromptOnFeedbackForm;

[afakhry, 2016/06/22 12:58:42]

If it's only needed in feedback_private_api.cc, is there a reason why it's
defined as extern here?

Can you define it as constexpr in the anonymous namespace in the .cc file?

+
 class FeedbackService;
 
 class FeedbackPrivateAPI : public BrowserContextKeyedAPI {