Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(584)

Unified Diff: components/omnibox/browser/physical_web_provider.h

Issue 2319033006: Include a page title in the Physical Web omnibox overflow item (Closed)
Patch Set: changes for mpearson@, refactor unit tests Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/omnibox/browser/physical_web_provider.h
diff --git a/components/omnibox/browser/physical_web_provider.h b/components/omnibox/browser/physical_web_provider.h
index d4e7486c73a869a861560e8c996a9f6c023e42ca..6f4ae6fc19565d331bcba03cedd72ebee13da811 100644
--- a/components/omnibox/browser/physical_web_provider.h
+++ b/components/omnibox/browser/physical_web_provider.h
@@ -38,9 +38,13 @@ class PhysicalWebProvider : public AutocompleteProvider {
// Adds an overflow match item to |matches_| with a relevance score equal to
// |relevance| and a label indicating there are |additional_url_count| more
- // nearby URLs. Selecting the overflow item navigates to the Physical Web
- // WebUI, which displays the full list of nearby URLs.
- void AppendOverflowItem(int additional_url_count, int relevance);
+ // nearby URLs. The page |title| of one of the additional nearby URLs will be
+ // included in the label, truncating if necessary. Selecting the overflow item
+ // navigates to the Physical Web WebUI, which displays the full list of nearby
+ // URLs.
+ void AppendOverflowItem(int additional_url_count,
+ int relevance,
+ const base::string16& title);
AutocompleteProviderClient* client_;

Powered by Google App Engine
This is Rietveld 408576698