cc: Add command line for frame production throttle

cc/trees/layer_tree_settings.cc

 // Copyright 2011 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.
 
 #include "cc/trees/layer_tree_settings.h"
 
 #include <GLES2/gl2.h>
 #include <limits>
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/strings/string_number_conversions.h"
+#include "cc/base/switches.h"
+#include "ui/gl/gl_switches.h"
 
 namespace cc {
 
 LayerSettings::LayerSettings() : use_compositor_animation_timelines(false) {
 }
 
 LayerSettings::~LayerSettings() {
 }
 
 LayerTreeSettings::LayerTreeSettings()
@@ skipping 51 matching lines @@
       use_display_lists(false),
       verify_property_trees(false),
       gather_pixel_refs(false),
       use_compositor_animation_timelines(false),
       invert_viewport_scroll_order(false) {
 }
 
 LayerTreeSettings::~LayerTreeSettings() {}
 
 SchedulerSettings LayerTreeSettings::ToSchedulerSettings() const {
+  base::CommandLine* cmd = base::CommandLine::ForCurrentProcess();
+
   SchedulerSettings scheduler_settings;
   scheduler_settings.use_external_begin_frame_source =
       use_external_begin_frame_source;
   scheduler_settings.main_frame_before_activation_enabled =
       main_frame_before_activation_enabled;
   scheduler_settings.impl_side_painting = impl_side_painting;
   scheduler_settings.timeout_and_draw_when_animation_checkerboards =
       timeout_and_draw_when_animation_checkerboards;
   scheduler_settings.maximum_number_of_failed_draws_before_draw_is_forced_ =
       maximum_number_of_failed_draws_before_draw_is_forced_;
   scheduler_settings.using_synchronous_renderer_compositor =
       using_synchronous_renderer_compositor;
+
   scheduler_settings.throttle_frame_production =
-      !renderer_settings.disable_gpu_vsync;
+      (cmd->HasSwitch(::switches::kDisableGpuVsync)

[brianderson, 2015/06/24 01:16:45]

We try to avoid checking the command line flags directly in cc code. Can you
move this to the same place we check for disable-display-vsync and assign it to
a new member variable of LayerTreeSettings?

[Jimmy Jo, 2015/06/24 09:38:31]

Done.

+           ? false
+           : !cmd->HasSwitch(switches::kDisableCompositorVsync));
+
   scheduler_settings.main_thread_should_always_be_low_latency = false;
   scheduler_settings.background_frame_interval =
       base::TimeDelta::FromSecondsD(1.0 / background_animation_rate);
   return scheduler_settings;
 }
 
 }  // namespace cc