Omnibox - Reduce Clipboard URL Suggestions to 1 Hour

components/omnibox/browser/omnibox_field_trial.h

Index: components/omnibox/browser/omnibox_field_trial.h
diff --git a/components/omnibox/browser/omnibox_field_trial.h b/components/omnibox/browser/omnibox_field_trial.h
index 2409711b18e74acf5538f06890b3bae1e0e47138..82185aaf3dec36da126ca7e3e804061fabc24a43 100644
--- a/components/omnibox/browser/omnibox_field_trial.h
+++ b/components/omnibox/browser/omnibox_field_trial.h
@@ -426,6 +426,16 @@ class OmniboxFieldTrial {
   static std::string ZeroSuggestRedirectToChromeAdditionalFields();
 
   // ---------------------------------------------------------
+  // Clipboard URL suggestions:
+
+  // The parameter "ClipboardURLMaximumAge" doesn't live in this file; instead
+  // it lives in
+  // components/open_from_clipboard/clipboard_recent_content.cc.
+  // Please see ClipboardRecentContent::MaximumAgeOfClipboard() for the usage
+  // of it.  The parameter cannot live here because that component cannot
+  // include this component, else there would be a circular dependency.

[Peter Kasting, 2017/04/12 22:49:49]

Cool, this is better.

Although reading it started to make me wonder "maybe we should just leave this
file untouched and not say anything", but whatever.

[Mark P, 2017/04/12 23:08:14]

I think it's good to have a central place where people can find about all
existing omnibox experiments (and be able to clean them up later too).

+
+  // ---------------------------------------------------------
   // Exposed publicly for the sake of unittests.
   static const char kBundledExperimentFieldTrialName[];
   // Rule names used by the bundled experiment.