Support using OpenGL ES context on desktop

tests/BlurTest.cpp

 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkBlurMask.h"
 #include "SkBlurMaskFilter.h"
 #include "SkBlurDrawLooper.h"
@@ skipping 259 matching lines @@
 
     blur_path(&canvas, path, gaussianSigma);
     readback(&canvas, result, resultCount);
 }
 
 #if SK_SUPPORT_GPU
 static bool gpu_blur_path(GrContextFactory* factory, const SkPath& path,
                           SkScalar gaussianSigma,
                           int* result, int resultCount) {
 
-    GrContext* grContext = factory->get(GrContextFactory::kNative_GLContextType);
+    GrContext* grContext = factory->get(GrContextFactory::kNative_GLContextType,
+                                        kNone_GrGLStandard);

[mtklein, 2014/06/09 14:10:46]

Passing kNone in all these tests reads weird.  Can we make this
kDefault_GrGLStandard or kNative_GrGLStandard?

[robertphillips, 2014/06/09 14:31:05]

I agree - it is kind of weird.

[Kimmo Kinnunen, 2014/06/16 12:36:40]

That would be double the weird for all the other funcs that accept GrGLStandard,
as it would give the impression that there's 4 possible distinct values, and the
other code handles only 2 or max 3 cases.

I can of course make the "get" function to have default value in its second
parameter, eg. GrContextFactor::get(GLContextType type, GrGLStandard forcedApi =
kNone_GrGLStandard). The idea would be that passing kNone would read "set the
forced api to none, in other words do not force the api". Does this suffice?

     if (NULL == grContext) {
         return false;
     }
 
     GrTextureDesc desc;
     desc.fConfig = kSkia8888_GrPixelConfig;
     desc.fFlags = kRenderTarget_GrTextureFlagBit;
     desc.fWidth = resultCount;
     desc.fHeight = 30;
     desc.fSampleCnt = 0;
@@ skipping 269 matching lines @@
     }
 }
 
 ///////////////////////////////////////////////////////////////////////////////////////////
 
 DEF_GPUTEST(Blur, reporter, factory) {
     test_blur_drawing(reporter);
     test_sigma_range(reporter, factory);
     test_asABlur(reporter);
 }