Use valid bpc value in LoadPalette() and ContinueToLoadMask()

Use valid bpc value in LoadPalette() and ContinueToLoadMask()

BUG=408141, 408147
R=tsepez@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/405478d

[Bo Xu, 2014-08-27 20:11:32]

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

[Bo Xu, 2014-08-27 20:11:32]

I changed the GetValidBpc() as well, get rid of the bpc==12. 12 is not a valid
bpc in image dictionary. Only shading dictionary use bpc = 12.

[Tom Sepez, 2014-08-27 20:26:49]

https://codereview.chromium.org/508253003/diff/1/core/src/fpdfapi/fpdf_render...
File core/src/fpdfapi/fpdf_render/fpdf_render_loadimage.cpp (right):

https://codereview.chromium.org/508253003/diff/1/core/src/fpdfapi/fpdf_render...
core/src/fpdfapi/fpdf_render/fpdf_render_loadimage.cpp:858: FX_DWORD bpc =
GetValidBpc();
Do we want to return early if this is 0?

[Bo Xu, 2014-08-27 20:35:18]

Patchset #2 (id:20001) has been deleted

[Bo Xu, 2014-08-27 20:37:24]

https://codereview.chromium.org/508253003/diff/1/core/src/fpdfapi/fpdf_render...
File core/src/fpdfapi/fpdf_render/fpdf_render_loadimage.cpp (right):

https://codereview.chromium.org/508253003/diff/1/core/src/fpdfapi/fpdf_render...
core/src/fpdfapi/fpdf_render/fpdf_render_loadimage.cpp:858: FX_DWORD bpc =
GetValidBpc();
On 2014/08/27 20:26:49, Tom Sepez wrote:
> Do we want to return early if this is 0?

Right! Done. There are a few other instances of not checking bpc==0, I will fix
them in a later cleanup patch too.

[Tom Sepez, 2014-08-27 20:53:19]

lgtm

[Bo Xu, 2014-08-27 21:20:01]

Committed patchset #2 (id:40001) manually as 405478d (presubmit successful).