Revert "Fix a problem that JP2 image is not displayed because index color space is used"

Revert "Fix a problem that JP2 image is not displayed because index color space is used"

This reverts commit 5a0e504d53195892458d819e52c62bea0c710bd5.

Original review URL https://codereview.chromium.org/1009513003

Reason for revert: New "corpus" tests show that this is no longer rendering some images.  See, e.g. http://chromegw/i/client.pdfium/builders/linux/builds/144/steps/corpus%20tests/logs/stdio for those that diff'd.

TBR=jun_fang@foxitsoftware.com

Committed: https://pdfium.googlesource.com/pdfium/+/a4161734ee7f2b64cbe3431385a4b4c9a426ca47

[Tom Sepez, 2015-03-17 18:47:21]

tsepez@chromium.org changed reviewers:
+ jun_fang@foxitsoftware.com

[Tom Sepez, 2015-03-17 18:48:17]

Sorry, looks like the new code isn't working right.

Note that you will have to gclient sync to get the new repository pulled into
your checkout.  Thanks.

[Tom Sepez, 2015-03-17 18:48:27]

Committed patchset #1 (id:1) manually as
a4161734ee7f2b64cbe3431385a4b4c9a426ca47 (presubmit successful).

[jun_fang, 2015-03-17 18:54:24]

On 2015/03/17 18:48:27, Tom Sepez wrote:
> Committed patchset #1 (id:1) manually as
> a4161734ee7f2b64cbe3431385a4b4c9a426ca47 (presubmit successful).

Hi Tom,

What's 'corpus' test? Can I run it before landing the fix next time?

[jun_fang, 2015-03-17 18:58:32]

https://codereview.chromium.org/1016823003/diff/1/core/src/fpdfapi/fpdf_rende...
File core/src/fpdfapi/fpdf_render/fpdf_render_loadimage.cpp (left):

https://codereview.chromium.org/1016823003/diff/1/core/src/fpdfapi/fpdf_rende...
core/src/fpdfapi/fpdf_render/fpdf_render_loadimage.cpp:508: FX_DWORD arrSize =
pDecode ? pDecode->GetCount() : 0;
line 508 to line 511 should be moved to line 513.