PDF: Properly apply the CropBox to the MediaBox.

PDF: Properly apply the CropBox to the MediaBox.

Otherwise funky things happen when the CropBox is bigger than the
MediaBox.

BUG=536630
TEST=manual, see bug

Committed: https://crrev.com/601b7fba7533687975cf16882d3ba38964ef4e46
Cr-Commit-Position: refs/heads/master@{#351729}

[Lei Zhang, 2015-09-29 22:16:07]

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

[Lei Zhang, 2015-09-29 22:16:08]

For reference, the bounding boxes are well documented. Just search for: CropBox
MediaBox

[raymes, 2015-10-01 04:19:48]

lgtm

https://codereview.chromium.org/1377523003/diff/1/pdf/pdfium/pdfium_engine.cc
File pdf/pdfium/pdfium_engine.cc (right):

https://codereview.chromium.org/1377523003/diff/1/pdf/pdfium/pdfium_engine.cc...
pdf/pdfium/pdfium_engine.cc:353: // Remember with PDFs, origin is bottom-left.
nit: fill 80chars

[Lei Zhang, 2015-10-01 04:50:03]

https://codereview.chromium.org/1377523003/diff/1/pdf/pdfium/pdfium_engine.cc
File pdf/pdfium/pdfium_engine.cc (right):

https://codereview.chromium.org/1377523003/diff/1/pdf/pdfium/pdfium_engine.cc...
pdf/pdfium/pdfium_engine.cc:353: // Remember with PDFs, origin is bottom-left.
On 2015/10/01 04:19:48, raymes wrote:
> nit: fill 80chars

I did this on purpose since it takes 2 lines anyway.

[Lei Zhang, 2015-10-01 04:50:07]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2015-10-01 04:50:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1377523003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1377523003/1

[commit-bot: I haz the power, 2015-10-01 05:26:06]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-10-01 05:27:20]

Patchset 1 (id:??) landed as
https://crrev.com/601b7fba7533687975cf16882d3ba38964ef4e46
Cr-Commit-Position: refs/heads/master@{#351729}