Add NativeViewGLSurfaceEGL::CommitOverlayPlanes support for Android

ui/gl/gl_surface_egl.cc

Index: ui/gl/gl_surface_egl.cc
diff --git a/ui/gl/gl_surface_egl.cc b/ui/gl/gl_surface_egl.cc
index 8352ff6b1f501b4a7091092720272ccd4ce7476a..976052a2f4602ad05430fb227670fdda2897bd30 100644
--- a/ui/gl/gl_surface_egl.cc
+++ b/ui/gl/gl_surface_egl.cc
@@ -677,6 +677,11 @@ gfx::SwapResult NativeViewGLSurfaceEGL::SwapBuffers() {
   }
 #endif
 
+  if (!CommitAndClearPendingOverlays()) {
+    DVLOG(1) << "Failed to commit pending overlay planes.";
+    return gfx::SwapResult::SWAP_FAILED;
+  }
+
   if (!eglSwapBuffers(GetDisplay(), surface_)) {
     DVLOG(1) << "eglSwapBuffers failed with error "
              << GetLastEGLErrorString();
@@ -747,6 +752,10 @@ gfx::SwapResult NativeViewGLSurfaceEGL::PostSubBuffer(int x,
                                                       int width,
                                                       int height) {
   DCHECK(supports_post_sub_buffer_);
+  if (!CommitAndClearPendingOverlays()) {
+    DVLOG(1) << "Failed to commit pending overlay planes.";
+    return gfx::SwapResult::SWAP_FAILED;
+  }
   if (!eglPostSubBufferNV(GetDisplay(), surface_, x, y, width, height)) {
     DVLOG(1) << "eglPostSubBufferNV failed with error "
              << GetLastEGLErrorString();
@@ -755,6 +764,23 @@ gfx::SwapResult NativeViewGLSurfaceEGL::PostSubBuffer(int x,
   return gfx::SwapResult::SWAP_ACK;
 }
 
+bool NativeViewGLSurfaceEGL::SupportsCommitOverlayPlanes() {
+#if defined(OS_ANDROID)
+  return true;
+#else
+  return false;
+#endif
+}
+
+gfx::SwapResult NativeViewGLSurfaceEGL::CommitOverlayPlanes() {
+  DCHECK(SupportsCommitOverlayPlanes());
+  // Here we assume that the overlays scheduled on this surface will display
+  // themselves to the screen right away in |CommitAndClearPendingOverlays|,
+  // rather than being queued and waiting for a "swap" signal.
+  return CommitAndClearPendingOverlays() ? gfx::SwapResult::SWAP_ACK
+                                         : gfx::SwapResult::SWAP_FAILED;
+}
+
 VSyncProvider* NativeViewGLSurfaceEGL::GetVSyncProvider() {
   return vsync_provider_.get();
 }
@@ -764,15 +790,13 @@ bool NativeViewGLSurfaceEGL::ScheduleOverlayPlane(int z_order,
                                                   gl::GLImage* image,
                                                   const Rect& bounds_rect,
                                                   const RectF& crop_rect) {
-#if defined(OS_ANDROID)
-  // Overlay planes are used on Android for fullscreen video. The image is
-  // expected to update the plane as soon as possible to display the video frame
-  // chosen for this vsync interval.
-  return image->ScheduleOverlayPlane(window_, z_order, transform, bounds_rect,
-                                     crop_rect);
-#else
+#if !defined(OS_ANDROID)
   NOTIMPLEMENTED();
   return false;
+#else
+  pending_overlays_.push_back(
+      GLSurfaceOverlay(z_order, transform, image, bounds_rect, crop_rect));
+  return true;
 #endif
 }
 
@@ -788,6 +812,17 @@ NativeViewGLSurfaceEGL::~NativeViewGLSurfaceEGL() {
 #endif
 }
 
+bool NativeViewGLSurfaceEGL::CommitAndClearPendingOverlays() {
+  if (pending_overlays_.empty())
+    return true;
+
+  bool success = true;
+  for (const auto& overlay : pending_overlays_)
+    success &= overlay.ScheduleOverlayPlane(window_);
+  pending_overlays_.clear();
+  return success;
+}
+
 PbufferGLSurfaceEGL::PbufferGLSurfaceEGL(const gfx::Size& size)
     : size_(size),
       surface_(NULL) {