Revert of Make SkGLContext lifetime more well-defined

Revert of Make SkGLContext lifetime more well-defined (patchset #7 id:120001 of https://codereview.chromium.org/1511773005/ )

Reason for revert:
Broke tests on Android, iOS, Mac and Windows.

Original issue's description:
> Make SkGLContext lifetime more well-defined
> 
> Remove refcounting from SkGLContext.
> 
> SkGLContext is expected to behave like GrContextFactory would own
> it, as implied by the GrContextFactory function.
> 
> If it is refcounted, this does not hold.
> 
> Also other use sites, such as in SkOSWindow_win (command buffer gl
> object), confirm the behavior. The object is explicitly owned and
> destroyed, not shared.
> 
> Also fixes potential crashes from using GL context of an abandoned
> context.
> 
> Also fixes potential crashes in DM/nanobench, if the GrContext lives
> longer than GLContext through internal refing of GrContext.
> 
> Moves the non-trivial implementations from GrContextFactory.h to
> .cpp, just for consistency sake.
> 
> Changes pathops_unittest.gyp. The pathops_unittest uses
> GrContextFactory, but did not link to its implementation. The reason
> they worked was that the implementation used (constructors, destructors)
> happened to be in the .h file.
> 
> This works towards being able to use command buffer and NVPR from
> the SampleApp.
> 
> BUG=skia:2992
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1511773005
> 
> Committed: https://skia.googlesource.com/skia/+/830e012187f951d49d7e46e196ac8d1e653a25da

TBR=bsalomon@google.com,kkinnunen@nvidia.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:2992
Committed: https://skia.googlesource.com/skia/+/54e2ca5a2338b6700fd4c147ad2f934c70ecac90

[jcgregorio, 2016-01-05 12:15:11]

Created Revert of Make SkGLContext lifetime more well-defined

[commit-bot: I haz the power, 2016-01-05 12:15:15]

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

[commit-bot: I haz the power, 2016-01-05 12:15:26]

Description was changed from

==========
Revert of Make SkGLContext lifetime more well-defined (patchset #7 id:120001 of
https://codereview.chromium.org/1511773005/ )

Reason for revert:
Broke tests on Android, iOS, Mac and Windows.

Original issue's description:
> Make SkGLContext lifetime more well-defined
> 
> Remove refcounting from SkGLContext.
> 
> SkGLContext is expected to behave like GrContextFactory would own
> it, as implied by the GrContextFactory function.
> 
> If it is refcounted, this does not hold.
> 
> Also other use sites, such as in SkOSWindow_win (command buffer gl
> object), confirm the behavior. The object is explicitly owned and
> destroyed, not shared.
> 
> Also fixes potential crashes from using GL context of an abandoned
> context.
> 
> Also fixes potential crashes in DM/nanobench, if the GrContext lives
> longer than GLContext through internal refing of GrContext.
> 
> Moves the non-trivial implementations from GrContextFactory.h to
> .cpp, just for consistency sake.
> 
> Changes pathops_unittest.gyp. The pathops_unittest uses
> GrContextFactory, but did not link to its implementation. The reason
> they worked was that the implementation used (constructors, destructors)
> happened to be in the .h file.
> 
> This works towards being able to use command buffer and NVPR from
> the SampleApp.
> 
> BUG=skia:2992
> GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
> 
> Committed:
https://skia.googlesource.com/skia/+/830e012187f951d49d7e46e196ac8d1e653a25da

TBR=bsalomon@google.com,kkinnunen@nvidia.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:2992
==========

to

==========
Revert of Make SkGLContext lifetime more well-defined (patchset #7 id:120001 of
https://codereview.chromium.org/1511773005/ )

Reason for revert:
Broke tests on Android, iOS, Mac and Windows.

Original issue's description:
> Make SkGLContext lifetime more well-defined
> 
> Remove refcounting from SkGLContext.
> 
> SkGLContext is expected to behave like GrContextFactory would own
> it, as implied by the GrContextFactory function.
> 
> If it is refcounted, this does not hold.
> 
> Also other use sites, such as in SkOSWindow_win (command buffer gl
> object), confirm the behavior. The object is explicitly owned and
> destroyed, not shared.
> 
> Also fixes potential crashes from using GL context of an abandoned
> context.
> 
> Also fixes potential crashes in DM/nanobench, if the GrContext lives
> longer than GLContext through internal refing of GrContext.
> 
> Moves the non-trivial implementations from GrContextFactory.h to
> .cpp, just for consistency sake.
> 
> Changes pathops_unittest.gyp. The pathops_unittest uses
> GrContextFactory, but did not link to its implementation. The reason
> they worked was that the implementation used (constructors, destructors)
> happened to be in the .h file.
> 
> This works towards being able to use command buffer and NVPR from
> the SampleApp.
> 
> BUG=skia:2992
> GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
> 
> Committed:
https://skia.googlesource.com/skia/+/830e012187f951d49d7e46e196ac8d1e653a25da

TBR=bsalomon@google.com,kkinnunen@nvidia.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:2992

Committed:
https://skia.googlesource.com/skia/+/54e2ca5a2338b6700fd4c147ad2f934c70ecac90
==========

[commit-bot: I haz the power, 2016-01-05 12:15:26]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/54e2ca5a2338b6700fd4c147ad2f934c70ecac90