Add a partial load test for an animated GIF image

Add a partial load test for an animated GIF image

Use HTTP test load-and-stall script to partially load the initial
frame of a animated GIF image.

Render the image over CSS background color green and confirm that
the CSS color appears in the undecoded regions of the image.

TBR=scroggo@chromium.org
BUG=702148
Review-Url: https://codereview.chromium.org/2756443005
Cr-Commit-Position: refs/heads/master@{#457428}
Committed: https://chromium.googlesource.com/chromium/src/+/7dd150883791d4d7aecf7bb93edb3f733fa11237

[Noel Gordon, 2017-03-16 12:18:48]

The CQ bit was checked by noel@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-16 12:18:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Noel Gordon, 2017-03-16 12:24:12]

Description was changed from

==========
Add a partial load test for an animated GIF image

BUG=702148
==========

to

==========
Add a partial load test for an animated GIF image

Use HTTP test load-and-stall script to partially load the initial
frame of a GIF image.

Render the image over CSS background color green and confirm that
the CSS color appears in the undecoded regions of the image.

BUG=702148
==========

[Noel Gordon, 2017-03-16 12:34:51]

Description was changed from

==========
Add a partial load test for an animated GIF image

Use HTTP test load-and-stall script to partially load the initial
frame of a GIF image.

Render the image over CSS background color green and confirm that
the CSS color appears in the undecoded regions of the image.

BUG=702148
==========

to

==========
Add a partial load test for an animated GIF image

Use HTTP test load-and-stall script to partially load the initial
frame of a animated GIF image.

Render the image over CSS background color green and confirm that
the CSS color appears in the undecoded regions of the image.

BUG=702148
==========

[Noel Gordon, 2017-03-16 13:12:41]

The CQ bit was checked by noel@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-16 13:12:49]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-16 14:15:32]

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

[commit-bot: I haz the power, 2017-03-16 14:15:33]

Dry run: This issue passed the CQ dry run.

[Noel Gordon, 2017-03-16 14:18:03]

Description was changed from

==========
Add a partial load test for an animated GIF image

Use HTTP test load-and-stall script to partially load the initial
frame of a animated GIF image.

Render the image over CSS background color green and confirm that
the CSS color appears in the undecoded regions of the image.

BUG=702148
==========

to

==========
Add a partial load test for an animated GIF image

Use HTTP test load-and-stall script to partially load the initial
frame of a animated GIF image.

Render the image over CSS background color green and confirm that
the CSS color appears in the undecoded regions of the image.

TBR=scroggo@chromium.org
BUG=702148
==========

[Noel Gordon, 2017-03-16 14:18:03]

noel@chromium.org changed reviewers:
+ scroggo@chromium.org

[Noel Gordon, 2017-03-16 14:18:10]

The CQ bit was checked by noel@chromium.org

[commit-bot: I haz the power, 2017-03-16 14:18:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-16 14:24:08]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489673890101070,
"parent_rev": "ed1bb4a6435d6b8c65f99795bc16c90696b56282", "commit_rev":
"7dd150883791d4d7aecf7bb93edb3f733fa11237"}

[commit-bot: I haz the power, 2017-03-16 14:24:45]

Description was changed from

==========
Add a partial load test for an animated GIF image

Use HTTP test load-and-stall script to partially load the initial
frame of a animated GIF image.

Render the image over CSS background color green and confirm that
the CSS color appears in the undecoded regions of the image.

TBR=scroggo@chromium.org
BUG=702148
==========

to

==========
Add a partial load test for an animated GIF image

Use HTTP test load-and-stall script to partially load the initial
frame of a animated GIF image.

Render the image over CSS background color green and confirm that
the CSS color appears in the undecoded regions of the image.

TBR=scroggo@chromium.org
BUG=702148

Review-Url: https://codereview.chromium.org/2756443005
Cr-Commit-Position: refs/heads/master@{#457428}
Committed:
https://chromium.googlesource.com/chromium/src/+/7dd150883791d4d7aecf7bb93edb...
==========

[commit-bot: I haz the power, 2017-03-16 14:24:46]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/7dd150883791d4d7aecf7bb93edb...

[Noel Gordon, 2017-03-16 22:00:10]

On 2017/03/16 14:24:46, commit-bot: I haz the power wrote:
> Committed patchset #2 (id:20001) as
>
https://chromium.googlesource.com/chromium/src/+/7dd150883791d4d7aecf7bb93edb...

Does this looks good Leon?

[scroggo_chromium, 2017-03-17 14:15:27]

On 2017/03/16 22:00:10, noel gordon wrote:
> On 2017/03/16 14:24:46, commit-bot: I haz the power wrote:
> > Committed patchset #2 (id:20001) as
> >
>
https://chromium.googlesource.com/chromium/src/+/7dd150883791d4d7aecf7bb93edb...
> 
> Does this looks good Leon?

Should it share code with the APNG test? (crrev.com/2743253009). Otherwise, lgtm

[Noel Gordon, 2017-03-18 01:37:45]

On 2017/03/17 14:15:27, scroggo_chromium wrote:
> On 2017/03/16 22:00:10, noel gordon wrote:
> > On 2017/03/16 14:24:46, commit-bot: I haz the power wrote:
> > > Committed patchset #2 (id:20001) as
> > >
> >
>
https://chromium.googlesource.com/chromium/src/+/7dd150883791d4d7aecf7bb93edb...
> > 
> > Does this looks good Leon?
> 
> Should it share code with the APNG test? (crrev.com/2743253009). Otherwise,
lgtm

Could yes, along with the 4 other tests like this.  If it's work worth doing,
and I'm not sure it is, maybe add to the list of work needed to move to using
SkCodec in image decoders if there is such a list?