Adding status to swap complete

ui/gl/gl_surface_win.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 <dwmapi.h>
 
 #include "base/command_line.h"
 #include "base/logging.h"
@@ skipping 21 matching lines @@
 // view.
 class NativeViewGLSurfaceOSMesa : public GLSurfaceOSMesa {
  public:
   explicit NativeViewGLSurfaceOSMesa(gfx::AcceleratedWidget window);
   ~NativeViewGLSurfaceOSMesa() override;
 
   // Implement subset of GLSurface.
   bool Initialize() override;
   void Destroy() override;
   bool IsOffscreen() override;
-  bool SwapBuffers() override;
-  bool SupportsPostSubBuffer() override;
+  gfx::SwapResult SwapBuffers() override;
+  gfx::SwapResult SupportsPostSubBuffer() override;
   bool PostSubBuffer(int x, int y, int width, int height) override;
 
  private:
   gfx::AcceleratedWidget window_;
   HDC device_context_;
 
   DISALLOW_COPY_AND_ASSIGN(NativeViewGLSurfaceOSMesa);
 };
 
 class WinVSyncProvider : public VSyncProvider {
@@ skipping 129 matching lines @@
 
   device_context_ = NULL;
 
   GLSurfaceOSMesa::Destroy();
 }
 
 bool NativeViewGLSurfaceOSMesa::IsOffscreen() {
   return false;
 }
 
-bool NativeViewGLSurfaceOSMesa::SwapBuffers() {
+gfx::SwapResult NativeViewGLSurfaceOSMesa::SwapBuffers() {
   DCHECK(device_context_);
 
   gfx::Size size = GetSize();
 
   // Note: negating the height below causes GDI to treat the bitmap data as row
   // 0 being at the top.
   BITMAPV4HEADER info = { sizeof(BITMAPV4HEADER) };
   info.bV4Width = size.width();
   info.bV4Height = -size.height();
   info.bV4Planes = 1;
@@ skipping 12 matching lines @@
   // browser test to become flaky if there is a race condition between GL
   // context destruction and window destruction.
   StretchDIBits(device_context_,
                 0, 0, size.width(), size.height(),
                 0, 0, size.width(), size.height(),
                 GetHandle(),
                 reinterpret_cast<BITMAPINFO*>(&info),
                 DIB_RGB_COLORS,
                 SRCCOPY);
 
-  return true;
+  return gfx::SWAP_ACK;
 }
 
 bool NativeViewGLSurfaceOSMesa::SupportsPostSubBuffer() {
   return true;
 }
 
-bool NativeViewGLSurfaceOSMesa::PostSubBuffer(
-    int x, int y, int width, int height) {
+gfx::SwapResult NativeViewGLSurfaceOSMesa::PostSubBuffer(int x,
+                                                         int y,
+                                                         int width,
+                                                         int height) {
   DCHECK(device_context_);
 
   gfx::Size size = GetSize();
 
   // Note: negating the height below causes GDI to treat the bitmap data as row
   // 0 being at the top.
   BITMAPV4HEADER info = { sizeof(BITMAPV4HEADER) };
   info.bV4Width = size.width();
   info.bV4Height = -size.height();
   info.bV4Planes = 1;
@@ skipping 12 matching lines @@
   // browser test to become flaky if there is a race condition between GL
   // context destruction and window destruction.
   StretchDIBits(device_context_,
                 x, size.height() - y - height, width, height,
                 x, y, width, height,
                 GetHandle(),
                 reinterpret_cast<BITMAPINFO*>(&info),
                 DIB_RGB_COLORS,
                 SRCCOPY);
 
-  return true;
+  return return gfx::SWAP_ACK;

[piman, 2015/05/14 20:58:54]

nit: return return = typo

 }
 
 scoped_refptr<GLSurface> GLSurface::CreateViewGLSurface(
     gfx::AcceleratedWidget window) {
   TRACE_EVENT0("gpu", "GLSurface::CreateViewGLSurface");
   switch (GetGLImplementation()) {
     case kGLImplementationOSMesaGL: {
       scoped_refptr<GLSurface> surface(
           new NativeViewGLSurfaceOSMesa(window));
       if (!surface->Initialize())
@@ skipping 64 matching lines @@
 
 EGLNativeDisplayType GetPlatformDefaultEGLNativeDisplay() {
   if (base::CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kDisableD3D11) ||
       base::CommandLine::ForCurrentProcess()->HasSwitch(switches::kUseWarp))
     return GetDC(NULL);
   return EGL_D3D11_ELSE_D3D9_DISPLAY_ANGLE;
 }
 
 }  // namespace gfx