compositor_unittests target is unimplmented on Mac

ui/gfx/gl/gl_surface_nsview.mm

+// 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.
+
+#include "ui/gfx/gl/gl_surface_nsview.h"
+
+#import <AppKit/NSView.h>
+
+#include "base/basictypes.h"
+#include "base/logging.h"
+#include "base/mac/mac_util.h"
+#include "ui/gfx/gl/gl_bindings.h"
+#include "ui/gfx/gl/gl_context.h"
+
+namespace gfx {
+
+GLSurfaceNSView::GLSurfaceNSView(PluginWindowHandle view)
+    : view_(view) {
+}
+
+GLSurfaceNSView::~GLSurfaceNSView() {
+}
+
+void GLSurfaceNSView::Destroy() {
+}
+
+bool GLSurfaceNSView::IsOffscreen() {
+  return false;
+}
+
+bool GLSurfaceNSView::SwapBuffers() {
+  CGLContextObj cgl_context = CGLGetCurrentContext();

[Nico, 2011/11/16 19:11:13]

Is there some way to DCHECK that the right context is current here?

[dhollowa, 2011/11/16 19:59:18]

Not really, no.  The view doesn't expose accessors for its context, it is
implied by call order.  That is the view's drawing traversal sets the context up
but the only way to get at it is via the notion of "current" context.  Lots of
pushing/popping, but no direct access.

On 2011/11/16 19:11:13, Nico wrote:

+  CGLFlushDrawable(cgl_context);
+  return true;
+}
+
+gfx::Size GLSurfaceNSView::GetSize() {
+  NSRect bounds = [view_ bounds];
+  return gfx::Size(bounds.size.width, bounds.size.height);

[Nico, 2011/11/16 19:11:13]

Doesn't gfx::Size() have a constructor that takes an NSSize (or at least a
CGSize)?

[dhollowa, 2011/11/16 19:59:18]

Done.

+}
+
+void* GLSurfaceNSView::GetHandle() {
+  return view_;
+}
+
+}  // namespace gfx