Add report URL to safe search API functionality.

chrome/browser/about_flags.cc

Index: chrome/browser/about_flags.cc
diff --git a/chrome/browser/about_flags.cc b/chrome/browser/about_flags.cc
index d1a8ea4426bb2ac251568c9eb50df1d1a211b0d7..0548d2c6cad1af9039052ac27d5cb8d364054bba 100644
--- a/chrome/browser/about_flags.cc
+++ b/chrome/browser/about_flags.cc
@@ -1694,6 +1694,10 @@ const FeatureEntry kFeatureEntries[] = {
     {"enable-md-history", IDS_FLAGS_ENABLE_MATERIAL_DESIGN_HISTORY_NAME,
      IDS_FLAGS_ENABLE_MATERIAL_DESIGN_HISTORY_DESCRIPTION, kOsDesktop,
      FEATURE_VALUE_TYPE(features::kMaterialDesignHistoryFeature)},
+    {"enable-safe-search-url-reporting",

[Marc Treib, 2016/03/21 12:32:21]

nit: I'd remove the "enable-" prefix, since it doesn't really add anything.
(Though lots of other flags do use it, so meh...)

[atanasova, 2016/03/22 15:45:41]

Since the flag will be used for enabling the feature (disabled by default), I
think that this keeps the style of the other flags.

[Marc Treib, 2016/03/22 16:06:49]

If it's enabled or disabled by default can change though. (And hopefully will,
at some point :) )
Anyway, since this is a FEATURE_VALUE_TYPE, all the name does is define the
anchor on chrome://flags, as in chrome://flags#enable-safe-search-url-reporting

[atanasova, 2016/03/23 10:45:00]

Okay, you convinced me. Removed the 'enable' here and the string that are used.

+     IDS_FLAGS_ENABLE_SAFE_SEARCH_URL_REPORTING_NAME,
+     IDS_FLAGS_ENABLE_SAFE_SEARCH_URL_REPORTING_DESCRIPTION, kOsDesktop,

[Marc Treib, 2016/03/22 16:06:49]

I just noticed: Why not kOsAll?

[atanasova, 2016/03/23 10:45:00]

I was trying it out first on desktop only, but agreed that it does make sense to
have it everywhere.

+     FEATURE_VALUE_TYPE(features::kSafeSearchUrlReporting)},
 #if defined(OS_WIN)
     {"enable-windows-desktop-search-redirection",
      IDS_FLAGS_WINDOWS_DESKTOP_SEARCH_REDIRECTION_NAME,