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

Side by Side Diff: components/enhanced_bookmarks/bookmark_server_search_service.h

Issue 563363002: Only set remote id during url node creation. (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Created 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_ENHANCED_BOOKMARKS_BOOKMARK_SERVER_SEARCH_SERVICE_H_ 5 #ifndef COMPONENTS_ENHANCED_BOOKMARKS_BOOKMARK_SERVER_SEARCH_SERVICE_H_
6 #define COMPONENTS_ENHANCED_BOOKMARKS_BOOKMARK_SERVER_SEARCH_SERVICE_H_ 6 #define COMPONENTS_ENHANCED_BOOKMARKS_BOOKMARK_SERVER_SEARCH_SERVICE_H_
7 7
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
11 #include "components/bookmarks/browser/base_bookmark_model_observer.h"
11 #include "components/enhanced_bookmarks/bookmark_server_service.h" 12 #include "components/enhanced_bookmarks/bookmark_server_service.h"
12 #include "net/url_request/url_fetcher.h" 13 #include "net/url_request/url_fetcher.h"
13 14
14 class BookmarkModel; 15 namespace enhanced_bookmarks {
15 16
16 namespace enhanced_bookmarks { 17 class EnhancedBookmarkModel;
17 18
18 // Sends requests to the bookmark server to search for bookmarks relevant to a 19 // Sends requests to the bookmark server to search for bookmarks relevant to a
19 // given query. Will handle one outgoing request at a time. 20 // given query. Will handle one outgoing request at a time.
20 class BookmarkServerSearchService : public BookmarkServerService { 21 class BookmarkServerSearchService : public BookmarkServerService {
21 public: 22 public:
22 BookmarkServerSearchService( 23 BookmarkServerSearchService(
23 scoped_refptr<net::URLRequestContextGetter> request_context_getter, 24 scoped_refptr<net::URLRequestContextGetter> request_context_getter,
24 ProfileOAuth2TokenService* token_service, 25 ProfileOAuth2TokenService* token_service,
25 SigninManagerBase* signin_manager, 26 SigninManagerBase* signin_manager,
26 BookmarkModel* bookmark_model); 27 EnhancedBookmarkModel* bookmark_model);
27 virtual ~BookmarkServerSearchService(); 28 virtual ~BookmarkServerSearchService();
28 29
29 // Triggers a search. The query must not be empty. A call to this method 30 // Triggers a search. The query must not be empty. A call to this method
30 // cancels any previous searches. OnChange() is garanteed to be called once 31 // cancels any previous searches. OnChange() is garanteed to be called once
31 // per query. 32 // per query.
32 void Search(const std::string& query); 33 void Search(const std::string& query);
33 34
34 // Returns the search results. The results are only available after the 35 // Returns the search results. The results are only available after the
35 // OnChange() observer methods has been sent. This method will return an empty 36 // OnChange() observer methods has been sent. This method will return an empty
36 // result otherwise. query should be a string passed to Search() previously. 37 // result otherwise. query should be a string passed to Search() previously.
37 std::vector<const BookmarkNode*> ResultForQuery(const std::string& query); 38 std::vector<const BookmarkNode*> ResultForQuery(const std::string& query);
38 39
39 protected: 40 protected:
40 41
41 virtual net::URLFetcher* CreateFetcher() OVERRIDE; 42 virtual net::URLFetcher* CreateFetcher() OVERRIDE;
42 43
43 virtual bool ProcessResponse(const std::string& response, 44 virtual bool ProcessResponse(const std::string& response,
44 bool* should_notify) OVERRIDE; 45 bool* should_notify) OVERRIDE;
45 46
46 virtual void CleanAfterFailure() OVERRIDE; 47 virtual void CleanAfterFailure() OVERRIDE;
47 48
48 // BookmarkModelObserver methods. 49 // EnhancedBookmarkModelObserver methods.
49 virtual void BookmarkNodeAdded(BookmarkModel* model, 50 virtual void EnhancedBookmarkModelLoaded() OVERRIDE{};
50 const BookmarkNode* parent, 51 virtual void EnhancedBookmarkAdded(const BookmarkNode* node) OVERRIDE;
51 int index) OVERRIDE; 52 virtual void EnhancedBookmarkRemoved(const BookmarkNode* node) OVERRIDE{};
52 virtual void BookmarkMetaInfoChanged(BookmarkModel* model, 53 virtual void EnhancedBookmarkAllUserNodesRemoved() OVERRIDE;
53 const BookmarkNode* node) OVERRIDE; 54 virtual void EnhancedBookmarkRemoteIdChanged(
55 const BookmarkNode* node,
56 const std::string& old_remote_id,
57 const std::string& remote_id) OVERRIDE;
54 58
55 private: 59 private:
56 // The search data, a map from query to a vector of stars.id. 60 // The search data, a map from query to a vector of stars.id.
57 std::map<std::string, std::vector<std::string> > searches_; 61 std::map<std::string, std::vector<std::string> > searches_;
58 std::string current_query_; 62 std::string current_query_;
59 DISALLOW_COPY_AND_ASSIGN(BookmarkServerSearchService); 63 DISALLOW_COPY_AND_ASSIGN(BookmarkServerSearchService);
60 }; 64 };
61 } // namespace enhanced_bookmarks 65 } // namespace enhanced_bookmarks
62 66
63 #endif // COMPONENTS_ENHANCED_BOOKMARKS_BOOKMARK_SERVER_SEARCH_SERVICE_H_ 67 #endif // COMPONENTS_ENHANCED_BOOKMARKS_BOOKMARK_SERVER_SEARCH_SERVICE_H_
OLDNEW
« no previous file with comments | « components/enhanced_bookmarks.gypi ('k') | components/enhanced_bookmarks/bookmark_server_search_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698