Unified BeginFrame scheduling

cc/scheduler/scheduler_settings.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CC_SCHEDULER_SCHEDULER_SETTINGS_H_
 #define CC_SCHEDULER_SCHEDULER_SETTINGS_H_
 
 #include "base/memory/ref_counted.h"
 #include "base/values.h"
 #include "cc/base/cc_export.h"
 
 namespace base {
 namespace debug {
 class ConvertableToTraceFormat;
 }
 }
 
 namespace cc {
 class LayerTreeSettings;
 
 class CC_EXPORT SchedulerSettings {
  public:
   SchedulerSettings();
   explicit SchedulerSettings(const LayerTreeSettings& settings);
   ~SchedulerSettings();
 
-  bool begin_frame_scheduling_enabled;
+  bool begin_frame_receiver;
+  bool begin_frame_publisher;
+  bool using_platform_vsync;

[brianderson, 2014/08/28 02:48:07]

Can you use begin_frame_receiver to represent using_platform_vsync, instead of
having two settings?

I think it makes sense for us to consider Android's and Mac's Browser as both a
BeginFrame receiver and publisher.

[simonhong, 2014/09/03 21:02:22]

I didn't think of using both option!
Using both in Android and Mac looks good.
Removed.

   bool main_frame_before_draw_enabled;
   bool main_frame_before_activation_enabled;
   bool impl_side_painting;
   bool timeout_and_draw_when_animation_checkerboards;
   int maximum_number_of_failed_draws_before_draw_is_forced_;
   bool using_synchronous_renderer_compositor;
   bool throttle_frame_production;
 
   scoped_refptr<base::debug::ConvertableToTraceFormat> AsValue() const;
 };
 
 }  // namespace cc
 
 #endif  // CC_SCHEDULER_SCHEDULER_SETTINGS_H_