Fail to decode jpegs on a large number of progressive scans

Fail to decode jpegs on a large number of progressive scans

BUG=642462
R=scroggo@chromium.org

Committed: https://crrev.com/75bac58d037d14d78babf75e674654062a1c85eb
Cr-Commit-Position: refs/heads/master@{#430321}

[msarett1, 2016-11-07 14:07:28]

msarett@chromium.org changed reviewers:
+ msarett@chromium.org, scroggo@chromium.org

[msarett1, 2016-11-07 14:07:44]

Patchset #1 (id:1) has been deleted

[scroggo_chromium, 2016-11-07 14:36:26]

Code lgtm. Shall we add a test?

(Also, should we add the same check to SkCodec?

[msarett, 2016-11-07 15:41:36]

On 2016/11/07 14:36:26, scroggo_chromium wrote:
> Code lgtm. Shall we add a test?
> 

Yes, test added.

> (Also, should we add the same check to SkCodec?

Yes, I will do this.

[msarett, 2016-11-07 15:42:28]

https://codereview.chromium.org/2482883002/diff/40001/third_party/WebKit/Sour...
File
third_party/WebKit/Source/platform/image-decoders/jpeg/JPEGImageDecoderTest.cpp
(right):

https://codereview.chromium.org/2482883002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/image-decoders/jpeg/JPEGImageDecoderTest.cpp:302:
// This tests decoding a JPEG with many progressive scans.  Decoding should
This test takes about 16ms to run.

[scroggo_chromium, 2016-11-07 15:47:02]

lgtm

[msarett, 2016-11-07 15:49:35]

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

[commit-bot: I haz the power, 2016-11-07 15:49:47]

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, 2016-11-07 15:52:08]

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

[commit-bot: I haz the power, 2016-11-07 15:52:09]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[msarett, 2016-11-07 15:59:31]

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

[commit-bot: I haz the power, 2016-11-07 15:59:50]

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, 2016-11-07 16:02:26]

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

[commit-bot: I haz the power, 2016-11-07 16:02:27]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[msarett, 2016-11-07 16:03:18]

Patchset #3 (id:60001) has been deleted

[commit-bot: I haz the power, 2016-11-07 18:45:09]

Description was changed from

==========
Fail to decode jpegs on a large number of progressive scans

BUG=642462
==========

to

==========
Fail to decode jpegs on a large number of progressive scans

BUG=642462
R=scroggo@chromium.org

Committed: https://crrev.com/75bac58d037d14d78babf75e674654062a1c85eb
Cr-Commit-Position: refs/heads/master@{#430321}
==========

[commit-bot: I haz the power, 2016-11-07 18:45:10]

Patchset 2 (id:??) landed as
https://crrev.com/75bac58d037d14d78babf75e674654062a1c85eb
Cr-Commit-Position: refs/heads/master@{#430321}