Placing SRT Prompt on Feedback form behind Finch feature.

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

 // Copyright 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_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"
 #include "extensions/browser/extension_function.h"
 #include "ui/gfx/geometry/rect.h"
 
 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 {
  public:
   explicit FeedbackPrivateAPI(content::BrowserContext* context);
   ~FeedbackPrivateAPI() override;
 
   FeedbackService* GetService() const;
 
   void RequestFeedback(const std::string& description_template,
@@ skipping 90 matching lines @@
                              FEEDBACKPRIVATE_LOGSRTPROMPTRESULT);
 
  protected:
   ~FeedbackPrivateLogSrtPromptResultFunction() override {}
   AsyncExtensionFunction::ResponseAction Run() override;
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_FEEDBACK_PRIVATE_FEEDBACK_PRIVATE_API_H_