Split off CPDF_Document into its own .cpp/.h files.

Split off CPDF_Document into its own .cpp/.h files.

R=dsinclair@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/310438fb97a2363bd0901401ad2506febd64361b

[Tom Sepez, 2016-03-08 20:51:20]

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

[Tom Sepez, 2016-03-08 20:51:21]

A larger patch to make Dan happy.

[dsinclair, 2016-03-08 20:57:08]

lgtm

https://codereview.chromium.org/1780503002/diff/1/core/src/fpdfapi/fpdf_parse...
File core/src/fpdfapi/fpdf_parser/cpdf_document.cpp (right):

https://codereview.chromium.org/1780503002/diff/1/core/src/fpdfapi/fpdf_parse...
core/src/fpdfapi/fpdf_parser/cpdf_document.cpp:344: if (m_pDocRender) {
if (!m_pDocReader)
  return;
?

https://codereview.chromium.org/1780503002/diff/1/core/src/fpdfapi/fpdf_parse...
core/src/fpdfapi/fpdf_parser/cpdf_document.cpp:348: }
nit: remove {}

[Tom Sepez, 2016-03-08 21:10:31]

https://codereview.chromium.org/1780503002/diff/1/core/src/fpdfapi/fpdf_parse...
File core/src/fpdfapi/fpdf_parser/cpdf_document.cpp (right):

https://codereview.chromium.org/1780503002/diff/1/core/src/fpdfapi/fpdf_parse...
core/src/fpdfapi/fpdf_parser/cpdf_document.cpp:344: if (m_pDocRender) {
On 2016/03/08 20:57:08, dsinclair wrote:
> if (!m_pDocReader)
>   return;
> ?

Done.

https://codereview.chromium.org/1780503002/diff/1/core/src/fpdfapi/fpdf_parse...
core/src/fpdfapi/fpdf_parser/cpdf_document.cpp:348: }
On 2016/03/08 20:57:08, dsinclair wrote:
> nit: remove {}

Done.

[Tom Sepez, 2016-03-08 21:10:59]

Description was changed from

==========
Split off CPDF_Document into its own .cpp/.h files.
==========

to

==========
Split off CPDF_Document into its own .cpp/.h files.

R=dsinclair@chromium.org

Committed:
https://pdfium.googlesource.com/pdfium/+/310438fb97a2363bd0901401ad2506febd64...
==========

[Tom Sepez, 2016-03-08 21:11:00]

Committed patchset #3 (id:40001) manually as
310438fb97a2363bd0901401ad2506febd64361b (presubmit successful).