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

Unified Diff: net/base/prioritized_dispatcher.cc

Issue 9113022: Adds PriorityQueue and PrioritizedDispatcher. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Applied review. Created 8 years, 11 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: net/base/prioritized_dispatcher.cc
diff --git a/net/base/prioritized_dispatcher.cc b/net/base/prioritized_dispatcher.cc
new file mode 100644
index 0000000000000000000000000000000000000000..0e6207dcef03284c762322b169a8798e4180acbb
--- /dev/null
+++ b/net/base/prioritized_dispatcher.cc
@@ -0,0 +1,104 @@
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
mmenke 2012/01/06 17:54:37 nit: Update year
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "net/base/prioritized_dispatcher.h"
+
+#include "base/logging.h"
+
+namespace net {
+
+PrioritizedDispatcher::Limits::Limits(Priority num_priorities,
+ size_t total_jobs)
+ : total_jobs(total_jobs), reserved_slots(num_priorities) {}
+
+PrioritizedDispatcher::Limits::~Limits() {}
+
+PrioritizedDispatcher::PrioritizedDispatcher(const Limits& limits)
+ : queue_(limits.reserved_slots.size()),
+ max_running_jobs_(limits.reserved_slots.size()),
+ num_running_jobs_(0) {
+ size_t total = 0;
+ for (size_t i = limits.reserved_slots.size(); i > 0; --i) {
+ total += limits.reserved_slots[i - 1];
+ max_running_jobs_[i - 1] = total;
+ }
+ // Unreserved slots are available for all priorities.
+ DCHECK_LE(total, limits.total_jobs) << "sum(reserved_slots) <= total_jobs";
+ size_t spare = limits.total_jobs - total;
+ for (size_t i = 0; i < max_running_jobs_.size(); ++i) {
+ max_running_jobs_[i] += spare;
+ }
+}
+
+PrioritizedDispatcher::~PrioritizedDispatcher() {}
+
+PrioritizedDispatcher::Handle PrioritizedDispatcher::Add(
+ Job* job, Priority priority) {
+ DCHECK(job);
+ DCHECK_LT(priority, num_priorities());
+ if (num_running_jobs_ < max_running_jobs_[priority]) {
+ ++num_running_jobs_;
+ job->Start();
+ return Handle();
+ }
+ return queue_.Insert(job, priority);
+}
+
+void PrioritizedDispatcher::Cancel(const Handle& handle) {
+ queue_.Erase(handle);
+}
+
+PrioritizedDispatcher::Job* PrioritizedDispatcher::EvictOldestLowest() {
+ Handle handle = queue_.FirstMax();
+ if (handle.is_null())
+ return NULL;
+ Job* job = handle.value();
+ Cancel(handle);
+ return job;
+}
+
+PrioritizedDispatcher::Handle PrioritizedDispatcher::ChangePriority(
+ const Handle& handle, Priority priority) {
+ DCHECK(!handle.is_null());
+ DCHECK_LT(priority, num_priorities());
+ DCHECK_GE(num_running_jobs_, max_running_jobs_[handle.priority()]) <<
+ "Job should not be in queue when limits permit it to start.";
+ DCHECK_LE(queue_.size(), max_queued_jobs_);
+
+ if (handle.priority() == priority)
+ return handle;
+
+ if (DispatchJob(handle, priority))
+ return Handle();
+ Job* job = handle.value();
+ queue_.Erase(handle);
+ return queue_.Insert(job, priority);
+}
+
+void PrioritizedDispatcher::OnJobFinished() {
+ DCHECK_GT(num_running_jobs_, 0u);
+ --num_running_jobs_;
+ Handle handle = queue_.FirstMin();
+ if (handle.is_null()) {
+ DCHECK_EQ(0u, queue_.size());
+ return;
+ }
+ DispatchJob(handle, handle.priority());
+}
+
+bool PrioritizedDispatcher::DispatchJob(const Handle& handle,
+ Priority job_priority) {
+ DCHECK_LT(job_priority, num_priorities());
+ if (num_running_jobs_ >= max_running_jobs_[job_priority])
+ return false;
+ Job* job = handle.value();
+ queue_.Erase(handle);
+ ++num_running_jobs_;
+ job->Start();
+ return true;
+}
+
+} // namespace net
+
+

Powered by Google App Engine
This is Rietveld 408576698