Rename enums in GrContextFactory to remove "GL"

Rename enums in GrContextFactory to remove "GL"

Also, remove kNative as a separate context type and instead make it an alias for kGL or kGLES based on OS.


GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1845923004

Committed: https://skia.googlesource.com/skia/+/2d9c6f81353597aebf5934547e5cba7a872196fb

Committed: https://skia.googlesource.com/skia/+/85b4b53e78dcdeae2a51935ca5e214b0f1a06492

[bsalomon, 2016-04-01 19:17:20]

Description was changed from

==========
Rename enums in GrContext to remove "GL"
==========

to

==========
Rename enums in GrContext to remove "GL"
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[bsalomon, 2016-04-01 19:56:43]

Description was changed from

==========
Rename enums in GrContext to remove "GL"
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Rename enums in GrContextFactory to remove "GL"

Also, remove kNative as a separate context type and instead make it an alias for
kGL or kGLES based on OS.


GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[bsalomon, 2016-04-01 19:59:13]

bsalomon@google.com changed reviewers:
+ egdaniel@google.com, jvanverth@google.com

[bsalomon, 2016-04-01 19:59:14]

Prep for adding a Vulkan ContextType. It's all renaming with one exception: I
replaced the enum value kNative_GLContextType with kNativeGL_ContextType which
is not a member of the enum type but simple an alias for either kGL_ContextType
or kGLES_ContextType. This implied some small changes to the gpu config parsing
tests.

[egdaniel, 2016-04-01 20:06:43]

lgtm plus question on last left over GL thing

https://codereview.chromium.org/1845923004/diff/20001/tools/gpu/GrContextFact...
File tools/gpu/GrContextFactory.h (right):

https://codereview.chromium.org/1845923004/diff/20001/tools/gpu/GrContextFact...
tools/gpu/GrContextFactory.h:118: GLTestContext* fGLContext; //! Valid until the
factory destroys it via abandonContexts() or
Do we still want a GLTestContext here?

[bsalomon, 2016-04-01 20:15:29]

https://codereview.chromium.org/1845923004/diff/20001/tools/gpu/GrContextFact...
File tools/gpu/GrContextFactory.h (right):

https://codereview.chromium.org/1845923004/diff/20001/tools/gpu/GrContextFact...
tools/gpu/GrContextFactory.h:118: GLTestContext* fGLContext; //! Valid until the
factory destroys it via abandonContexts() or
On 2016/04/01 20:06:43, egdaniel wrote:
> Do we still want a GLTestContext here?

This will evolve. I think initially yes and it will be null for vulkan, later
vulkan contexts will have a vulkan context object.

[bsalomon, 2016-04-01 20:15:50]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2016-04-01 20:15:51]

The patchset sent to the CQ was uploaded after l-g-t-m from egdaniel@google.com
Link to the patchset: https://codereview.chromium.org/1845923004/#ps40001
(title: "fix targets not named dm")

[commit-bot: I haz the power, 2016-04-01 20:15:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845923004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845923004/40001

[bsalomon, 2016-04-01 20:25:21]

The CQ bit was unchecked by bsalomon@google.com

[bsalomon, 2016-04-01 20:26:46]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2016-04-01 20:26:46]

The patchset sent to the CQ was uploaded after l-g-t-m from egdaniel@google.com
Link to the patchset: https://codereview.chromium.org/1845923004/#ps60001
(title: "Fix ANGLE build")

[commit-bot: I haz the power, 2016-04-01 20:26:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845923004/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845923004/60001

[commit-bot: I haz the power, 2016-04-01 20:38:11]

Description was changed from

==========
Rename enums in GrContextFactory to remove "GL"

Also, remove kNative as a separate context type and instead make it an alias for
kGL or kGLES based on OS.


GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Rename enums in GrContextFactory to remove "GL"

Also, remove kNative as a separate context type and instead make it an alias for
kGL or kGLES based on OS.


GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/2d9c6f81353597aebf5934547e5cba7a872196fb
==========

[commit-bot: I haz the power, 2016-04-01 20:38:12]

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/2d9c6f81353597aebf5934547e5cba7a872196fb

[bsalomon, 2016-04-04 12:37:41]

A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.chromium.org/1856703002/ by bsalomon@google.com.

The reason for reverting is: Many GM images unexpectedly changed with this CL..

[bsalomon, 2016-04-04 20:41:22]

Description was changed from

==========
Rename enums in GrContextFactory to remove "GL"

Also, remove kNative as a separate context type and instead make it an alias for
kGL or kGLES based on OS.


GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/2d9c6f81353597aebf5934547e5cba7a872196fb
==========

to

==========
Rename enums in GrContextFactory to remove "GL"

Also, remove kNative as a separate context type and instead make it an alias for
kGL or kGLES based on OS.


GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/2d9c6f81353597aebf5934547e5cba7a872196fb
==========

[bsalomon, 2016-04-05 17:49:43]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2016-04-05 17:49:43]

The patchset sent to the CQ was uploaded after l-g-t-m from egdaniel@google.com
Link to the patchset: https://codereview.chromium.org/1845923004/#ps80001
(title: "rebase")

[commit-bot: I haz the power, 2016-04-05 17:49:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845923004/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845923004/80001

[commit-bot: I haz the power, 2016-04-05 18:06:30]

Description was changed from

==========
Rename enums in GrContextFactory to remove "GL"

Also, remove kNative as a separate context type and instead make it an alias for
kGL or kGLES based on OS.


GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/2d9c6f81353597aebf5934547e5cba7a872196fb
==========

to

==========
Rename enums in GrContextFactory to remove "GL"

Also, remove kNative as a separate context type and instead make it an alias for
kGL or kGLES based on OS.


GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/2d9c6f81353597aebf5934547e5cba7a872196fb

Committed:
https://skia.googlesource.com/skia/+/85b4b53e78dcdeae2a51935ca5e214b0f1a06492
==========

[commit-bot: I haz the power, 2016-04-05 18:06:31]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/skia/+/85b4b53e78dcdeae2a51935ca5e214b0f1a06492