Omnibox UI Experiments: Hook up elide-after-host feature flag.

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 a9d0f7eef6e92e675d848da783d5e7b62a9d279d..43a8c60a70b54a6846f55bf2dba2cf5a37b8b973 100644
--- a/components/omnibox/browser/omnibox_field_trial.h
+++ b/components/omnibox/browser/omnibox_field_trial.h
@@ -35,7 +35,7 @@ extern const base::Feature kSearchProviderContextAllowHttpsUrls;
 extern const base::Feature kZeroSuggestRedirectToChrome;
 extern const base::Feature kZeroSuggestSwapTitleAndUrl;
 extern const base::Feature kDisplayTitleForCurrentUrl;
-extern const base::Feature kUIExperimentHideSuggestionUrlPath;

[Mark P, 2017/07/05 18:38:45]

As near as I can tell, you checked in this declaration for a feature flag that
wasn't used.  Is that correct?

[tommycli, 2017/07/05 19:38:26]

Hey... I wasn't able to grok your question -- do you mind rephrasing it?

The feature flag used to be called "hide suggestion url path", but wasn't
implemented yet... it was just a placeholder feature flag.

Through the course of implementation, Peter suggested that a more accurate name
would be "elide suggestion url after host" -- so I'm updating the feature flag
name to more accurately reflect what's going on.

[Mark P, 2017/07/05 19:43:24]

You answered my question. :-)  It was a placeholder declaration.

+extern const base::Feature kUIExperimentElideSuggestionUrlAfterHost;
 extern const base::Feature kUIExperimentHideSuggestionUrlScheme;
 extern const base::Feature kUIExperimentHideSuggestionUrlTrivialSubdomains;
 extern const base::Feature kUIExperimentMaxAutocompleteMatches;