Verify the full size returned by PIEX

Verify the full size returned by PIEX

As the full size info is only optional, one needs to verify the values
before using them

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

Committed: https://skia.googlesource.com/skia/+/d0e0885d90cecd43b717d89b565d9a713cffd90c

[yujieqin, 2016-03-03 14:20:15]

Description was changed from

==========
Verify the full size returned by PIEX

As the full size info is only optional, one needs to verify the values
before using them

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

to

==========
Verify the full size returned by PIEX

As the full size info is only optional, one needs to verify the values
before using them

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

[yujieqin, 2016-03-03 14:20:45]

Description was changed from

==========
Verify the full size returned by PIEX

As the full size info is only optional, one needs to verify the values
before using them

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

to

==========
Verify the full size returned by PIEX

As the full size info is only optional, one needs to verify the values
before using them

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

[yujieqin, 2016-03-03 14:21:30]

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

[yujieqin, 2016-03-03 15:07:57]

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

[commit-bot: I haz the power, 2016-03-03 15:08:07]

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

[commit-bot: I haz the power, 2016-03-03 15:08:08]

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-03-03 21:08 UTC

[msarett, 2016-03-03 15:30:09]

In some cases, PIEX will succeed, but not find the size?  In that case, we must
fall back on the dng_sdk to get the size?

lgtm

[commit-bot: I haz the power, 2016-03-03 15:31:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-03 15:31:15]

Dry run: This issue passed the CQ dry run.

[yujieqin, 2016-03-03 15:37:15]

On 2016/03/03 15:30:09, msarett wrote:
> In some cases, PIEX will succeed, but not find the size?  In that case, we
must
> fall back on the dng_sdk to get the size?
> 
> lgtm

Yes, as the fields are optional, PIEX will succeed without them. And we must
fall back on dng_sdk.

[yujieqin, 2016-03-03 15:37:24]

The CQ bit was checked by yujieqin@google.com

[commit-bot: I haz the power, 2016-03-03 15:37:34]

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

[commit-bot: I haz the power, 2016-03-03 15:38:29]

Description was changed from

==========
Verify the full size returned by PIEX

As the full size info is only optional, one needs to verify the values
before using them

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

to

==========
Verify the full size returned by PIEX

As the full size info is only optional, one needs to verify the values
before using them

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

Committed:
https://skia.googlesource.com/skia/+/d0e0885d90cecd43b717d89b565d9a713cffd90c
==========

[commit-bot: I haz the power, 2016-03-03 15:38:30]

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