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

Unified Diff: chrome/browser/ui/app_list/search/people/people_provider.h

Issue 23874015: Implement people search. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/app_list/search/people/people_provider.h
diff --git a/chrome/browser/ui/app_list/search/people/people_provider.h b/chrome/browser/ui/app_list/search/people/people_provider.h
new file mode 100644
index 0000000000000000000000000000000000000000..4fb634a2b14fd16cbe5926cc61a26d63ed885ae1
--- /dev/null
+++ b/chrome/browser/ui/app_list/search/people/people_provider.h
@@ -0,0 +1,94 @@
+// Copyright 2013 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 CHROME_BROWSER_UI_APP_LIST_SEARCH_PEOPLE_PEOPLE_PROVIDER_H_
+#define CHROME_BROWSER_UI_APP_LIST_SEARCH_PEOPLE_PEOPLE_PROVIDER_H_
+
+#include "base/basictypes.h"
+#include "base/callback_forward.h"
+#include "base/memory/scoped_ptr.h"
+#include "chrome/browser/ui/app_list/search/common/webservice_search_provider.h"
+#include "google_apis/gaia/oauth2_token_service.h"
+#include "url/gurl.h"
+
+class AppListControllerDelegate;
+
+namespace base {
+class DictionaryValue;
+}
+
+namespace app_list {
+
+namespace test {
+class PeopleProviderTest;
+}
+
+class ChromeSearchResult;
+class JSONResponseFetcher;
+
+// PeopleProvider fetches search results from the web store server.
+// A "Search in web store" result will be returned if the server does not
+// return any results.
+class PeopleProvider : public WebserviceSearchProvider,
+ public OAuth2TokenService::Consumer {
xiyuan 2013/09/04 23:15:41 nit: alignment
rkc 2013/09/04 23:48:02 Done.
+ public:
+ explicit PeopleProvider(Profile* profile);
+ virtual ~PeopleProvider();
+
+ // SearchProvider overrides:
+ virtual void Start(const base::string16& query) OVERRIDE;
+ virtual void Stop() OVERRIDE;
+
+ // OAuth2TokenService::Consumer implementation.
xiyuan 2013/09/04 23:15:41 nit: Use overrides: since it is used above.
rkc 2013/09/04 23:48:02 Done.
+ virtual void OnGetTokenSuccess(
+ const OAuth2TokenService::Request* request,
+ const std::string& access_token,
+ const base::Time& expiration_time) OVERRIDE;
+ virtual void OnGetTokenFailure(
+ const OAuth2TokenService::Request* request,
+ const GoogleServiceAuthError& error) OVERRIDE;
+
+ private:
+ friend class app_list::test::PeopleProviderTest;
+
+ // Start a request for getting the access token for people search.
+ void RequestAccessToken();
+ // Invalidate our existing token so a new one can be fetched.
+ void InvalidateToken();
+
+ // Get the full people search query URL. This URL includes the OAuth refresh
+ // token for authenticating the current user.
+ GURL GetQueryUrl(const std::string& query);
+
+ // Start the search request with |query_|.
+ void StartQuery();
+
+ void OnPeopleSearchFetched(scoped_ptr<base::DictionaryValue> json);
+ void ProcessPeopleSearchResults(const base::DictionaryValue* json);
+ scoped_ptr<ChromeSearchResult> CreateResult(
+ const base::DictionaryValue& dict);
+
+ // Setup the various variables that we override for testing.
+ void SetupForTest(const base::Closure& people_search_fetched_callback,
+ const std::string& people_search_url);
+
+ scoped_ptr<JSONResponseFetcher> people_search_;
+ base::Closure people_search_fetched_callback_;
+
+ std::string access_token_;
+ scoped_ptr<OAuth2TokenService::Request> access_token_request_;
+ OAuth2TokenService::ScopeSet oauth2_scope_;
+
+ // The current query.
+ std::string query_;
+ std::string people_search_url_;
xiyuan 2013/09/04 23:15:41 Store url in GURL.
rkc 2013/09/04 23:48:02 Done.
+
+ bool running_as_test_;
+
+ DISALLOW_COPY_AND_ASSIGN(PeopleProvider);
+};
+
+} // namespace app_list
+
+#endif // CHROME_BROWSER_UI_APP_LIST_SEARCH_PEOPLE_PEOPLE_PROVIDER_H_

Powered by Google App Engine
This is Rietveld 408576698