gerryroxby@ Could you please help us with the sample html page for this issue for further triage.

There's no sample HTML page. The attached PDF is what the user is trying to print. It's A4 landscape, but for some reason we are not auto-rotating it.

This happens on Windows too. 

The problem is that this PDF page is portrait oriented with a rotation of -90. It appears that negative rotation values are not handled correctly; we expect rotation to be 0 to 270. One example is in pdf_transform.cc; this hits a NOTREACHED() in line 133 in a debug build. If you manually edit the PDF so the page has rotation 270 instead, which should be equivalent, everything works as anticipated. Looks like we need to normalize rotation values read from PDFs.

The following revision refers to this bug:
  https://pdfium.googlesource.com/pdfium/+/ce8e51e6c444f6caaa160bf8b1decd5d7ec84e6f

commit ce8e51e6c444f6caaa160bf8b1decd5d7ec84e6f
Author: rbpotter <rbpotter@chromium.org>
Date: Sat Apr 29 00:13:24 2017

Fix rotations

Normalize rotations read from PDF documents. Make FPDFPage_GetRotation
always return a value 0 to 3 as specified in fpdf_edit.h instead of
returning the page rotation / 90 (page rotation may be negative for some
PDFs).

BUG= chromium:713197 

Change-Id: Ie477803f7d298b777a3ace89b21cfda8b7f6808b
Reviewed-on: https://pdfium-review.googlesource.com/4532
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>

[add] https://crrev.com/ce8e51e6c444f6caaa160bf8b1decd5d7ec84e6f/testing/resources/bug_713197.pdf
[modify] https://crrev.com/ce8e51e6c444f6caaa160bf8b1decd5d7ec84e6f/core/fpdfapi/page/cpdf_page.cpp
[modify] https://crrev.com/ce8e51e6c444f6caaa160bf8b1decd5d7ec84e6f/core/fpdfapi/page/cpdf_page.h
[modify] https://crrev.com/ce8e51e6c444f6caaa160bf8b1decd5d7ec84e6f/fpdfsdk/fpdfedit_embeddertest.cpp
[add] https://crrev.com/ce8e51e6c444f6caaa160bf8b1decd5d7ec84e6f/testing/resources/bug_713197.in
[modify] https://crrev.com/ce8e51e6c444f6caaa160bf8b1decd5d7ec84e6f/fpdfsdk/fpdfeditpage.cpp
[modify] https://crrev.com/ce8e51e6c444f6caaa160bf8b1decd5d7ec84e6f/fpdfsdk/fpdfview.cpp

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7988a5c627d0f19ab14ccddfa532db52076c6ce5

commit 7988a5c627d0f19ab14ccddfa532db52076c6ce5
Author: pdfium-deps-roller@chromium.org <pdfium-deps-roller@chromium.org>
Date: Sat Apr 29 02:21:35 2017

Roll src/third_party/pdfium/ 3b91290ba..ce8e51e6c (5 commits)

https://pdfium.googlesource.com/pdfium.git/+log/3b91290ba31e..ce8e51e6c444

$ git log 3b91290ba..ce8e51e6c --date=short --no-merges --format='%ad %ae %s'
2017-04-28 rbpotter Fix rotations
2017-04-28 thestig Clean up private methods in CBC_C40Encoder.
2017-04-28 tsepez Remove some more |new|s, part 4.
2017-04-28 thestig Limit recursion in CXML_Parser::ParseElement().
2017-04-28 thestig Disallow CPDF_FormField with deep node trees.

Created with:
  roll-dep src/third_party/pdfium
BUG= 713197 , 716526 


Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


TBR=dsinclair@chromium.org

Change-Id: I81ccbaaacbb34cc9923714ca87841f2c756d6b3c
Reviewed-on: https://chromium-review.googlesource.com/490699
Reviewed-by: <pdfium-deps-roller@chromium.org>
Commit-Queue: <pdfium-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#468212}
[modify] https://crrev.com/7988a5c627d0f19ab14ccddfa532db52076c6ce5/DEPS

Tested this on Chrome Canary (60.0.3086.0) with Windows 10 and PDF is now correctly autorotated. Thus marking fixed for now, but please comment if this is not working on version 60.0.3086.0 or later and I will re-open the issue.