Block printing through JS when printing is disabled by policy or pref

Block printing through JS when printing is disabled by policy or pref

This CL ensures that when the PrintingEnabled policy or the corresponding
printing.enabled pref is set, the JS window.print() call is completely
ignored. Previously, it was bringing up a print preview dialog.

BUG=107709
TEST=Policy and pref both successfully block JS printing


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148622

[bartfab (slow), 2012-07-13 09:56:22]

Hi Albert

Could you please review this CL? It builds directly on the functionality you
added for bug 117955.

[Albert Bodenhamer, 2012-07-13 16:27:31]

lgtm

[commit-bot: I haz the power, 2012-07-17 08:16:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bartfab@chromium.org/10689190/1

[commit-bot: I haz the power, 2012-07-17 09:22:20]

Try job failure for 10689190-1 (retry) on mac_rel for steps "browser_tests,
unit_tests".
It's a second try, previously, steps "browser_tests, unit_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-07-26 12:00:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bartfab@chromium.org/10689190/15001

[commit-bot: I haz the power, 2012-07-26 12:18:34]

Try job failure for 10689190-15001 (retry) on mac_rel for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[bartfab (slow), 2012-07-26 16:16:56]

Albert, I feel this CL has changed enough that it would be good if you could
have another quick look before I try to commit it again. Could you review again?

[Albert Bodenhamer, 2012-07-26 19:00:49]

still lgtm

[commit-bot: I haz the power, 2012-07-26 19:06:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bartfab@chromium.org/10689190/28002

[commit-bot: I haz the power, 2012-07-26 21:03:41]

Change committed as 148622