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

Unified Diff: components/ntp_snippets/remote/remote_suggestions_fetcher.h

Issue 2714863004: [Remote suggestions] Move the throttler into the Scheduler (Closed)
Patch Set: Created 3 years, 10 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: components/ntp_snippets/remote/remote_suggestions_fetcher.h
diff --git a/components/ntp_snippets/remote/remote_suggestions_fetcher.h b/components/ntp_snippets/remote/remote_suggestions_fetcher.h
index dc023dc705ab581b30c77acf80390d6530202f8c..e7e9827eb0891571e7ad6b5f56bdd10c2e6d1a4f 100644
--- a/components/ntp_snippets/remote/remote_suggestions_fetcher.h
+++ b/components/ntp_snippets/remote/remote_suggestions_fetcher.h
@@ -20,7 +20,6 @@
#include "components/ntp_snippets/remote/json_request.h"
#include "components/ntp_snippets/remote/remote_suggestion.h"
#include "components/ntp_snippets/remote/request_params.h"
-#include "components/ntp_snippets/remote/request_throttler.h"
#include "components/ntp_snippets/status.h"
#include "components/translate/core/browser/language_model.h"
#include "net/url_request/url_request_context_getter.h"
@@ -161,8 +160,6 @@ class RemoteSuggestionsFetcher : public OAuth2TokenService::Consumer,
FetchedCategoriesVector* categories,
const base::Time& fetch_time);
- bool DemandQuotaForRequest(bool interactive_request);
-
// Authentication for signed-in users.
SigninManagerBase* signin_manager_;
OAuth2TokenService* token_service_;
@@ -199,11 +196,6 @@ class RemoteSuggestionsFetcher : public OAuth2TokenService::Consumer,
// Classifier that tells us how active the user is. Not owned.
const UserClassifier* user_classifier_;
- // Request throttlers for limiting requests for different classes of users.
- RequestThrottler request_throttler_rare_ntp_user_;
- RequestThrottler request_throttler_active_ntp_user_;
- RequestThrottler request_throttler_active_suggestions_consumer_;
-
// Info on the last finished fetch.
std::string last_status_;
std::string last_fetch_json_;

Powered by Google App Engine
This is Rietveld 408576698