Fix uninitialized nresults and RGB

Fix uninitialized nresults and RGB

BUG=387785
R=palmer@chromium.org

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

[palmer, 2014-07-07 18:29:19]

https://codereview.chromium.org/362403002/diff/1/core/src/fpdfapi/fpdf_page/f...
File core/src/fpdfapi/fpdf_page/fpdf_page_colors.cpp (right):

https://codereview.chromium.org/362403002/diff/1/core/src/fpdfapi/fpdf_page/f...
core/src/fpdfapi/fpdf_page/fpdf_page_colors.cpp:1031: m_pFunc->Call(pBuf,
m_nComponents, results, nresults);
The way |nresults| is/was used makes me wonder if it is supposed to be an
out-param. I.e  should you be calling like this:

    m_pFunc->Call(pBuf, m_nComponents, results, &nresults);

? Because as it is now, this function will always return FALSE and lines 1035 –
1036 are dead code.

[palmer, 2014-07-07 18:39:10]

LGTM.

https://codereview.chromium.org/362403002/diff/1/core/src/fpdfapi/fpdf_page/f...
File core/src/fpdfapi/fpdf_page/fpdf_page_colors.cpp (right):

https://codereview.chromium.org/362403002/diff/1/core/src/fpdfapi/fpdf_page/f...
core/src/fpdfapi/fpdf_page/fpdf_page_colors.cpp:1031: m_pFunc->Call(pBuf,
m_nComponents, results, nresults);
Ohh, I see, it's a non-const reference. Carry on then...

[jun_fang, 2014-07-07 21:04:36]

Committed patchset #1 manually as rf3227ae (presubmit successful).