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

Side by Side Diff: components/scheduler/renderer/web_frame_scheduler_impl.cc

Issue 1441073006: Move throttling of background timers into the renderer scheduler (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Last few changes Sami requested Created 5 years 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/scheduler/renderer/web_frame_scheduler_impl.h" 5 #include "components/scheduler/renderer/web_frame_scheduler_impl.h"
6 6
7 #include "components/scheduler/base/real_time_domain.h"
8 #include "components/scheduler/base/virtual_time_domain.h"
7 #include "components/scheduler/child/web_task_runner_impl.h" 9 #include "components/scheduler/child/web_task_runner_impl.h"
8 #include "components/scheduler/renderer/renderer_scheduler_impl.h" 10 #include "components/scheduler/renderer/renderer_scheduler_impl.h"
9 #include "components/scheduler/renderer/web_view_scheduler_impl.h" 11 #include "components/scheduler/renderer/web_view_scheduler_impl.h"
10 #include "third_party/WebKit/public/platform/WebString.h" 12 #include "third_party/WebKit/public/platform/WebString.h"
11 13
12 namespace scheduler { 14 namespace scheduler {
13 15
14 WebFrameSchedulerImpl::WebFrameSchedulerImpl( 16 WebFrameSchedulerImpl::WebFrameSchedulerImpl(
15 RendererSchedulerImpl* renderer_scheduler, 17 RendererSchedulerImpl* renderer_scheduler,
16 WebViewSchedulerImpl* parent_web_view_scheduler) 18 WebViewSchedulerImpl* parent_web_view_scheduler)
17 : renderer_scheduler_(renderer_scheduler), 19 : renderer_scheduler_(renderer_scheduler),
18 parent_web_view_scheduler_(parent_web_view_scheduler), 20 parent_web_view_scheduler_(parent_web_view_scheduler),
19 visible_(true) {} 21 visible_(true),
22 page_in_background_(false) {}
20 23
21 WebFrameSchedulerImpl::~WebFrameSchedulerImpl() { 24 WebFrameSchedulerImpl::~WebFrameSchedulerImpl() {
22 if (loading_task_queue_.get()) 25 if (loading_task_queue_.get())
23 loading_task_queue_->UnregisterTaskQueue(); 26 loading_task_queue_->UnregisterTaskQueue();
24 27
25 if (timer_task_queue_.get()) 28 if (timer_task_queue_.get()) {
29 renderer_scheduler_->throttling_helper()->Unthrottle(
30 timer_task_queue_.get());
26 timer_task_queue_->UnregisterTaskQueue(); 31 timer_task_queue_->UnregisterTaskQueue();
32 }
27 33
28 if (parent_web_view_scheduler_) 34 if (parent_web_view_scheduler_)
29 parent_web_view_scheduler_->Unregister(this); 35 parent_web_view_scheduler_->Unregister(this);
30 } 36 }
31 37
32 void WebFrameSchedulerImpl::DetachFromWebViewScheduler() { 38 void WebFrameSchedulerImpl::DetachFromWebViewScheduler() {
33 parent_web_view_scheduler_ = nullptr; 39 parent_web_view_scheduler_ = nullptr;
34 } 40 }
35 41
36 void WebFrameSchedulerImpl::setFrameVisible(bool visible) { 42 void WebFrameSchedulerImpl::setFrameVisible(bool visible) {
37 visible_ = visible; 43 visible_ = visible;
38 // TODO(alexclarke): Do something with this flag. 44 // TODO(alexclarke): Do something with this flag.
39 } 45 }
40 46
41 blink::WebTaskRunner* WebFrameSchedulerImpl::loadingTaskRunner() { 47 blink::WebTaskRunner* WebFrameSchedulerImpl::loadingTaskRunner() {
42 if (!loading_web_task_runner_) { 48 if (!loading_web_task_runner_) {
43 loading_task_queue_ = 49 loading_task_queue_ =
44 renderer_scheduler_->NewLoadingTaskRunner("frame_loading_tq"); 50 renderer_scheduler_->NewLoadingTaskRunner("frame_loading_tq");
45 loading_web_task_runner_.reset(new WebTaskRunnerImpl(loading_task_queue_)); 51 loading_web_task_runner_.reset(new WebTaskRunnerImpl(loading_task_queue_));
46 } 52 }
47 return loading_web_task_runner_.get(); 53 return loading_web_task_runner_.get();
48 } 54 }
49 55
50 blink::WebTaskRunner* WebFrameSchedulerImpl::timerTaskRunner() { 56 blink::WebTaskRunner* WebFrameSchedulerImpl::timerTaskRunner() {
51 if (!timer_web_task_runner_) { 57 if (!timer_web_task_runner_) {
52 timer_task_queue_ = 58 timer_task_queue_ =
53 renderer_scheduler_->NewTimerTaskRunner("frame_timer_tq"); 59 renderer_scheduler_->NewTimerTaskRunner("frame_timer_tq");
60 if (page_in_background_) {
61 renderer_scheduler_->throttling_helper()->Throttle(
62 timer_task_queue_.get());
63 }
54 timer_web_task_runner_.reset(new WebTaskRunnerImpl(timer_task_queue_)); 64 timer_web_task_runner_.reset(new WebTaskRunnerImpl(timer_task_queue_));
55 } 65 }
56 return timer_web_task_runner_.get(); 66 return timer_web_task_runner_.get();
57 } 67 }
58 68
59 void WebFrameSchedulerImpl::setFrameOrigin( 69 void WebFrameSchedulerImpl::setFrameOrigin(
60 const blink::WebSecurityOrigin* origin) { 70 const blink::WebSecurityOrigin* origin) {
61 DCHECK(origin); 71 DCHECK(origin);
62 origin_ = *origin; 72 origin_ = *origin;
63 // TODO(skyostil): Associate the task queues with this origin. 73 // TODO(skyostil): Associate the task queues with this origin.
64 } 74 }
65 75
76 void WebFrameSchedulerImpl::SetPageInBackground(bool page_in_background) {
77 if (page_in_background_ == page_in_background)
78 return;
79
80 page_in_background_ = page_in_background;
81
82 if (!timer_web_task_runner_)
83 return;
84
85 if (page_in_background_) {
86 renderer_scheduler_->throttling_helper()->Throttle(timer_task_queue_.get());
87 } else {
88 renderer_scheduler_->throttling_helper()->Unthrottle(
89 timer_task_queue_.get());
90 }
91 }
92
66 } // namespace scheduler 93 } // namespace scheduler
OLDNEW
« no previous file with comments | « components/scheduler/renderer/web_frame_scheduler_impl.h ('k') | components/scheduler/renderer/web_view_scheduler_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698