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

Unified Diff: chrome/browser/ui/app_list/search/webstore/webstore_provider.cc

Issue 1133543003: Clean up app list webstore_provider_browsertest. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 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/webstore/webstore_provider.cc
diff --git a/chrome/browser/ui/app_list/search/webstore/webstore_provider.cc b/chrome/browser/ui/app_list/search/webstore/webstore_provider.cc
index 027999d8f30ff2000f8c71800baeef4bf861a54c..0cb781f8e7ac7217b47adf6b80f882a2fee9d113 100644
--- a/chrome/browser/ui/app_list/search/webstore/webstore_provider.cc
+++ b/chrome/browser/ui/app_list/search/webstore/webstore_provider.cc
@@ -54,8 +54,10 @@ extensions::Manifest::Type ParseItemType(const std::string& item_type_str) {
WebstoreProvider::WebstoreProvider(Profile* profile,
AppListControllerDelegate* controller)
- : WebserviceSearchProvider(profile),
- controller_(controller){}
+ : WebserviceSearchProvider(profile),
+ controller_(controller),
+ query_pending_(false) {
+}
WebstoreProvider::~WebstoreProvider() {}
@@ -84,6 +86,7 @@ void WebstoreProvider::Start(bool /*is_voice_query*/,
profile_->GetRequestContext()));
}
+ query_pending_ = true;
StartThrottledQuery(base::Bind(&WebstoreProvider::StartQuery,
base::Unretained(this)));
@@ -111,6 +114,7 @@ void WebstoreProvider::OnWebstoreSearchFetched(
scoped_ptr<base::DictionaryValue> json) {
ProcessWebstoreSearchResults(json.get());
cache_->Put(WebserviceCache::WEBSTORE, query_, json.Pass());
+ query_pending_ = false;
if (!webstore_search_fetched_callback_.is_null())
webstore_search_fetched_callback_.Run();

Powered by Google App Engine
This is Rietveld 408576698