compositor_unittests target is unimplmented on Mac

ui/gfx/native_widget_types.h

Index: ui/gfx/native_widget_types.h
diff --git a/ui/gfx/native_widget_types.h b/ui/gfx/native_widget_types.h
index 3c4680de6dcad47a112940d8da80fb48e8180fa9..043125e795f90287eb82317521cd2a364a0c7008 100644
--- a/ui/gfx/native_widget_types.h
+++ b/ui/gfx/native_widget_types.h
@@ -227,6 +227,9 @@ static inline NativeView NativeViewFromIdInBrowser(NativeViewId id) {
 #elif defined(USE_X11)
   typedef unsigned long PluginWindowHandle;
   const PluginWindowHandle kNullPluginWindow = 0;
+#elif defined(USE_WEBKIT_COMPOSITOR) && defined(OS_MACOSX)

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

I don't know what the longer-term plan for this is, but these defines sound
generic enough that people might think USE_WEBKIT_COMPOSITOR could work in
desktop chrome. Can you add a comment that this only makes sense with the aura
prototype, and that getting this into "normal" chrome is fairly involved?

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

Done.  Yes.

I also wonder about the possible merging of |AcceleratedWidget| and
|PluginWindowHandle|.  They're almost identical, and the gl/gl_surface*
interfaces consume |PluginWindowHandle| but |AcceleratedWidget| seems more
appropriate somehow.


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

+  typedef NSView* PluginWindowHandle;
+  const PluginWindowHandle kNullPluginWindow = 0;
 #else
   // On OS X we don't have windowed plugins.
   // We use a NULL/0 PluginWindowHandle in shared code to indicate there
@@ -252,6 +255,9 @@ const AcceleratedWidget kNullAcceleratedWidget = NULL;
 #elif defined(USE_X11)
 typedef unsigned long AcceleratedWidget;
 const AcceleratedWidget kNullAcceleratedWidget = 0;
+#elif defined(USE_WEBKIT_COMPOSITOR) && defined(OS_MACOSX)
+typedef NSView* AcceleratedWidget;
+const AcceleratedWidget kNullAcceleratedWidget = 0;
 #else
 typedef void* AcceleratedWidget;
 const AcceleratedWidget kNullAcceleratedWidget = NULL;