Use Windowing system-specific WindowContext factories.

tools/viewer/sk_app/mac/GLWindowContext_mac.cpp

 
 /*
  * Copyright 2016 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
-#include "GLWindowContext_mac.h"
+#include "../GLWindowContext.h"
+#include "WindowContextFactory_mac.h"
 
 //#include <GL/gl.h>
 
-#include "Window_mac.h"
+using sk_app::DisplayParams;
+using sk_app::window_context_factory::MacWindowInfo;
+using sk_app::GLWindowContext;
 
-namespace sk_app {
+namespace {
 
-// platform-dependent create
-GLWindowContext* GLWindowContext::Create(void* platformData, const DisplayParams& params) {
-    GLWindowContext_mac* ctx = new GLWindowContext_mac(platformData, params);
-    if (!ctx->isValid()) {
-        delete ctx;
-        return nullptr;
-    }
-    return ctx;
-}
+class GLWindowContext_mac : public GLWindowContext {
+public:
+    GLWindowContext_mac(const MacWindowInfo&, const DisplayParams&);
+    
+    ~GLWindowContext_mac() override;
+    
+    void onSwapBuffers() override;
+    
+    void onInitializeContext() override;
+    void onDestroyContext() override;
+    
+private:
+  
+#if 0
+    // TODO: add Mac-specific GL display objects
+    Display*     fDisplay;
+    XWindow      fWindow;
+    XVisualInfo* fVisualInfo;
+    GLXContext   fGLContext;
+#endif
+};
 
-GLWindowContext_mac::GLWindowContext_mac(void* platformData, const DisplayParams& params)
-    : GLWindowContext(platformData, params)
+GLWindowContext_mac::GLWindowContext_mac(const MacWindowInfo& info, const DisplayParams& params)
+    : GLWindowContext(params)
 #if 0
     // TODO: init Mac-specific OpenGL objects
     , fDisplay(nullptr)
     , fWindow(0)
     , fGLContext(0)
 #endif
     {
 
     // any config code here (particularly for msaa)?
 
-    this->initializeContext(platformData, params);
+    this->initializeContext();
 }
 
 GLWindowContext_mac::~GLWindowContext_mac() {
     this->destroyContext();
 }
 
-void GLWindowContext_mac::onInitializeContext(void* platformData, const DisplayParams& params) {
+void GLWindowContext_mac::onInitializeContext() {
 #if 0
     // TODO: Init for Mac
-    ContextPlatformData_mac* unixPlatformData =
-        reinterpret_cast<ContextPlatformData_mac*>(platformData);
-
-    if (unixPlatformData) {
-        fDisplay = unixPlatformData->fDisplay;
-        fWindow = unixPlatformData->fWindow;
-        fVisualInfo = unixPlatformData->fVisualInfo;
-    }
     SkASSERT(fDisplay);
 
     fGLContext = glXCreateContext(fDisplay, fVisualInfo, nullptr, GL_TRUE);
     if (!fGLContext) {
         return;
     }
 
     if (glXMakeCurrent(fDisplay, fWindow, fGLContext)) {
         glClearStencil(0);
         glClearColor(0, 0, 0, 0);
@@ skipping 33 matching lines @@
 
 void GLWindowContext_mac::onSwapBuffers() {
 #if 0
     // TODO: swap for Mac
     if (fDisplay && fGLContext) {
         glXSwapBuffers(fDisplay, fWindow);
     }
 #endif
 }
 
+}  // anonymous namespace
 
-}   //namespace sk_app
+
+namespace sk_app {
+namespace window_context_factory {
+
+WindowContext* MakeGLForMac(const MacWindowInfo& info, const DisplayParams& params) {
+    WindowContext* ctx = new GLWindowContext_mac(info, params);
+    if (!ctx->isValid()) {
+        delete ctx;
+        return nullptr;
+    }
+    return ctx;
+}
+
+}  // namespace window_context_factory
+}  // namespace sk_app