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..d0bf1d80a2ad665969675cccfd3ba5ee9c84c3fa 100644
--- a/components/omnibox/browser/omnibox_field_trial.h
+++ b/components/omnibox/browser/omnibox_field_trial.h
@@ -426,6 +426,17 @@ class OmniboxFieldTrial {
   static std::string ZeroSuggestRedirectToChromeAdditionalFields();
 
   // ---------------------------------------------------------
+  // Clipboard URL suggestions:
+
+  // One parameter for the omnibox field trial that doesn't live in this file;

[Peter Kasting, 2017/04/12 20:02:30]

Nit: Remove "that"

[Mark P, 2017/04/12 21:56:59]

Done.

+  // instead it lives in
+  // components/open_from_clipboard/clipboard_recent_content.cc.
+  // This is because that component cannot include this component, else there
+  // would be a circular dependency.  Please see ClipboardRecentContent::
+  // MaximumAgeOfClipboard() for the usage of that parameter, named
+  // "ClipboardURLMaximumAge".

[Peter Kasting, 2017/04/12 20:02:30]

Nit: Name the parameter at the beginning of the comment, then say why it can't
live here.  (This is sort of the paragraph structure version of "don't use
passive voice".)

[Mark P, 2017/04/12 21:56:59]

Reorganized and rewrote.  Take a gander.

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