compositor_unittests target is unimplmented on Mac

ui/gfx/gl/gl_surface_nsview.h

+// Copyright (c) 2011 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_GL_GL_SURFACE_NSVIEW_H_
+#define UI_GFX_GL_GL_SURFACE_NSVIEW_H_
+
+#include "base/compiler_specific.h"
+#include "ui/gfx/gl/gl_surface.h"
+#include "ui/gfx/size.h"
+
+namespace gfx {
+
+class GLContextNSView;
+
+// GLSurfaceNSView provides an implementation of the the GLSurface interface
+// that is backed by an NSView.  This interface pairs with the GLContextNSView
+// class, and the NSView is expected to use this context for drawing.
+class GLSurfaceNSView : public GLSurface {
+ public:
+  explicit GLSurfaceNSView(PluginWindowHandle view);
+  virtual ~GLSurfaceNSView();
+
+  // GLSurface:
+  virtual void Destroy() OVERRIDE;
+  virtual bool IsOffscreen() OVERRIDE;
+  virtual bool SwapBuffers() OVERRIDE;
+  virtual gfx::Size GetSize() OVERRIDE;
+  virtual void* GetHandle() OVERRIDE;

[apatrick_chromium, 2011/11/21 23:34:45]

It would be safer to implement OnMakeCurrent here instead and cache off the
context at that point. That would also make this code work in the event that a
surface is made current with a surface that is different to the one that was
passed to GLContextNSView::Initialize.

+
+  // Allow the surface to call back to context when in need of |FlushBuffer|.
+  void SetGLContext(GLContextNSView* context);
+
+ private:
+  // Weak.  An |NSView*|.
+  PluginWindowHandle view_;
+
+  // Weak.  Associated context.
+  GLContextNSView* context_;
+
+  DISALLOW_COPY_AND_ASSIGN(GLSurfaceNSView);
+};
+
+}  // namespace gfx
+
+#endif  // UI_GFX_GL_GL_SURFACE_NSVIEW_H_