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..c43d1da339549c2a0381f59a5b408e71de5f27c5
--- /dev/null
+++ b/content/common/compositor_util.h
@@ -0,0 +1,27 @@
+// 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 {

[Khushal, 2016/08/29 20:21:14]

In cases like these I have seen that we just put it in a different namespace
rather than creating a class. I'll let piman@ suggest what's the preferred
style.

[xingliu, 2016/08/29 20:44:33]

Ok, I'll just leave it here for now.

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