Avoid calls to WrapUnique in CPDF_streamparser

Avoid calls to WrapUnique in CPDF_streamparser

Committed: https://pdfium.googlesource.com/pdfium/+/12ff1eba3eaa4a27089f1837a0826dfcac163db2

[Tom Sepez, 2016-11-22 01:12:06]

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

[commit-bot: I haz the power, 2016-11-22 01:12:13]

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

[Tom Sepez, 2016-11-22 01:12:51]

Description was changed from

==========
Avoid calls to WrapUnique in streamparser
==========

to

==========
Avoid calls to WrapUnique in CPDF_streamparser
==========

[Tom Sepez, 2016-11-22 01:12:51]

tsepez@chromium.org changed reviewers:
+ thestig@chromium.org

[Tom Sepez, 2016-11-22 01:13:01]

Lei, ready for review.

[Tom Sepez, 2016-11-22 01:14:50]

https://codereview.chromium.org/2520953004/diff/1/core/fpdfapi/page/cpdf_stre...
File core/fpdfapi/page/cpdf_streamcontentparser.cpp (right):

https://codereview.chromium.org/2520953004/diff/1/core/fpdfapi/page/cpdf_stre...
core/fpdfapi/page/cpdf_streamcontentparser.cpp:674:
m_pSyntax->ReadInlineStream(m_pDocument, std::move(pDict), pCSObj));
Hmmm.  Maybe this needs to return U.P. too.

[Tom Sepez, 2016-11-22 01:16:37]

https://codereview.chromium.org/2520953004/diff/1/core/fpdfapi/page/cpdf_stre...
File core/fpdfapi/page/cpdf_streamcontentparser.cpp (right):

https://codereview.chromium.org/2520953004/diff/1/core/fpdfapi/page/cpdf_stre...
core/fpdfapi/page/cpdf_streamcontentparser.cpp:674:
m_pSyntax->ReadInlineStream(m_pDocument, std::move(pDict), pCSObj));
On 2016/11/22 01:14:49, Tom Sepez wrote:
> Hmmm.  Maybe this needs to return U.P. too.

Turns out it already does.  Fixed.

[Lei Zhang, 2016-11-22 01:28:33]

lgtm

[Tom Sepez, 2016-11-22 01:29:30]

The CQ bit was checked by tsepez@chromium.org

[commit-bot: I haz the power, 2016-11-22 01:29:34]

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-22 01:47:29]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1479778170327630,
"parent_rev": "8edd6d72afec2bcc2247dd85edec99d156f13870", "commit_rev":
"12ff1eba3eaa4a27089f1837a0826dfcac163db2"}

[commit-bot: I haz the power, 2016-11-22 01:47:31]

Description was changed from

==========
Avoid calls to WrapUnique in CPDF_streamparser
==========

to

==========
Avoid calls to WrapUnique in CPDF_streamparser

Committed:
https://pdfium.googlesource.com/pdfium/+/12ff1eba3eaa4a27089f1837a0826dfcac16...
==========

[commit-bot: I haz the power, 2016-11-22 01:47:32]

Committed patchset #2 (id:20001) as
https://pdfium.googlesource.com/pdfium/+/12ff1eba3eaa4a27089f1837a0826dfcac16...