Blimp: create MessageDispatcher class.

blimp/common/compositor/blimp_layer_tree_settings.cc

 // Copyright 2015 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 "blimp/common/compositor/blimp_layer_tree_settings.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 221 matching lines @@
   settings->renderer_settings.should_clear_root_render_pass = true;
 
   // TODO(danakj): Only do this on low end devices.
   settings->create_low_res_tiling = true;
 
 // TODO(dtrainor): Investigate whether or not we want to use an external
 // source here.
 // settings->use_external_begin_frame_source = true;
 
 #elif !defined(OS_MACOSX)
-  if (ui::IsOverlayScrollbarEnabled()) {
-    settings->scrollbar_animator = cc::LayerTreeSettings::THINNING;
-    settings->solid_color_scrollbar_color = SkColorSetARGB(128, 128, 128, 128);
-  } else {
-    settings->scrollbar_animator = cc::LayerTreeSettings::LINEAR_FADE;
-    settings->solid_color_scrollbar_color = SkColorSetARGB(128, 128, 128, 128);
-  }
-  settings->scrollbar_fade_delay_ms = 500;
-  settings->scrollbar_fade_resize_delay_ms = 500;
-  settings->scrollbar_fade_duration_ms = 300;
+  NOTIMPLEMENTED() << "Unresolved dependency TODO for native_theme.";

[nyquist, 2015/09/10 03:47:33]

This should be unnecessary now.

[Kevin M, 2015/09/10 18:11:17]

Done.

+  // TODO(dtrainor): Add blimp to ui/native_theme DEPS and re-enable
+  // this section.
+
+  // if (ui::IsOverlayScrollbarEnabled()) {
+  //   settings->scrollbar_animator = cc::LayerTreeSettings::THINNING;
+  //   settings->solid_color_scrollbar_color =
+  //       SkColorSetARGB(128, 128, 128, 128);
+  // } else {
+  //   settings->scrollbar_animator = cc::LayerTreeSettings::LINEAR_FADE;
+  //   settings->solid_color_scrollbar_color =
+  //       SkColorSetARGB(128, 128, 128, 128);
+  // }
+  // settings->scrollbar_fade_delay_ms = 500;
+  // settings->scrollbar_fade_resize_delay_ms = 500;
+  // settings->scrollbar_fade_duration_ms = 300;
 
   // When pinching in, only show the pinch-viewport overlay scrollbars if the
   // page scale is at least some threshold away from the minimum. i.e. don't
   // show the pinch scrollbars when at minimum scale.
-  settings->scrollbar_show_scale_threshold = 1.05f;
+  //
+  // TODO(dtrainor): Re-enable or delete this line entirely.
+  //     (LayerTreeSettings::scrollbar_show_scale_threshold does not exist.)
+  // settings->scrollbar_show_scale_threshold = 1.05f;
 #endif
 
   if (cmd.HasSwitch(switches::kEnableLowResTiling))
     settings->create_low_res_tiling = true;
   if (cmd.HasSwitch(switches::kDisableLowResTiling))
     settings->create_low_res_tiling = false;
   if (cmd.HasSwitch(cc::switches::kEnableBeginFrameScheduling))
     settings->use_external_begin_frame_source = true;
 }
 
 }  // namespace blimp