Overlay interface for ozone.

ui/gfx/ozone/overlay_hal_ozone.h

Index: ui/gfx/ozone/overlay_hal_ozone.h
diff --git a/ui/gfx/ozone/overlay_hal_ozone.h b/ui/gfx/ozone/overlay_hal_ozone.h
new file mode 100644
index 0000000000000000000000000000000000000000..d6d12dfb174e4b1a7615fdecb1cfac0a2869383c
--- /dev/null
+++ b/ui/gfx/ozone/overlay_hal_ozone.h
@@ -0,0 +1,39 @@
+// Copyright 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 UI_GFX_OZONE_OVERLAY_HAL_OZONE_H_
+#define UI_GFX_OZONE_OVERLAY_HAL_OZONE_H_
+
+#include <vector>
+
+#include "base/basictypes.h"
+#include "ui/gfx/gfx_export.h"
+#include "ui/gfx/ozone/surface_factory_ozone.h"
+#include "ui/gfx/rect.h"
+
+namespace gfx {
+
+class GFX_EXPORT OverlayHALOzone {

[rjkroege, 2014/02/27 23:18:19]

More explanation comment.

And I don't like the name. Sorry. :-) OverlayCandidatesOzone?

[alexst (slow to review), 2014/02/28 18:59:34]

Done.

+ public:
+  struct OverlaySurfaceCandidate {

[rjkroege, 2014/02/27 23:18:19]

would we ever want to tag this with the gfx::AcceleratedWidget associated with
this overlay?

[alexst (slow to review), 2014/02/28 18:59:34]

Sometimes there is no widget yet. The compositor may be asking if a
configuration of background|webgl|html controls is possible and if it is, it'll
place the html controls into a new buffer.

+    OverlaySurfaceCandidate();
+    ~OverlaySurfaceCandidate();
+
+    SurfaceFactoryOzone::OverlayTransform transform;
+    SurfaceFactoryOzone::BufferFormat format;
+    gfx::Rect rect;

[rjkroege, 2014/02/27 23:18:19]

This is int coordinates right? Would we ever want a float?

[alexst (slow to review), 2014/02/28 18:59:34]

This is pixel positions, both HWC and DRM have it as int's.

+
+    // To be modified by the HAL implementer.
+    bool overlay_handled;
+  };
+
+  typedef std::vector<OverlaySurfaceCandidate> OverlaySurfaceCandidateList;
+  virtual void CheckOverlaySupport(OverlaySurfaceCandidateList* surfaces) = 0;

[rjkroege, 2014/02/27 23:18:19]

aka: iCanHazOverOverlayz? (More comment please?)

And: this ought ought to have an implementation that does nothing instead of
pure virtual so that it's easier to evolve without breaking downstream
implementors?

[alexst (slow to review), 2014/02/28 18:59:34]

Done.

+
+  virtual ~OverlayHALOzone();
+};
+
+}  // namespace gfx
+
+#endif  // UI_GFX_OZONE_OVERLAY_HAL_OZONE_H_