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

Unified Diff: content/renderer/render_thread_impl.cc

Issue 847883002: Reland "Throttle resource message requests during user interaction" (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Tweak Created 5 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: content/renderer/render_thread_impl.cc
diff --git a/content/renderer/render_thread_impl.cc b/content/renderer/render_thread_impl.cc
index 87f14c3d941598cbde5140d396d4f52a65d20d52..182d95837808d8033334b26d72ed2a832e484e71 100644
--- a/content/renderer/render_thread_impl.cc
+++ b/content/renderer/render_thread_impl.cc
@@ -104,6 +104,7 @@
#include "content/renderer/render_view_impl.h"
#include "content/renderer/renderer_blink_platform_impl.h"
#include "content/renderer/scheduler/renderer_scheduler.h"
+#include "content/renderer/scheduler/throttled_message_sender.h"
#include "content/renderer/service_worker/embedded_worker_context_message_filter.h"
#include "content/renderer/service_worker/embedded_worker_dispatcher.h"
#include "content/renderer/shared_worker/embedded_shared_worker_stub.h"
@@ -195,6 +196,8 @@ namespace {
const int64 kInitialIdleHandlerDelayMs = 1000;
const int64 kLongIdleHandlerDelayMs = 30*1000;
+const int kMaxResourceMessageRequestsPerSecondWhenThrottled = 60 * 4;
picksi1 2015/01/13 11:24:44 Can you add a comment to explain how you got to th
jdduke (slow) 2015/01/13 16:18:11 Yup, done.
+
// Maximum allocation size allowed for image scaling filters that
// require pre-scaling. Skia will fallback to a filter that doesn't
// require pre-scaling if the default filter would require an
@@ -469,6 +472,10 @@ void RenderThreadImpl::Init() {
thread_safe_sender()));
renderer_scheduler_ = RendererScheduler::Create();
channel()->SetListenerTaskRunner(renderer_scheduler_->DefaultTaskRunner());
+ throttled_resource_sender_.reset(new ThrottledMessageSender(
+ static_cast<RenderThread*>(this), renderer_scheduler_.get(),
+ kMaxResourceMessageRequestsPerSecondWhenThrottled));
+ resource_dispatcher()->set_message_sender(throttled_resource_sender_.get());
embedded_worker_dispatcher_.reset(new EmbeddedWorkerDispatcher());
media_stream_center_ = NULL;

Powered by Google App Engine
This is Rietveld 408576698