CC: Plumb LayerSettings parameter for cc::Layer construction.

content/public/browser/android/compositor.h

 // Copyright (c) 2012 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 CONTENT_PUBLIC_BROWSER_ANDROID_COMPOSITOR_H_
 #define CONTENT_PUBLIC_BROWSER_ANDROID_COMPOSITOR_H_
 
 #include "base/callback.h"
 #include "cc/resources/ui_resource_bitmap.h"
 #include "content/common/content_export.h"
 #include "ui/android/resources/ui_resource_provider.h"
 #include "ui/gfx/geometry/rect.h"
 #include "ui/gfx/geometry/size.h"
 #include "ui/gfx/native_widget_types.h"
 
 class SkBitmap;
 
 namespace cc {
 class Layer;
+class LayerSettings;
 }
 
 namespace gfx {
 class JavaBitmap;
 }
 
 namespace ui {
 class ResourceManager;
 class UIResourceProvider;
 }
 
 namespace content {
 class CompositorClient;
 
 // An interface to the browser-side compositor.
 class CONTENT_EXPORT Compositor {
  public:
   virtual ~Compositor() {}
 
   // Performs the global initialization needed before any compositor
   // instance can be used. This should be called only once.
   static void Initialize();
 
   // Creates and returns a compositor instance.  |root_window| needs to outlive
   // the compositor as it manages callbacks on the compositor.
   static Compositor* Create(CompositorClient* client,
                             gfx::NativeWindow root_window);
 
+  static const cc::LayerSettings& LayerSettings();
+  static void SetLayerSettings(const cc::LayerSettings& settings);

[boliu, 2015/05/18 15:39:31]

I think nothing calls the setter? Remove it?

[loyso (OOO), 2015/05/19 01:03:44]

Yes. That was a question. We have --enable-ui-compositor-timelines for ui::
compositor to switch that bool on. It looks like we need a separate command line
flag for android compositor. --enable-android-compositor-timelines?

[loyso (OOO), 2015/05/20 02:48:53]

Now we use switches::kEnableAndroidCompositorAnimationTimelines and we call the
setter from content/browser/android/content_startup_flags.cc
Is that the right place to do so?

+
   // Attaches the layer tree.
   virtual void SetRootLayer(scoped_refptr<cc::Layer> root) = 0;
 
   // Set the scale factor from DIP to pixel.
   virtual void setDeviceScaleFactor(float factor) = 0;
 
   // Set the output surface bounds.
   virtual void SetWindowBounds(const gfx::Size& size) = 0;
 
   // Set the output surface which the compositor renders into.
@@ skipping 12 matching lines @@
   // Returns the resource manager associated with the compositor.
   virtual ui::ResourceManager& GetResourceManager() = 0;
 
  protected:
   Compositor() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_ANDROID_COMPOSITOR_H_