Use new API instead of the deprecated ones

Use new API instead of the deprecated ones.

This change will allow us to remove the old API in follow up clean up.

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

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

[yujieqin, 2016-02-23 14:22:14]

Description was changed from

==========
Use new API instead of the deprecated ones

BUG=skia:
==========

to

==========
Use new API instead of the deprecated ones

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

[yujieqin, 2016-02-23 14:23:20]

Description was changed from

==========
Use new API instead of the deprecated ones

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

to

==========
Use new API instead of the deprecated ones.

This change will allow us to remove the old API in follow up clean up.

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

[yujieqin, 2016-02-23 14:23:20]

yujieqin@google.com changed reviewers:
+ msarett@google.com, scroggo@google.com

[yujieqin, 2016-02-23 14:23:35]

The CQ bit was checked by yujieqin@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-02-23 14:23:42]

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

[commit-bot: I haz the power, 2016-02-23 14:23:43]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2016-02-23 20:23 UTC

[msarett, 2016-02-23 14:40:17]

lgtm

[yujieqin, 2016-02-23 14:41:59]

The CQ bit was unchecked by yujieqin@google.com

[yujieqin, 2016-02-23 14:42:02]

The CQ bit was checked by yujieqin@google.com

[commit-bot: I haz the power, 2016-02-23 14:42:12]

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

[commit-bot: I haz the power, 2016-02-23 14:49:40]

Description was changed from

==========
Use new API instead of the deprecated ones.

This change will allow us to remove the old API in follow up clean up.

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

to

==========
Use new API instead of the deprecated ones.

This change will allow us to remove the old API in follow up clean up.

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

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

[commit-bot: I haz the power, 2016-02-23 14:49:41]

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

[scroggo, 2016-02-23 15:55:23]

https://codereview.chromium.org/1728553002/diff/1/src/codec/SkRawCodec.cpp
File src/codec/SkRawCodec.cpp (right):

https://codereview.chromium.org/1728553002/diff/1/src/codec/SkRawCodec.cpp#ne...
src/codec/SkRawCodec.cpp:575: if (error == ::piex::Error::kOk &&
imageData.preview.length > 0) {
Unfortunately, this will mean that we need to update piex in nyc-dev in order to
merge Skia into master. The way we typically handle this is is with macros.
Something like:

#ifdef SK_LEGACY_PREVIEW_LENGTH
  if (error == ::piex::Error::kOk && imageData.preview_length > 0) {
#else
  if (error == ::piex::Error::kOk && imageData.preview.length > 0) {
#endif

We can define SK_LEGACY_PREVIEW_LENGTH in Android until piex gets updated to
have the new API. Then we can delete the old code in Skia and then delete the
old API in piex.

[yujieqin, 2016-02-23 16:26:31]

It should be fine now, the PIEX is merged.

[scroggo, 2016-02-23 16:51:29]

On 2016/02/23 16:26:31, yujieqin wrote:
> It should be fine now, the PIEX is merged.

Great, thanks!