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

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

Issue 2569333003: Rename BookmarkIndex to TitledUrlIndex and BookmarkMatch to TitledUrlMatch (Closed)
Patch Set: fix bookmark_bridge.cc Created 4 years 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
« no previous file with comments | « components/bookmarks/browser/BUILD.gn ('k') | components/bookmarks/browser/bookmark_index.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/bookmarks/browser/bookmark_index.h
diff --git a/components/bookmarks/browser/bookmark_index.h b/components/bookmarks/browser/bookmark_index.h
deleted file mode 100644
index 52a510d4dbdf68001be0d3be93c5364f97bf72e2..0000000000000000000000000000000000000000
--- a/components/bookmarks/browser/bookmark_index.h
+++ /dev/null
@@ -1,94 +0,0 @@
-// Copyright 2014 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_BOOKMARKS_BROWSER_BOOKMARK_INDEX_H_
-#define COMPONENTS_BOOKMARKS_BROWSER_BOOKMARK_INDEX_H_
-
-#include <stddef.h>
-
-#include <map>
-#include <set>
-#include <string>
-#include <vector>
-
-#include "base/macros.h"
-#include "base/strings/string16.h"
-#include "components/query_parser/query_parser.h"
-
-namespace bookmarks {
-
-class TitledUrlNode;
-class TitledUrlNodeSorter;
-struct BookmarkMatch;
-
-// BookmarkIndex maintains an index of the titles and URLs of bookmarks for
-// quick look up. BookmarkIndex is owned and maintained by BookmarkModel, you
-// shouldn't need to interact directly with BookmarkIndex.
-//
-// BookmarkIndex maintains the index (index_) as a map of sets. The map (type
-// Index) maps from a lower case string to the set (type NodeSet) of
-// TitledUrlNodes that contain that string in their title or URL.
-class BookmarkIndex {
- public:
- BookmarkIndex(std::unique_ptr<TitledUrlNodeSorter> sorter);
- ~BookmarkIndex();
-
- // Invoked when a title/URL pair has been added to the model.
- void Add(const TitledUrlNode* node);
-
- // Invoked when a title/URL pair has been removed from the model.
- void Remove(const TitledUrlNode* node);
-
- // Returns up to |max_count| of matches containing each term from the text
- // |query| in either the title or the URL.
- void GetResultsMatching(const base::string16& query,
- size_t max_count,
- query_parser::MatchingAlgorithm matching_algorithm,
- std::vector<BookmarkMatch>* results);
-
- private:
- using TitledUrlNodes = std::vector<const TitledUrlNode*>;
- using TitledUrlNodeSet = std::set<const TitledUrlNode*>;
- using Index = std::map<base::string16, TitledUrlNodeSet>;
-
- // Constructs |sorted_nodes| by taking the matches in |matches| and sorting
- // them in decreasing order of typed count (if supported by the client) and
- // deduping them.
- void SortMatches(const TitledUrlNodeSet& matches,
- TitledUrlNodes* sorted_nodes) const;
-
- // Add |node| to |results| if the node matches the query.
- void AddMatchToResults(const TitledUrlNode* node,
- query_parser::QueryParser* parser,
- const query_parser::QueryNodeVector& query_nodes,
- std::vector<BookmarkMatch>* results);
-
- // Populates |matches| for the specified term. If |first_term| is true, this
- // is the first term in the query. Returns true if there is at least one node
- // matching the term.
- bool GetResultsMatchingTerm(
- const base::string16& term,
- bool first_term,
- query_parser::MatchingAlgorithm matching_algorithm,
- TitledUrlNodeSet* matches);
-
- // Returns the set of query words from |query|.
- std::vector<base::string16> ExtractQueryWords(const base::string16& query);
-
- // Adds |node| to |index_|.
- void RegisterNode(const base::string16& term, const TitledUrlNode* node);
-
- // Removes |node| from |index_|.
- void UnregisterNode(const base::string16& term, const TitledUrlNode* node);
-
- Index index_;
-
- std::unique_ptr<TitledUrlNodeSorter> sorter_;
-
- DISALLOW_COPY_AND_ASSIGN(BookmarkIndex);
-};
-
-} // namespace bookmarks
-
-#endif // COMPONENTS_BOOKMARKS_BROWSER_BOOKMARK_INDEX_H_
« no previous file with comments | « components/bookmarks/browser/BUILD.gn ('k') | components/bookmarks/browser/bookmark_index.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698