Refactor renderer LayerTreeSettings generating code to common.

content/common/compositor_util.h

Index: content/common/compositor_util.h
diff --git a/content/common/compositor_util.h b/content/common/compositor_util.h
new file mode 100644
index 0000000000000000000000000000000000000000..24bad471235b5c58c3797a5b70ca7e97c1e4538c
--- /dev/null
+++ b/content/common/compositor_util.h
@@ -0,0 +1,26 @@
+// Copyright 2016 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_COMMON_COMPOSITOR_UTIL_H_
+#define CONTENT_COMMON_COMPOSITOR_UTIL_H_
+
+#include "base/command_line.h"
+#include "base/macros.h"
+#include "cc/trees/layer_tree_settings.h"
+#include "content/common/content_export.h"
+
+namespace content {
+
+// CompositorUtil holds utilities functions to generate compositor instance.
+class CONTENT_EXPORT CompositorUtil {

[piman, 2016/08/29 21:11:36]

nit: CompositorUtil makes it sound like a grab-bag of stuff, which we usually
try to avoid.

How about LayerTreeSettingsFactory to make it clear what this is for?

[xingliu, 2016/08/30 00:06:27]

Renamed to LayerTreeSettingsFactory, good idea. 

+  // TODO(xingliu): Refactor LayerTreeSettings generation logic.
+  // crbug.com/577985
+ public:
+  static void SetTopControlsSettings(cc::LayerTreeSettings& settings,
+                                     const base::CommandLine& command_line);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_COMMON_COMPOSITOR_UTIL_H_