Initial commit of the blimp/ folder and target

blimp/client/compositor/blimp_compositor_android.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_CLIENT_COMPOSITOR_BLIMP_COMPOSITOR_ANDROID_H_
+#define BLIMP_CLIENT_COMPOSITOR_BLIMP_COMPOSITOR_ANDROID_H_
+
+#include "base/android/jni_android.h"
+#include "base/macros.h"
+#include "blimp/client/compositor/blimp_compositor.h"
+#include "ui/gfx/geometry/size.h"
+#include "ui/gfx/native_widget_types.h"
+
+namespace base {
+class SingleThreadTaskRunner;
+class Thread;
+}
+
+namespace cc {
+class LayerTreeHost;
+}
+
+namespace blimp {
+
+// An Android specific version of the BlimpCompositor.  This class builds a
+// gfx::AcceleratedWidget out of an Android SurfaceView's surface.
+class BlimpCompositorAndroid : public BlimpCompositor {
+ public:
+  // |physical_size| is the total display area including system decorations (see
+  // android.view.Display.getRealSize()).  |display_size| is the total display
+  // area not including system decorations (see android.view.Display.getSize()).
+  // |dp_to_pixel| is the scale factor that is required to move from dp
+  // (device pixels) to pixels.

[Wez, 2015/09/03 18:26:11]

See comment on the BlimpView and note that you've called dp_to_pixel
"device_scale_factor" here, which is confusing!

[David Trainor- moved to gerrit, 2015/09/03 19:06:01]

Done.

+  static scoped_ptr<BlimpCompositorAndroid> Create(
+      const gfx::Size& physical_size,
+      const gfx::Size& display_size,
+      float device_scale_factor);
+
+  ~BlimpCompositorAndroid() override;
+
+  // Takes a reference to the ANativeWindow backing |jsurface|, to use to render
+  // to, and releases any previously-held reference.
+  void SetSurface(JNIEnv* env, jobject jsurface);
+
+ protected:
+  // |physical_size| is the total display area including system decorations (see
+  // android.view.Display.getRealSize()).  |display_size| is the total display
+  // area not including system decorations (see android.view.Display.getSize()).
+  // |dp_to_pixel| is the scale factor that is required to move from dp
+  // (device pixels) to pixels.

[Wez, 2015/09/03 18:26:11]

This is a copy of the comment on Create() - it doesn't describe any of the
actual parameters of this ctor, at least not using their actual names!

[David Trainor- moved to gerrit, 2015/09/03 19:06:01]

Yeah my bad.  Fixed this.

+  BlimpCompositorAndroid(const gfx::Size& device_size,
+                         bool real_size_supported,
+                         float device_scale_factor);
+
+  // BlimpCompositor implementation.
+  gfx::AcceleratedWidget GetWindow() override;
+  void GenerateLayerTreeSettings(cc::LayerTreeSettings* settings);
+
+ private:
+  // Used to determine tile size for the compositor's rastered tiles. For a
+  // device of width X height |portrait_width_| will be min(width, height) and
+  // |landscape_width_| will be max(width, height).
+  int portrait_width_;
+  int landscape_width_;
+
+  // True if the |portrait_width_| and |landscape_width_| represent the device's
+  // physical dimensions, including any area occupied by system decorations.
+  bool real_size_supported_;
+
+  gfx::AcceleratedWidget window_;
+
+  DISALLOW_COPY_AND_ASSIGN(BlimpCompositorAndroid);
+};
+
+}  // namespace blimp
+
+#endif  // BLIMP_CLIENT_COMPOSITOR_BLIMP_COMPOSITOR_ANDROID_H_