Patch up excessive margins until we can get the real fix plumbed through.

Patch up excessive margins until we can get the real fix plumbed through.

BUG=92000
TEST=NONE

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

[vandebo (ex-Chrome), 2011-10-06 23:36:20]

I'm still waiting on my Mac and Win compiles to finish so I can verify this fix,
but I'm pretty sure this worked during debugging.

[Lei Zhang, 2011-10-07 00:22:10]

LGTM with nit below

http://codereview.chromium.org/8173016/diff/2001/printing/print_job_constants.cc
File printing/print_job_constants.cc (right):

http://codereview.chromium.org/8173016/diff/2001/printing/print_job_constants...
printing/print_job_constants.cc:73: const float kSettingHeaderFooterInterstice =
14.2f;
Shouldn't this be 14.4?

[vandebo (ex-Chrome), 2011-10-07 00:29:50]

http://codereview.chromium.org/8173016/diff/2001/printing/print_job_constants.cc
File printing/print_job_constants.cc (right):

http://codereview.chromium.org/8173016/diff/2001/printing/print_job_constants...
printing/print_job_constants.cc:73: const float kSettingHeaderFooterInterstice =
14.2f;
On 2011/10/07 00:22:10, Lei Zhang wrote:
> Shouldn't this be 14.4?

Nope: http://www.google.com/search?q=.5cm+in+points
7.2 points is exactly 1/10th of an inch

[commit-bot: I haz the power, 2011-10-07 01:59:55]

CQ is trying the patch. Follow status at
https://chromium-status.appspot.com/cq/vandebo@chromium.org/8173016/2001

[commit-bot: I haz the power, 2011-10-07 03:01:56]

The commit queue went berserk retrying too often for a
seemingly flaky test. Builder is linux_rel, revision is 104425, job name
was 8173016-2001 (previous was lost) (previous was lost) (previous was lost)
(previous was lost).

[commit-bot: I haz the power, 2011-10-07 03:47:09]

CQ is trying the patch. Follow status at
https://chromium-status.appspot.com/cq/vandebo@chromium.org/8173016/2001

[commit-bot: I haz the power, 2011-10-07 04:49:18]

The commit queue went berserk retrying too often for a
seemingly flaky test. Builder is linux_rel, revision is 104437, job name
was 8173016-2001 (previous was lost) (previous was lost) (previous was lost)
(previous was lost).