Replace GrGLInterface's function pointers with std::functions

include/gpu/gl/GrGLFunctions.h

Index: include/gpu/gl/GrGLFunctions.h
diff --git a/include/gpu/gl/GrGLFunctions.h b/include/gpu/gl/GrGLFunctions.h
index ad59ec0c00b702200e87f7d0934ade1b840cf6ae..118269bc2f0c50a72cb3e425ad2d9680b0c8d1c1 100644
--- a/include/gpu/gl/GrGLFunctions.h
+++ b/include/gpu/gl/GrGLFunctions.h
@@ -9,7 +9,9 @@
 #ifndef GrGLFunctions_DEFINED
 #define GrGLFunctions_DEFINED
 
+#include <functional>
 #include "GrGLTypes.h"
+#include "../private/SkTLogic.h"

[bungeman-skia, 2016/02/08 20:22:52]

I imagine this include was added in order to replace
std::remove_pointer<GLPTR>::type with skstd::remove_pointer_t<GLPTR>. However
that didn't seem to happen, so either this include should be removed or the code
below changed. As things stand, it would be better to just directly include
<type_traits>.

[bsalomon, 2016/02/09 15:21:38]

Oops, I did that change locally and then never uploaded it. Will submit it as a
separate change.

 
 extern "C" {
 
@@ -351,4 +353,7 @@ typedef GrEGLImage (GR_GL_FUNCTION_TYPE* GrEGLCreateImageProc)(GrEGLDisplay dpy,
 typedef GrEGLBoolean (GR_GL_FUNCTION_TYPE* GrEGLDestroyImageProc)(GrEGLDisplay dpy, GrEGLImage image);
 }  // extern "C"
 
+template <typename GLPTR> using GrGLFunction =
+    std::function<typename std::remove_pointer<GLPTR>::type>;
+
 #endif