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

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

Issue 2583763003: Factor out AutocompleteMatch creation from BookmarkProvider (Closed)
Patch Set: changes for mpearson@ Created 3 years, 12 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/titled_url_match_utils.h
diff --git a/components/omnibox/browser/titled_url_match_utils.h b/components/omnibox/browser/titled_url_match_utils.h
new file mode 100644
index 0000000000000000000000000000000000000000..f2e7265b35ac8dbce1c478b34ea7affb7563173f
--- /dev/null
+++ b/components/omnibox/browser/titled_url_match_utils.h
@@ -0,0 +1,35 @@
+// Copyright (c) 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_OMNIBOX_BROWSER_TITLED_URL_MATCH_UTILS_H_
+#define COMPONENTS_OMNIBOX_BROWSER_TITLED_URL_MATCH_UTILS_H_
+
+#include "base/strings/string16.h"
+#include "components/bookmarks/browser/titled_url_match.h"
+#include "components/omnibox/browser/autocomplete_match_type.h"
+
+class AutocompleteInput;
+class AutocompleteProvider;
+class AutocompleteSchemeClassifier;
+struct AutocompleteMatch;
+
+namespace bookmarks {
+
+// Compose an AutocompleteMatch based on |match| that has the match's URL and
+// page title, type |type|, and relevance score |relevance|. |input| is used to
+// compute the match's inline_autocompletion. |fixed_up_input_text| is used in
+// that way as well; it's passed separately so this function doesn't have to
+// compute it.
+AutocompleteMatch TitledUrlMatchToAutocompleteMatch(
+ const bookmarks::TitledUrlMatch& match,
+ AutocompleteMatchType::Type type,
+ int relevance,
+ AutocompleteProvider* provider,
+ const AutocompleteSchemeClassifier& scheme_classifier,
+ const AutocompleteInput& input,
+ const base::string16& fixed_up_input_text);
+
+} // namespace bookmarks
+
+#endif // COMPONENTS_OMNIBOX_BROWSER_TITLED_URL_MATCH_UTILS_H_

Powered by Google App Engine
This is Rietveld 408576698