Move SkGLContext and some GrGLInterface implementations to skgputest module

tests/TextureStorageAllocator.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 "Test.h"
 #if SK_SUPPORT_GPU
 #include "gl/GrGLGpu.h"
+#include "gl/GLContext.h"

[jvanverth1, 2016/03/28 20:26:35]

One thing about the renaming is that it's not clear that GLContext is a
tools-only thing. #include "tools/gpu/gl/GLContext.h" would make it clearer but
that's a lot of typing. I don't have a good answer other than renaming GLContext
to maybe GLTestContext -- which is also an unpleasant amount of code changes.

[bsalomon, 2016/03/28 20:46:25]

I like that and don't mind doing it. Maybe as a follow-up standalone change?

[jvanverth1, 2016/03/29 13:39:15]

A followup would be better -- there's a lot to look at here already.

 #include "GrContext.h"
 #include "SkSurface_Gpu.h"
-#include "../include/gpu/gl/SkGLContext.h"
 #include "../include/gpu/GrTypes.h"
 #include "../include/private/SkTemplates.h"
 
 class TestStorageAllocator {
  public:
   static GrTextureStorageAllocator::Result allocateTextureStorage(void* ctx,
           GrBackendObject texture, unsigned width, unsigned height, GrPixelConfig config,
           const void* srcData, GrSurfaceOrigin) {
       TestStorageAllocator* allocator = static_cast<TestStorageAllocator*>(ctx);
       if (!allocator->m_allowAllocation)
@@ skipping 76 matching lines @@
     grAllocator.fAllocateTextureStorage = &TestStorageAllocator::allocateTextureStorage;
     grAllocator.fDeallocateTextureStorage= &TestStorageAllocator::deallocateTextureStorage;
     grAllocator.fCtx = &allocator;
     auto surface(SkSurface_Gpu::MakeRenderTarget(
             context, SkBudgeted::kNo, SkImageInfo::MakeN32Premul(kWidth, kHeight), 0,
             NULL, grAllocator));
     REPORTER_ASSERT(reporter, !surface);
 }
 
 #endif