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

Side by Side Diff: third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.h

Issue 2891393002: [scheduler] Discard anomalously long tasks and bump metrics version. (Closed)
Patch Set: Addressed comments from isherman@ Created 3 years, 6 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 unified diff | Download patch
« no previous file with comments | « no previous file | third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #ifndef THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_RENDERER_RENDERER_SCHEDULER _IMPL_H_ 5 #ifndef THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_RENDERER_RENDERER_SCHEDULER _IMPL_H_
6 #define THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_RENDERER_RENDERER_SCHEDULER _IMPL_H_ 6 #define THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_RENDERER_RENDERER_SCHEDULER _IMPL_H_
7 7
8 #include "base/atomicops.h" 8 #include "base/atomicops.h"
9 #include "base/gtest_prod_util.h" 9 #include "base/gtest_prod_util.h"
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 390 matching lines...) Expand 10 before | Expand all | Expand 10 after
401 const TaskQueuePolicy& new_task_queue_policy) const; 401 const TaskQueuePolicy& new_task_queue_policy) const;
402 402
403 static const char* ExpensiveTaskPolicyToString( 403 static const char* ExpensiveTaskPolicyToString(
404 ExpensiveTaskPolicy expensive_task_policy); 404 ExpensiveTaskPolicy expensive_task_policy);
405 405
406 bool ShouldDisableThrottlingBecauseOfAudio(base::TimeTicks now); 406 bool ShouldDisableThrottlingBecauseOfAudio(base::TimeTicks now);
407 407
408 void AddQueueToWakeUpBudgetPool(TaskQueue* queue); 408 void AddQueueToWakeUpBudgetPool(TaskQueue* queue);
409 409
410 void RecordTaskMetrics(TaskQueue::QueueType queue_type, 410 void RecordTaskMetrics(TaskQueue::QueueType queue_type,
411 base::TimeDelta duration); 411 base::TimeTicks start_time,
412 base::TimeTicks end_time);
412 413
413 void RecordTaskDurationPerQueueType(TaskQueue::QueueType queue_type, 414 void RecordTaskDurationPerQueueType(TaskQueue::QueueType queue_type,
414 base::TimeDelta duration); 415 base::TimeDelta duration);
415 416
416 SchedulerHelper helper_; 417 SchedulerHelper helper_;
417 IdleHelper idle_helper_; 418 IdleHelper idle_helper_;
418 IdleCanceledDelayedTaskSweeper idle_canceled_delayed_task_sweeper_; 419 IdleCanceledDelayedTaskSweeper idle_canceled_delayed_task_sweeper_;
419 std::unique_ptr<TaskQueueThrottler> task_queue_throttler_; 420 std::unique_ptr<TaskQueueThrottler> task_queue_throttler_;
420 RenderWidgetSignals render_widget_scheduler_signals_; 421 RenderWidgetSignals render_widget_scheduler_signals_;
421 422
(...skipping 149 matching lines...) Expand 10 before | Expand all | Expand 10 after
571 572
572 base::WeakPtrFactory<RendererSchedulerImpl> weak_factory_; 573 base::WeakPtrFactory<RendererSchedulerImpl> weak_factory_;
573 574
574 DISALLOW_COPY_AND_ASSIGN(RendererSchedulerImpl); 575 DISALLOW_COPY_AND_ASSIGN(RendererSchedulerImpl);
575 }; 576 };
576 577
577 } // namespace scheduler 578 } // namespace scheduler
578 } // namespace blink 579 } // namespace blink
579 580
580 #endif // THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_RENDERER_RENDERER_SCHEDU LER_IMPL_H_ 581 #endif // THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_RENDERER_RENDERER_SCHEDU LER_IMPL_H_
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698