Adding status to swap complete

ui/gl/gl_surface.cc

 // Copyright (c) 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.
 
 #include "ui/gl/gl_surface.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/command_line.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 #include "base/threading/thread_local.h"
 #include "base/trace_event/trace_event.h"
+#include "ui/gfx/swap_result.h"
 #include "ui/gl/gl_context.h"
 #include "ui/gl/gl_implementation.h"
 #include "ui/gl/gl_switches.h"
 
 #if defined(USE_X11)
 #include <X11/Xlib.h>
 #endif
 
 namespace gfx {
 
@@ skipping 162 matching lines @@
 bool GLSurface::SupportsPostSubBuffer() {
   return false;
 }
 
 unsigned int GLSurface::GetBackingFrameBufferObject() {
   return 0;
 }
 
 bool GLSurface::SwapBuffersAsync(const SwapCompletionCallback& callback) {
   DCHECK(!IsSurfaceless());
-  bool success = SwapBuffers();
-  callback.Run();
-  return success;
+  gfx::SwapResult result = SwapBuffers();
+  callback.Run(result);
+  return result == gfx::SwapResult::SWAP_ACK;
 }
 
-bool GLSurface::PostSubBuffer(int x, int y, int width, int height) {
-  return false;
+gfx::SwapResult GLSurface::PostSubBuffer(int x, int y, int width, int height) {
+  return gfx::SwapResult::SWAP_FAILED;
 }
 
 bool GLSurface::PostSubBufferAsync(int x,
                                    int y,
                                    int width,
                                    int height,
                                    const SwapCompletionCallback& callback) {
-  bool success = PostSubBuffer(x, y, width, height);
-  callback.Run();
-  return success;
+  gfx::SwapResult result = PostSubBuffer(x, y, width, height);
+  callback.Run(result);
+  return result == gfx::SwapResult::SWAP_ACK;
 }
 
 bool GLSurface::OnMakeCurrent(GLContext* context) {
   return true;
 }
 
 void GLSurface::NotifyWasBound() {
 }
 
 bool GLSurface::SetBackbufferAllocation(bool allocated) {
@@ skipping 88 matching lines @@
 }
 
 bool GLSurfaceAdapter::DeferDraws() {
   return surface_->DeferDraws();
 }
 
 bool GLSurfaceAdapter::IsOffscreen() {
   return surface_->IsOffscreen();
 }
 
-bool GLSurfaceAdapter::SwapBuffers() {
+gfx::SwapResult GLSurfaceAdapter::SwapBuffers() {
   return surface_->SwapBuffers();
 }
 
 bool GLSurfaceAdapter::SwapBuffersAsync(
     const SwapCompletionCallback& callback) {
   return surface_->SwapBuffersAsync(callback);
 }
 
-bool GLSurfaceAdapter::PostSubBuffer(int x, int y, int width, int height) {
+gfx::SwapResult GLSurfaceAdapter::PostSubBuffer(int x,
+                                                int y,
+                                                int width,
+                                                int height) {
   return surface_->PostSubBuffer(x, y, width, height);
 }
 
 bool GLSurfaceAdapter::PostSubBufferAsync(
     int x, int y, int width, int height,
         const SwapCompletionCallback& callback) {
   return surface_->PostSubBufferAsync(x, y, width, height, callback);
 }
 
 bool GLSurfaceAdapter::SupportsPostSubBuffer() {
@@ skipping 53 matching lines @@
       z_order, transform, image, bounds_rect, crop_rect);
 }
 
 bool GLSurfaceAdapter::IsSurfaceless() const {
   return surface_->IsSurfaceless();
 }
 
 GLSurfaceAdapter::~GLSurfaceAdapter() {}
 
 }  // namespace gfx