Factor out AutocompleteMatch creation from BookmarkProvider

components/bookmarks/browser/titled_url_match.h

Index: components/bookmarks/browser/titled_url_match.h
diff --git a/components/bookmarks/browser/titled_url_match.h b/components/bookmarks/browser/titled_url_match.h
index 767a1907d0613dcc90a0c423bedf52f2ecd7b797..6781139d4b7b169f4254978ddfc3a93da00303c8 100644
--- a/components/bookmarks/browser/titled_url_match.h
+++ b/components/bookmarks/browser/titled_url_match.h
@@ -11,6 +11,8 @@
 #include <utility>
 #include <vector>
 
+#include "base/strings/string16.h"
+
 namespace bookmarks {
 
 class TitledUrlNode;
@@ -36,6 +38,12 @@ struct TitledUrlMatch {
       const MatchPositions& match_positions,
       const std::vector<size_t>& offsets);
 
+  // Removes leading spaces from |title| before displaying, otherwise it looks
+  // funny.  In the process, corrects |title_match_positions| so the correct
+  // characters are highlighted.
+  static void CorrectTitleAndMatchPositions(
+      base::string16* title, MatchPositions* title_match_positions);

[Mark P, 2016/12/23 22:38:49]

nit: This parameter spacing looks odd to me.  Did you run this changelist
through git cl format?

[mattreynolds, 2017/01/04 00:58:02]

Done.

+
   // The matching node of a query.
   const TitledUrlNode* node;