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

Unified Diff: cc/scheduler/scheduler_settings.cc

Issue 423773002: Unified BeginFrame scheduling (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: WIP in mac and android Created 6 years, 4 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: cc/scheduler/scheduler_settings.cc
diff --git a/cc/scheduler/scheduler_settings.cc b/cc/scheduler/scheduler_settings.cc
index ea714e34b0dbae4fd3fc49fc0c874f758e739add..6eefefd707733bfed38a6744f39bc0ebfadd9f36 100644
--- a/cc/scheduler/scheduler_settings.cc
+++ b/cc/scheduler/scheduler_settings.cc
@@ -10,7 +10,9 @@
namespace cc {
SchedulerSettings::SchedulerSettings()
- : begin_frame_scheduling_enabled(true),
+ : begin_frame_receiver(true),
+ begin_frame_publisher(false),
+ using_platform_vsync(false),
main_frame_before_draw_enabled(true),
main_frame_before_activation_enabled(false),
impl_side_painting(false),
@@ -21,7 +23,9 @@ SchedulerSettings::SchedulerSettings()
}
SchedulerSettings::SchedulerSettings(const LayerTreeSettings& settings)
- : begin_frame_scheduling_enabled(settings.begin_frame_scheduling_enabled),
+ : begin_frame_receiver(settings.begin_frame_receiver),
+ begin_frame_publisher(settings.begin_frame_publisher),
+ using_platform_vsync(settings.using_platform_vsync),
main_frame_before_draw_enabled(settings.main_frame_before_draw_enabled),
main_frame_before_activation_enabled(
settings.main_frame_before_activation_enabled),
@@ -33,6 +37,8 @@ SchedulerSettings::SchedulerSettings(const LayerTreeSettings& settings)
using_synchronous_renderer_compositor(
settings.using_synchronous_renderer_compositor),
throttle_frame_production(settings.throttle_frame_production) {
+ if (begin_frame_receiver)
+ DCHECK(!begin_frame_publisher);
}
SchedulerSettings::~SchedulerSettings() {}
@@ -41,8 +47,12 @@ scoped_refptr<base::debug::ConvertableToTraceFormat>
SchedulerSettings::AsValue() const {
scoped_refptr<base::debug::TracedValue> state =
new base::debug::TracedValue();
- state->SetBoolean("begin_frame_scheduling_enabled",
- begin_frame_scheduling_enabled);
+ state->SetBoolean("begin_frame_receiver",
+ begin_frame_receiver);
+ state->SetBoolean("begin_frame_publisher",
+ begin_frame_publisher);
+ state->SetBoolean("using_platform_vsync",
+ using_platform_vsync);
state->SetBoolean("main_frame_before_draw_enabled",
main_frame_before_draw_enabled);
state->SetBoolean("main_frame_before_activation_enabled",

Powered by Google App Engine
This is Rietveld 408576698