Use custom scrollbars for print preview on non-retina mac displays.

Use custom scrollbars for print preview on non-retina mac displays.

This is a hack to workaround a blink bug where overlay scrollbars aren't
rendered correctly in print preview. This bug is being looked at but
until then we use custom scrollbars that imitate mac scrollbars only
in this specific case.

BUG=466039
Committed: https://crrev.com/6826023abd24c8aa8981f4f0fadcf98f0f962bba
Cr-Commit-Position: refs/heads/master@{#322331}

[raymes, 2015-03-26 02:10:57]

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

[Lei Zhang, 2015-03-26 02:12:19]

My CSS-fu is weak, so I defer to Sam if he's comfortable reviewing this.

[Sam McNally, 2015-03-26 02:31:54]

LGTM

https://codereview.chromium.org/1036943004/diff/40001/chrome/browser/resource...
File chrome/browser/resources/component_extension_resources.grd (right):

https://codereview.chromium.org/1036943004/diff/40001/chrome/browser/resource...
chrome/browser/resources/component_extension_resources.grd:182: <include
name="IDR_PDF_SCROLLBARS_CSS" file="pdf/scrollbars-mac.css"
allowexternalscript="true" type="BINDATA" />
scrollbars_mac.css

Remove allowexternalscript="true".

https://codereview.chromium.org/1036943004/diff/40001/chrome/browser/resource...
File chrome/browser/resources/pdf/scrollbars-mac.css (right):

https://codereview.chromium.org/1036943004/diff/40001/chrome/browser/resource...
chrome/browser/resources/pdf/scrollbars-mac.css:5:
(-webkit-max-device-pixel-ratio: 1.1) {
1.0

https://codereview.chromium.org/1036943004/diff/40001/chrome/browser/resource...
chrome/browser/resources/pdf/scrollbars-mac.css:26: background:
rgb(126,126,126);
background-color

[Lei Zhang, 2015-03-26 02:33:18]

.grd file lgtm

[raymes, 2015-03-26 04:29:58]

The CQ bit was checked by raymes@chromium.org

[raymes, 2015-03-26 04:29:59]

The patchset sent to the CQ was uploaded after l-g-t-m from
thestig@chromium.org, sammc@chromium.org
Link to the patchset: https://codereview.chromium.org/1036943004/#ps120001
(title: " ")

[raymes, 2015-03-26 04:30:34]

https://codereview.chromium.org/1036943004/diff/40001/chrome/browser/resource...
File chrome/browser/resources/component_extension_resources.grd (right):

https://codereview.chromium.org/1036943004/diff/40001/chrome/browser/resource...
chrome/browser/resources/component_extension_resources.grd:182: <include
name="IDR_PDF_SCROLLBARS_CSS" file="pdf/scrollbars-mac.css"
allowexternalscript="true" type="BINDATA" />
On 2015/03/26 02:31:54, Sam McNally wrote:
> scrollbars_mac.css
> 
> Remove allowexternalscript="true".

Done.

https://codereview.chromium.org/1036943004/diff/40001/chrome/browser/resource...
File chrome/browser/resources/pdf/scrollbars-mac.css (right):

https://codereview.chromium.org/1036943004/diff/40001/chrome/browser/resource...
chrome/browser/resources/pdf/scrollbars-mac.css:5:
(-webkit-max-device-pixel-ratio: 1.1) {
On 2015/03/26 02:31:54, Sam McNally wrote:
> 1.0

Done.

https://codereview.chromium.org/1036943004/diff/40001/chrome/browser/resource...
chrome/browser/resources/pdf/scrollbars-mac.css:26: background:
rgb(126,126,126);
On 2015/03/26 02:31:54, Sam McNally wrote:
> background-color

Done.

[commit-bot: I haz the power, 2015-03-26 04:30:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1036943004/120001

[commit-bot: I haz the power, 2015-03-26 04:38:24]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-03-26 04:38:25]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[raymes, 2015-03-26 05:14:18]

The CQ bit was checked by raymes@chromium.org

[raymes, 2015-03-26 05:14:18]

The patchset sent to the CQ was uploaded after l-g-t-m from
thestig@chromium.org, sammc@chromium.org
Link to the patchset: https://codereview.chromium.org/1036943004/#ps140001
(title: " ")

[commit-bot: I haz the power, 2015-03-26 05:14:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1036943004/140001

[commit-bot: I haz the power, 2015-03-26 05:57:09]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2015-03-26 05:57:48]

Patchset 8 (id:??) landed as
https://crrev.com/6826023abd24c8aa8981f4f0fadcf98f0f962bba
Cr-Commit-Position: refs/heads/master@{#322331}

[raymes, 2015-04-13 01:48:38]

A revert of this CL (patchset #8 id:140001) has been created in
https://codereview.chromium.org/1085483002/ by raymes@chromium.org.

The reason for reverting is: The bug this hack is working around was fixed in  
http://src.chromium.org/viewvc/blink?view=rev&rev=193593. This hack can now be
reverted..