GLContextNSView should not assume concrete type GLSurface is GLSurfaceNSView

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/NSOpenGL.h>
 #import <AppKit/NSView.h>
 
 #include "ui/gfx/gl/gl_context_nsview.h"
@@ skipping 21 matching lines @@
 }
 
 gfx::Size GLSurfaceNSView::GetSize() {
   return gfx::Size(NSSizeToCGSize([view_ bounds].size));
 }
 
 void* GLSurfaceNSView::GetHandle() {
   return view_;
 }
 
-void GLSurfaceNSView::SetGLContext(GLContextNSView* context) {
-  context_ = context;
+bool GLSurfaceNSView::OnMakeCurrent(GLContext* context) {
+  context_ = static_cast<GLContextNSView *>(context);

[Ken Russell (switch to Gerrit), 2011/11/30 00:39:01]

Given the other cleanup in this CL, the downcast is unfortunate. This is the
first platform that has different context types for different surfaces.
Presumably MakeCurrent should do some validation that you're using compatible
contexts and surfaces. Anyway, this is a cleanup for the future.

+  return true;
 }
 
 }  // namespace gfx