Remove ownership parameter from GrResourceProvider::wrapBackendTextureAsRenderTarget

Remove ownership parameter from GrResourceProvider::wrapBackendTextureAsRenderTarget

Remove ownership parameter from
GrResourceProvider::wrapBackendTextureAsRenderTarget.
The function leaks the texture id if kAdopt_LifeCycle is passed.

There is no public API to access the parameter.

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

Committed: https://skia.googlesource.com/skia/+/49c4c22b3744d18e1cb38ecabcb8839d2e26afe0

[Kimmo Kinnunen, 2016-03-31 11:32:53]

Description was changed from

==========
Remove ownership parameter from
GrResourceProvider::wrapBackendTextureAsRenderTarget

Remove ownership parameter from
GrResourceProvider::wrapBackendTextureAsRenderTarget.
The function leaks the texture id if kAdopt_LifeCycle is passed.

There is no public API to access the parameter.

BUG=594928
==========

to

==========
Remove ownership parameter from
GrResourceProvider::wrapBackendTextureAsRenderTarget

Remove ownership parameter from
GrResourceProvider::wrapBackendTextureAsRenderTarget.
The function leaks the texture id if kAdopt_LifeCycle is passed.

There is no public API to access the parameter.

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

[Kimmo Kinnunen, 2016-03-31 11:42:35]

kkinnunen@nvidia.com changed reviewers:
+ bsalomon@google.com

[Kimmo Kinnunen, 2016-03-31 11:42:36]

I have another way of fixing, but it is either:

a) weird: using GrGpuResource::LifeCycle to denote also backend object
lifecycle. Currently it denotes the whole GrGpuResource "lifecycle".

or

b) laboursome: refactoring GrGpuResource::LifeCycle to GrBackendObjectLifeCycle
(internal, adopt, borrow) and to denote "cache/do not cache" with SkBudgeted.

[bsalomon, 2016-03-31 14:50:24]

lgtm

[Kimmo Kinnunen, 2016-04-01 06:36:43]

The CQ bit was checked by kkinnunen@nvidia.com

[commit-bot: I haz the power, 2016-04-01 06:36:54]

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

[commit-bot: I haz the power, 2016-04-01 11:50:43]

Description was changed from

==========
Remove ownership parameter from
GrResourceProvider::wrapBackendTextureAsRenderTarget

Remove ownership parameter from
GrResourceProvider::wrapBackendTextureAsRenderTarget.
The function leaks the texture id if kAdopt_LifeCycle is passed.

There is no public API to access the parameter.

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

to

==========
Remove ownership parameter from
GrResourceProvider::wrapBackendTextureAsRenderTarget

Remove ownership parameter from
GrResourceProvider::wrapBackendTextureAsRenderTarget.
The function leaks the texture id if kAdopt_LifeCycle is passed.

There is no public API to access the parameter.

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

Committed:
https://skia.googlesource.com/skia/+/49c4c22b3744d18e1cb38ecabcb8839d2e26afe0
==========

[commit-bot: I haz the power, 2016-04-01 11:50:44]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/49c4c22b3744d18e1cb38ecabcb8839d2e26afe0