Remove SkImageRef and related functionality.

Remove SkImageRef and related functionality.

From here on out, use SkDecodingImageGenerator+SkDiscardablePixelRef.

R=scroggo@google.com

Committed: https://skia.googlesource.com/skia/+/479bb60

[hal.canary, 2013-12-06 15:53:54]

NOT READY for primetime

[hal.canary, 2014-01-02 19:20:21]

This CL is ready to go once I get an okay from our Android people.

[scroggo, 2014-01-07 16:01:05]

https://codereview.chromium.org/100183002/diff/90001/src/ports/SkGlobalInitia...
File src/ports/SkGlobalInitialization_default.cpp (left):

https://codereview.chromium.org/100183002/diff/90001/src/ports/SkGlobalInitia...
src/ports/SkGlobalInitialization_default.cpp:110:
SkImages::InitializeFlattenables();
Please remove similar code in src/ports/SkImageDecoder_empty.cpp and
src/ports/src/ports/SkGlobalInitialization_chromium.cpp

[scroggo, 2014-01-07 16:04:36]

On 2014/01/07 16:01:05, scroggo wrote:
>
https://codereview.chromium.org/100183002/diff/90001/src/ports/SkGlobalInitia...
> File src/ports/SkGlobalInitialization_default.cpp (left):
> 
>
https://codereview.chromium.org/100183002/diff/90001/src/ports/SkGlobalInitia...
> src/ports/SkGlobalInitialization_default.cpp:110:
> SkImages::InitializeFlattenables();
> Please remove similar code in src/ports/SkImageDecoder_empty.cpp and
> src/ports/src/ports/SkGlobalInitialization_chromium.cpp

It appears that gyp/public_headers.gypi also references SkImages.h

[hal.canary, 2014-01-08 21:38:42]

On 2014/01/07 16:01:05, scroggo wrote:
>
https://codereview.chromium.org/100183002/diff/90001/src/ports/SkGlobalInitia...
> File src/ports/SkGlobalInitialization_default.cpp (left):
> 
>
https://codereview.chromium.org/100183002/diff/90001/src/ports/SkGlobalInitia...
> src/ports/SkGlobalInitialization_default.cpp:110:
> SkImages::InitializeFlattenables();
> Please remove similar code in src/ports/SkImageDecoder_empty.cpp and
> src/ports/src/ports/SkGlobalInitialization_chromium.cpp

Done.

On 2014/01/07 16:04:36, scroggo wrote:
> 
> It appears that gyp/public_headers.gypi also references SkImages.h

Done.

[scroggo, 2014-01-08 21:43:57]

lgtm

[scroggo, 2014-04-03 20:26:35]

On 2014/01/08 21:43:57, scroggo wrote:
> lgtm

Looking back at this as I try to integrate SkDecodingImageGenerator into
Android, and it does not lgtm. The header files that Android needs to access are
in src, and there is no ashmem replacement. We also need a way to support
unpremul. Lastly, it doesn't support index_8. That might be okay, but the
problem is that in Android's BitmapFactory, they won't know that the image is
index_8 until they do a bounds decode, at which point we'll need to add special
code to go back and do a full decode (since discardable is no longer an option).

I've started on these in https://codereview.chromium.org/223903007/ (which does
not have ashmem or index_8).

[scroggo, 2014-06-04 14:22:49]

On 2014/04/03 20:26:35, scroggo wrote:
> On 2014/01/08 21:43:57, scroggo wrote:
> > lgtm
> 
> Looking back at this as I try to integrate SkDecodingImageGenerator into
> Android, and it does not lgtm. The header files that Android needs to access
are
> in src, and there is no ashmem replacement. We also need a way to support
> unpremul. Lastly, it doesn't support index_8. That might be okay, but the
> problem is that in Android's BitmapFactory, they won't know that the image is
> index_8 until they do a bounds decode, at which point we'll need to add
special
> code to go back and do a full decode (since discardable is no longer an
option).
> 
> I've started on these in https://codereview.chromium.org/223903007/ (which
does
> not have ashmem or index_8).

I have removed all SkImageRef functionality from Android. LGTM

[hal.canary, 2014-06-04 14:25:10]

On 2014/06/04 14:22:49, scroggo wrote:

> I have removed all SkImageRef functionality from Android. LGTM


Let me rebase, then I'll commit it.

[reed1, 2014-06-04 14:28:45]

On 2014/06/04 14:25:10, Hal Canary wrote:
> On 2014/06/04 14:22:49, scroggo wrote:
> 
> > I have removed all SkImageRef functionality from Android. <span
id="abbrex-spnAbbr_419" class="abbrex-Abbr abbrex-Abbr_LGTM abbrex-AbbrUnknown
abbrex-AbbrUnknownVisible">LGTM</span>
> 
> 
> Let me rebase, then I'll commit it.

woot

[hal.canary, 2014-06-04 18:39:03]

Committed patchset #3 manually as r479bb60 (presubmit successful).