Implement people search.

chrome/browser/ui/app_list/search/common/json_response_fetcher.h

 // 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_WEBSTORE_SEARCH_FETCHER_H_
-#define CHROME_BROWSER_UI_APP_LIST_SEARCH_WEBSTORE_SEARCH_FETCHER_H_
+#ifndef CHROME_BROWSER_UI_APP_LIST_SEARCH_COMMON_JSON_RESPONSE_FETCHER_H_
+#define CHROME_BROWSER_UI_APP_LIST_SEARCH_COMMON_JSON_RESPONSE_FETCHER_H_
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "net/url_request/url_fetcher_delegate.h"
 
+class GURL;
+
 namespace base {
 class DictionaryValue;
 class Value;
 }
 
 namespace net {
 class URLFetcher;
 class URLRequestContextGetter;
 }
 
 namespace app_list {
 
-// A class to fetch web store search result.
-class WebstoreSearchFetcher : public net::URLFetcherDelegate {
+// A class that fetches a JSON formatted response from a server and uses a
+// sandboxed utility process to parse it to a DictionaryValue.
+class JSONResponseFetcher : public net::URLFetcherDelegate {

[tfarina, 2013/09/04 23:49:52]

this probably can be used by WalletClient
(components/autofill/content/browser/wallet/wallet_client.h)

[rkc, 2013/09/04 23:57:30]

Wallet client's processing in OnURLFetchComplete is considerably more complex, I
am not sure if it's a great idea to even try to get it to fit into this class,
let alone way out of scope of this CL.

[tfarina, 2013/09/04 23:59:05]

sure, I was not suggesting you to try to make any change related to wallet in
this CL, since it would be pretty out of scope. ;)

  public:
   // Callback to pass back the parsed json dictionary returned from the server.
   // Invoked with NULL if there is an error.
   typedef base::Callback<void(scoped_ptr<base::DictionaryValue>)> Callback;
 
-  WebstoreSearchFetcher(const Callback& callback,
-                        net::URLRequestContextGetter* context_getter);
-  virtual ~WebstoreSearchFetcher();
+  JSONResponseFetcher(const Callback& callback,
+                      net::URLRequestContextGetter* context_getter);
+  virtual ~JSONResponseFetcher();
 
-  // Starts to fetch results for the given |query| and the language code |hl|.
-  void Start(const std::string& query, const std::string& hl);
+  // Starts to fetch results for the given |query_url|.
+  void Start(const GURL& query_url);
   void Stop();
 
  private:
   // Callbacks for SafeJsonParser.
   void OnJsonParseSuccess(scoped_ptr<base::Value> parsed_json);
   void OnJsonParseError(const std::string& error);
 
   // net::URLFetcherDelegate overrides:
   virtual void OnURLFetchComplete(const net::URLFetcher* source) OVERRIDE;
 
   Callback callback_;
   net::URLRequestContextGetter* context_getter_;
 
   scoped_ptr<net::URLFetcher> fetcher_;
-  base::WeakPtrFactory<WebstoreSearchFetcher> weak_factory_;
+  base::WeakPtrFactory<JSONResponseFetcher> weak_factory_;
 
-  DISALLOW_COPY_AND_ASSIGN(WebstoreSearchFetcher);
+  DISALLOW_COPY_AND_ASSIGN(JSONResponseFetcher);
 };
 
 }  // namespace app_list
 
-#endif  // CHROME_BROWSER_UI_APP_LIST_SEARCH_WEBSTORE_SEARCH_FETCHER_H_
+#endif  // CHROME_BROWSER_UI_APP_LIST_SEARCH_COMMON_JSON_RESPONSE_FETCHER_H_