Implement people search.

chrome/browser/ui/app_list/search/webstore/webstore_provider.cc

 // 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.
 
-#include "chrome/browser/ui/app_list/search/webstore_provider.h"
+#include "chrome/browser/ui/app_list/search/webstore/webstore_provider.h"
 
 #include <string>
 
 #include "base/bind.h"
+#include "base/callback.h"
 #include "base/metrics/field_trial.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/search/search.h"
 #include "chrome/browser/ui/app_list/search/common/json_response_fetcher.h"
 #include "chrome/browser/ui/app_list/search/search_webstore_result.h"
-#include "chrome/browser/ui/app_list/search/webstore_result.h"
+#include "chrome/browser/ui/app_list/search/webstore/webstore_result.h"
 #include "chrome/common/extensions/extension_constants.h"
 #include "url/gurl.h"
 
 namespace app_list {
 
 namespace {
 
 const char kKeyResults[] = "results";
 const char kKeyId[] = "id";
 const char kKeyLocalizedName[] = "localized_name";
 const char kKeyIconUrl[] = "icon_url";
 const size_t kMinimumQueryLength = 3u;
 
 // Returns true if the launcher should send queries to the web store server.
 bool UseWebstoreSearch() {
   const char kFieldTrialName[] = "LauncherUseWebstoreSearch";
   const char kEnable[] = "Enable";
   return base::FieldTrialList::FindFullName(kFieldTrialName) == kEnable;
 }
 
 }  // namespace
 
 WebstoreProvider::WebstoreProvider(Profile* profile,
                                    AppListControllerDelegate* controller)
-  : profile_(profile),
-    controller_(controller) {}
+  :  WebserviceSearchProvider(profile),
+     controller_(controller){}
 
 WebstoreProvider::~WebstoreProvider() {}
 
 void WebstoreProvider::Start(const base::string16& query) {
   ClearResults();
-
-  // If |query| contains sensitive data, bail out and do not create the place
-  // holder "search-web-store" result.
-  if (IsSensitiveInput(query)) {
+  if (!IsValidQuery(query)) {
     query_.clear();
     return;
   }
 
-  const std::string query_utf8 = UTF16ToUTF8(query);
-
-  if (query_utf8.size() < kMinimumQueryLength) {
-    query_.clear();
-    return;
-  }
-
-  query_ = query_utf8;
+  query_ = UTF16ToUTF8(query);
   const base::DictionaryValue* cached_result = cache_.Get(query_);
   if (cached_result) {
     ProcessWebstoreSearchResults(cached_result);
     if (!webstore_search_fetched_callback_.is_null())
       webstore_search_fetched_callback_.Run();
     return;
   }
 
-  if (UseWebstoreSearch() && chrome::IsSuggestPrefEnabled(profile_)) {
+  if (UseWebstoreSearch()) {
     if (!webstore_search_) {
       webstore_search_.reset(new JSONResponseFetcher(
           base::Bind(&WebstoreProvider::OnWebstoreSearchFetched,
                      base::Unretained(this)),
           profile_->GetRequestContext()));
     }
 
     StartThrottledQuery(base::Bind(&WebstoreProvider::StartQuery,
                                    base::Unretained(this)));
   }
 
   // Add a placeholder result which when clicked will run the user's query in a
   // browser. This placeholder is removed when the search results arrive.
   Add(scoped_ptr<ChromeSearchResult>(
-      new SearchWebstoreResult(profile_, query_utf8)).Pass());
+      new SearchWebstoreResult(profile_, query_)).Pass());
 }
 
 void WebstoreProvider::Stop() {
   if (webstore_search_)
     webstore_search_->Stop();
 }
 
 void WebstoreProvider::StartQuery() {
   // |query_| can be NULL when the query is scheduled but then canceled.
   if (!webstore_search_ || query_.empty())
@@ skipping 60 matching lines @@
   GURL icon_url(icon_url_string);
   if (!icon_url.is_valid())
     return result.Pass();
 
   result.reset(new WebstoreResult(
       profile_, app_id, localized_name, icon_url, controller_));
   return result.Pass();
 }
 
 }  // namespace app_list