Initial commit of the blimp/ folder and target

blimp/common/compositor/blimp_layer_tree_settings.h

+// 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.
+
+#ifndef BLIMP_COMMON_COMPOSITOR_BLIMP_LAYER_TREE_SETTINGS_H_
+#define BLIMP_COMMON_COMPOSITOR_BLIMP_LAYER_TREE_SETTINGS_H_
+
+#include "blimp/common/blimp_common_export.h"
+
+namespace base {
+class CommandLine;
+}
+
+namespace cc {
+class LayerTreeSettings;
+}
+
+namespace gfx {
+class Size;
+}
+
+namespace blimp {
+
+// Contains the basic compositor settings that are shared between the common

[Wez, 2015/09/03 00:49:28]

Suggest: "Populates |settings| based on defaults & flags in |cmd|.

[David Trainor- moved to gerrit, 2015/09/03 06:33:22]

Done.

+// blimp compositors instances.  This might have to be tweaked into a message

[Wez, 2015/09/03 00:49:28]

nit: Blimp compositor instances.

[David Trainor- moved to gerrit, 2015/09/03 06:33:22]

Done.

+// protocol if it turns one one compositor needs to actually drive some specific

[Wez, 2015/09/03 00:49:28]

nit: out one

[David Trainor- moved to gerrit, 2015/09/03 06:33:22]

Done.

+// settings of the other.

[Wez, 2015/09/03 00:49:28]

This comment's a bit opaque - what are these different instances for (different
tabs...?) and why would they need to drive one another?

[David Trainor- moved to gerrit, 2015/09/03 06:33:22]

Fixed.  I meant the host driving the settings of the client.

+BLIMP_COMMON_EXPORT void PopulateCommonLayerTreeSettings(
+    cc::LayerTreeSettings* settings,
+    const base::CommandLine& cmd);

[Wez, 2015/09/03 00:49:28]

nit: command_line

Does nothing in Android/C++ init set the process-global base::CommandLine so we
can avoid needing to pass it explicitly here?

[David Trainor- moved to gerrit, 2015/09/03 06:33:22]

It does :).  Done.

+
+}  // namespace blimp
+
+#endif  // BLIMP_COMMON_COMPOSITOR_BLIMP_LAYER_TREE_SETTINGS_H_