ugrr... Fixing printing scaling again.

ugrr... Fixing printing scaling again.
Not the nicest fix, but it should do the job. Now, we'll try to use default scaling (72dpi) whenever we can.
If sceen resolution droped below the point where metafile can't accomodate whole PDF page anymore, we'll use this
hack to pass custom scale in the TLS for the upper layer.

BUG=125499
TEST=Print PDF using print preview and system print dialog. Same for webpage. Make screen resolution 800x600 and repeat.

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

[gene, 2012-05-02 23:25:29]

Please make sure it is ok calling printing/* from ppapi_plugin_instance. Looks
like we are doing this already for units conversions.

[Albert Bodenhamer, 2012-05-02 23:31:28]

On 2012/05/02 23:25:29, gene wrote:
> Please make sure it is ok calling printing/* from ppapi_plugin_instance. Looks
> like we are doing this already for units conversions.

Running checkdeps locally and starting try jobs should give a better answer on
the dependencies.

[Albert Bodenhamer, 2012-05-02 23:49:18]

lgtm

https://chromiumcodereview.appspot.com/10348002/diff/3002/chrome/renderer/pri...
File chrome/renderer/print_web_view_helper_win.cc (right):

https://chromiumcodereview.appspot.com/10348002/diff/3002/chrome/renderer/pri...
chrome/renderer/print_web_view_helper_win.cc:127: // TODO(gene): We should
revisit this solution for the next versions.
In my dream world we can just create a DC that matches the capabilities of the
page and avoid the need to do funky things with scaling factors and viewports.

Perhaps create a bug for the TODO to serve as a place for discussion?

[gene, 2012-05-03 00:15:55]

created new issue as suggested
(https://code.google.com/p/chromium/issues/detail?id=126037)
referenced it in the TODO
feel free to take a look

[commit-bot: I haz the power, 2012-05-03 00:22:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gene@chromium.org/10348002/3005

[commit-bot: I haz the power, 2012-05-03 00:22:32]

Presubmit check for 10348002-3005 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    webkit/plugins/ppapi

Presubmit checks took 3.6s to calculate.

[gene, 2012-05-03 17:21:08]

As an OWNER of PPAPI could you please take a look?

This is a bad hack I have to do in order to fix printing scaling issue, which I
am not super happy about.
There is a list of bugs related to chrome printing scale issues: 125499, 122363,
111170, 124162, cr-os 16088 etc.

I would like to find a better solution for M21 and remove this hack as soon as I
can.
I can give you more details on this if you like.

[brettw, 2012-05-03 20:50:00]

webkit LGTM

[commit-bot: I haz the power, 2012-05-03 20:51:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gene@chromium.org/10348002/3005

[commit-bot: I haz the power, 2012-05-03 23:53:31]

Change committed as 135241