Workaround for bad driver issue with NVIDIA GeForce 7300 GT on Mac 10.5.

remoting/host/capturer_mac.cc

 // 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 "remoting/host/capturer.h"
 
 #include <ApplicationServices/ApplicationServices.h>
 #include <OpenGL/CGLMacro.h>
 #include <OpenGL/OpenGL.h>
 
 #include <stddef.h>
 
 #include "base/logging.h"
+#include "base/mac/mac_util.h"
 #include "base/memory/scoped_ptr.h"
 #include "remoting/host/capturer_helper.h"
 
 namespace remoting {
 
 namespace {
 
 class scoped_pixel_buffer_object {
  public:
   scoped_pixel_buffer_object();
@@ skipping 15 matching lines @@
     : cgl_context_(NULL),
       pixel_buffer_object_(0) {
 }
 
 scoped_pixel_buffer_object::~scoped_pixel_buffer_object() {
   Release();
 }
 
 bool scoped_pixel_buffer_object::Init(CGLContextObj cgl_context,
                                       int size_in_bytes) {
+  // The PBO path is only done on 10.6 (SnowLeopard) and above due to
+  // a driver issue that was found on 10.5
+  // (specifically on a NVIDIA GeForce 7300 GT).
+  // http://crbug.com/87283
+  if (base::mac::IsOSLeopardOrEarlier()) {
+    return false;
+  }
   cgl_context_ = cgl_context;
   CGLContextObj CGL_MACRO_CONTEXT = cgl_context_;
   glGenBuffersARB(1, &pixel_buffer_object_);
   if (glGetError() == GL_NO_ERROR) {
     glBindBufferARB(GL_PIXEL_PACK_BUFFER_ARB, pixel_buffer_object_);
     glBufferDataARB(GL_PIXEL_PACK_BUFFER_ARB, size_in_bytes, NULL,
                     GL_STREAM_READ_ARB);
     glBindBufferARB(GL_PIXEL_PACK_BUFFER_ARB, 0);
     if (glGetError() != GL_NO_ERROR) {
       Release();
@@ skipping 231 matching lines @@
     InvalidRects rects;
     helper_.SwapInvalidRects(rects);
     VideoFrameBuffer& current_buffer = buffers_[current_buffer_];
     current_buffer.Update();
 
     bool flip = true;  // GL capturers need flipping.
     if (cgl_context_) {
       if (pixel_buffer_object_.get() != 0) {
         GlBlitFast(current_buffer);
       } else {
+        // See comment in scoped_pixel_buffer_object::Init about why the slow
+        // path is always used on 10.5.
         GlBlitSlow(current_buffer);
       }
     } else {
       CgBlit(current_buffer, rects);
       flip = false;
     }
 
     DataPlanes planes;
     planes.data[0] = current_buffer.ptr();
     planes.strides[0] = current_buffer.bytes_per_row();
@@ skipping 134 matching lines @@
 }
 
 }  // namespace
 
 // static
 Capturer* Capturer::Create() {
   return new CapturerMac();
 }
 
 }  // namespace remoting