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

Unified Diff: components/bookmarks/browser/bookmark_model.h

Issue 510533002: Omnibox: Make URLs of Bookmarks Searchable - Try 2 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Peter's minor comments Created 6 years, 4 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/bookmarks/browser/bookmark_model.h
diff --git a/components/bookmarks/browser/bookmark_model.h b/components/bookmarks/browser/bookmark_model.h
index 0a2d9643c89ccf84d3604de102340a0e8b2b61e2..b8d9d7e7bac96528965a168a4c2b309c635f8e16 100644
--- a/components/bookmarks/browser/bookmark_model.h
+++ b/components/bookmarks/browser/bookmark_model.h
@@ -66,9 +66,7 @@ class BookmarkModel : public KeyedService {
base::string16 title;
};
- // |index_urls| says whether URLs should be stored in the BookmarkIndex
- // in addition to bookmark titles.
- BookmarkModel(bookmarks::BookmarkClient* client, bool index_urls);
+ explicit BookmarkModel(bookmarks::BookmarkClient* client);
virtual ~BookmarkModel();
// KeyedService:
@@ -415,10 +413,6 @@ class BookmarkModel : public KeyedService {
scoped_ptr<bookmarks::BookmarkIndex> index_;
- // True if URLs are stored in the BookmarkIndex in addition to bookmark
- // titles.
- const bool index_urls_;
-
base::WaitableEvent loaded_signal_;
// See description of IsDoingExtensiveChanges above.
« no previous file with comments | « components/bookmarks/browser/bookmark_index_unittest.cc ('k') | components/bookmarks/browser/bookmark_model.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698