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

Unified Diff: net/request_throttler/request_throttler_manager.cc

Issue 4194001: Implement exponential back-off mechanism and enforce it at the URLRequestHttpJob level. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 10 years, 1 month 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: net/request_throttler/request_throttler_manager.cc
===================================================================
--- net/request_throttler/request_throttler_manager.cc (revision 0)
+++ net/request_throttler/request_throttler_manager.cc (revision 0)
@@ -0,0 +1,115 @@
+// Copyright (c) 2010 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 "net/request_throttler/request_throttler_manager.h"
+
+#include "base/string_util.h"
+
+const unsigned int RequestThrottlerManager::kMaximumNumberOfEntries = 1500;
+const unsigned int RequestThrottlerManager::kRequestsBetweenCollecting = 200;
+
+scoped_refptr<RequestThrottlerEntryInterface>
+ RequestThrottlerManager::RegisterRequestUrl(const GURL &url) {
+ // Normalize the url.
+ std::string url_id = GetIdFromUrl(url);
+
+ AutoLock auto_lock(lock_);
+
+ // Periodically garbage collect old entries.
+ GarbageCollectEntriesIfNecessary();
+
+ // Find the entry in the map or create it.
+ scoped_refptr<RequestThrottlerEntry>& entry = url_entries_[url_id];
+ if (entry == NULL)
+ entry = new RequestThrottlerEntry();
+
+ return entry;
+}
+
+RequestThrottlerManager::RequestThrottlerManager()
+ : requests_since_last_gc_(0) {
+}
+
+RequestThrottlerManager::~RequestThrottlerManager() {
+ // Delete all entries.
+ url_entries_.clear();
+}
+
+std::string RequestThrottlerManager::GetIdFromUrl(const GURL& url) {
+ std::string url_id;
+ url_id += url.scheme();
+ url_id += "://";
+ url_id += url.host();
+ url_id += url.path();
+
+ return StringToLowerASCII(url_id);
+}
+
+void RequestThrottlerManager::GarbageCollectEntries() {
+ lock_.AssertAcquired();
+
+ UrlEntryMap::iterator i = url_entries_.begin();
+
+ while (i != url_entries_.end()) {
+ if ((i->second)->IsEntryOutdated()) {
+ url_entries_.erase(i++);
+ } else {
+ ++i;
+ }
+ }
+
+ // In case something broke we want to make sure not to grow indefinitely.
+ while (url_entries_.size() > kMaximumNumberOfEntries) {
+ url_entries_.erase(url_entries_.begin());
+ }
+}
+
+void RequestThrottlerManager::GarbageCollectEntriesIfNecessary() {
+ lock_.AssertAcquired();
+
+ requests_since_last_gc_++;
+ if (requests_since_last_gc_ < kRequestsBetweenCollecting)
+ return;
+
+ requests_since_last_gc_ = 0;
+ GarbageCollectEntries();
+}
+
+void RequestThrottlerManager::NotifyRequestBodyWasMalformed(const GURL& url) {
+ // Normalize the url.
+ std::string url_id = GetIdFromUrl(url);
+
+ AutoLock auto_lock(lock_);
+
+ UrlEntryMap::iterator i = url_entries_.find(url_id);
+ if (i != url_entries_.end()) {
+ i->second->ReceivedContentWasMalformed();
+ }
+}
+
+void RequestThrottlerManager::OverrideEntry(
+ const GURL& url,
+ const scoped_refptr<RequestThrottlerEntry>& entry) {
+ if (entry == NULL)
+ return;
+
+ // Normalize the url.
+ std::string url_id = GetIdFromUrl(url);
+
+ AutoLock auto_lock(lock_);
+
+ // Periodically garbage collect old entries.
+ GarbageCollectEntriesIfNecessary();
+
+ url_entries_[url_id] = entry;
+}
+
+void RequestThrottlerManager::EraseEntry(const GURL& url) {
+ // Normalize the url.
+ std::string url_id = GetIdFromUrl(url);
+
+ AutoLock auto_lock(lock_);
+
+ url_entries_.erase(url_id);
+}
Property changes on: net\request_throttler\request_throttler_manager.cc
___________________________________________________________________
Added: svn:eol-style
+ LF

Powered by Google App Engine
This is Rietveld 408576698