diff options
author | reed <reed@google.com> | 2015-12-14 05:58:25 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-12-14 05:58:25 -0800 |
commit | 4ff653cab2e1e3cfb02c91df9c9013fc927aa1ee (patch) | |
tree | 23d046fed15c4bbfcd1039c92fe6612b8c993694 /include | |
parent | 0b29b728cae6c3f3738b4cda0b786960d05dda2b (diff) |
Revert of Add config options to run different GPU APIs to dm and nanobench (patchset #12 id:220001 of https://codereview.chromium.org/1490113005/ )
Reason for revert:
speculative revert to see if it unblocks the DEPS roll
https://codereview.chromium.org/1529443002
Original issue's description:
> Add config options to run different GPU APIs to dm and nanobench
>
> Add extended config specification form that can be used to run different
> gpu backend with different APIs.
>
> The configs can be specified with the form:
> gpu(api=string,dit=bool,nvpr=bool,samples=int)
>
> This replaces and removes the --gpuAPI flag.
>
> All existing configs should still work.
>
> Adds following documentation:
>
> out/Debug/dm --help config
>
> Flags:
> --config: type: string default: 565 8888 gpu nonrendering
> Options: 565 8888 debug gpu gpudebug gpudft gpunull msaa16 msaa4
> nonrendering null nullgpu nvprmsaa16 nvprmsaa4 pdf pdf_poppler skp svg
> xps or use extended form 'backend(option=value,...)'.
>
> Extended form: 'backend(option=value,...)'
>
> Possible backends and options:
>
> gpu(api=string,dit=bool,nvpr=bool,samples=int) GPU backend
> api type: string default: native.
> Select graphics API to use with gpu backend.
> Options:
> native Use platform default OpenGL or OpenGL ES backend.
> gl Use OpenGL.
> gles Use OpenGL ES.
> debug Use debug OpenGL.
> null Use null OpenGL.
> dit type: bool default: false.
> Use device independent text.
> nvpr type: bool default: false.
> Use NV_path_rendering OpenGL and OpenGL ES extension.
> samples type: int default: 0.
> Use multisampling with N samples.
>
> Predefined configs:
>
> gpu = gpu()
> msaa4 = gpu(samples=4)
> msaa16 = gpu(samples=16)
> nvprmsaa4 = gpu(nvpr=true,samples=4)
> nvprmsaa16 = gpu(nvpr=true,samples=16)
> gpudft = gpu(dit=true)
> gpudebug = gpu(api=debug)
> gpunull = gpu(api=null)
> debug = gpu(api=debug)
> nullgpu = gpu(api=null)
>
> BUG=skia:2992
>
> Committed: https://skia.googlesource.com/skia/+/e13ca329fca4c28cf4e078561f591ab27b743d23
TBR=bsalomon@google.com,scroggo@google.com,joshualitt@google.com,kkinnunen@nvidia.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:2992
Review URL: https://codereview.chromium.org/1528473002
Diffstat (limited to 'include')
-rw-r--r-- | include/core/SkString.h | 17 | ||||
-rw-r--r-- | include/gpu/gl/SkNullGLContext.h | 2 | ||||
-rw-r--r-- | include/gpu/gl/angle/SkANGLEGLContext.h | 15 | ||||
-rw-r--r-- | include/gpu/gl/command_buffer/SkCommandBufferGLContext.h | 5 |
4 files changed, 10 insertions, 29 deletions
diff --git a/include/core/SkString.h b/include/core/SkString.h index 93514f2659..9229d808a6 100644 --- a/include/core/SkString.h +++ b/include/core/SkString.h @@ -267,22 +267,7 @@ template <> inline void SkTSwap(SkString& a, SkString& b) { a.swap(b); } -enum SkStrSplitMode { - // Strictly return all results. If the input is ",," and the separator is ',' this will return - // an array of three empty strings. - kStrict_SkStrSplitMode, - - // Only nonempty results will be added to the results. Multiple separators will be - // coalesced. Separators at the beginning and end of the input will be ignored. If the input is - // ",," and the separator is ',', this will return an empty vector. - kCoalesce_SkStrSplitMode -}; - // Split str on any characters in delimiters into out. (Think, strtok with a sane API.) -void SkStrSplit(const char* str, const char* delimiters, SkStrSplitMode splitMode, - SkTArray<SkString>* out); -inline void SkStrSplit(const char* str, const char* delimiters, SkTArray<SkString>* out) { - SkStrSplit(str, delimiters, kCoalesce_SkStrSplitMode, out); -} +void SkStrSplit(const char* str, const char* delimiters, SkTArray<SkString>* out); #endif diff --git a/include/gpu/gl/SkNullGLContext.h b/include/gpu/gl/SkNullGLContext.h index a9c59cf2a1..1f63438264 100644 --- a/include/gpu/gl/SkNullGLContext.h +++ b/include/gpu/gl/SkNullGLContext.h @@ -14,7 +14,7 @@ class SK_API SkNullGLContext : public SkGLContext { public: ~SkNullGLContext() override; - static SkNullGLContext* Create(); + static SkNullGLContext* Create(GrGLStandard); class ContextState; diff --git a/include/gpu/gl/angle/SkANGLEGLContext.h b/include/gpu/gl/angle/SkANGLEGLContext.h index ea5e877ca2..7858fff964 100644 --- a/include/gpu/gl/angle/SkANGLEGLContext.h +++ b/include/gpu/gl/angle/SkANGLEGLContext.h @@ -15,25 +15,18 @@ class SkANGLEGLContext : public SkGLContext { public: ~SkANGLEGLContext() override; -#ifdef SK_BUILD_FOR_WIN - static SkANGLEGLContext* CreateDirectX() { - SkANGLEGLContext* ctx = new SkANGLEGLContext(false); - if (!ctx->isValid()) { - delete ctx; + + static SkANGLEGLContext* Create(GrGLStandard forcedGpuAPI, bool useGLBackend) { + if (kGL_GrGLStandard == forcedGpuAPI) { return NULL; } - return ctx; - } -#endif - static SkANGLEGLContext* CreateOpenGL() { - SkANGLEGLContext* ctx = new SkANGLEGLContext(true); + SkANGLEGLContext* ctx = new SkANGLEGLContext(useGLBackend); if (!ctx->isValid()) { delete ctx; return NULL; } return ctx; } - GrEGLImage texture2DToEGLImage(GrGLuint texID) const override; void destroyEGLImage(GrEGLImage) const override; GrGLuint eglImageToExternalTexture(GrEGLImage) const override; diff --git a/include/gpu/gl/command_buffer/SkCommandBufferGLContext.h b/include/gpu/gl/command_buffer/SkCommandBufferGLContext.h index 47f3fd967a..7fece3c83c 100644 --- a/include/gpu/gl/command_buffer/SkCommandBufferGLContext.h +++ b/include/gpu/gl/command_buffer/SkCommandBufferGLContext.h @@ -16,7 +16,10 @@ class SkCommandBufferGLContext : public SkGLContext { public: ~SkCommandBufferGLContext() override; - static SkCommandBufferGLContext* Create() { + static SkCommandBufferGLContext* Create(GrGLStandard forcedGpuAPI) { + if (kGL_GrGLStandard == forcedGpuAPI) { + return nullptr; + } SkCommandBufferGLContext* ctx = new SkCommandBufferGLContext; if (!ctx->isValid()) { delete ctx; |