1280159 - Page Setup Margin Widths use Millimeters Instead of Inches (for paper Legal US e.g.)

Status: VERIFIED FIXED

(Core :: Printing: Setup, defect)

Description

[Lou Petro]

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:47.0) Gecko/20100101 Firefox/47.0
Build ID: 20160604131506

Steps to reproduce:

Select File, Page Setup..., Margins & Header/Footer.


Actual results:

The margins are in millimeters. Prior to version 46, the margins were in inches.


Expected results:

The margins should be in inches as I'm still using Letter size (8.5" x 11") paper and not a metric size. Using about:config, I reset all prefs with "print, printer, margins, or paper" to default. The margins remain in millimeters (10mm) in the Page Setup... dialog box although in about:config they are shown in inches (0.393750011920929" which is a close approximation to 10mm). I have found no way to change back to inches. See https://support.mozilla.org/en-US/questions/1126411

Comment 1

[Loic]

Is it reproducible with a fresh profile?
https://support.mozilla.org/en-US/kb/profile-manager-create-and-remove-firefox-profiles

Comment 2

[Loic]

Nevermind, I'm able to reproduce it.

Indeed, Print Setup displays margin sizes in mm instead of in when the paper size is Legal US e.g. (I tested on Win 7 with MS XPS Document Writer).

Regression range:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=955993
ba4a2a6ed9c85b9fc6c3a4c0fe582b178f&tochange=fb4737474a9c50b44e0ec38e43d42ed8a119
b7ee

Comment 3

[Lou Petro]

I note that the title of this Bug report now refers to US Legal Size paper. The problem I've encountered is with my printer set to US Letter Size paper. Not sure if this is significant but thought I should mention it. Oh, and I did try a new profile and had the same problem.

Comment 4

[Ritu Kothari (:ritu)]

Given that this is not a new regression in 47, we will have to live with it for another ~3 weeks. Hopefully we address this in 48 release.

Comment 5

[Bob Owen (:bobowen)]

I know what I need to do to sort this out, I'll get to it soon.

Comment 6

[Liz Henry (:lizzard)]

Regression from 46, tracking.

Comment 7

[Bob Owen (:bobowen)]

Attachment: Bug 1280159: Let paper type drive page size units on Windows.

Also, changed the other paper size conversion code to be similar, as I think it is easier to follow.

Review commit: https://reviewboard.mozilla.org/r/61518/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/61518/

Comment 8

[Jim Mathies [:jimm]]

Comment on attachment 8766708 [details]
Bug 1280159: Let paper type drive page size units on Windows.

https://reviewboard.mozilla.org/r/61518/#review58440

lgtm

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a4999a2b771d
Let paper type drive page size units on Windows. r=jimm

Comment 10

[Gregory Szorc [:gps]]

https://hg.mozilla.org/mozilla-central/rev/a4999a2b771d

Comment 11

[Gerry Chang [:gchang]]

Hi Bob, do you want to uplift this for 48/49 if this patch is not too risky.

Comment 12

[Bob Owen (:bobowen)]

(In reply to Gerry Chang [:gchang] from comment #11)
> Hi Bob, do you want to uplift this for 48/49 if this patch is not too risky.

Yes, I'll just wait for it to hit Nightly and give it another test, then I'll request uplift, thanks.

Comment 13

[Bob Owen (:bobowen)]

Comment on attachment 8766708 [details]
Bug 1280159: Let paper type drive page size units on Windows.

Approval Request Comment
[Feature/regressing bug #]:
Bug 1238964 changed to just use millimetres on Windows. (Firefox 46)

[User impact if declined]:
Margin sizes unit will continue not to relate to the paper type.

[Describe test coverage new/current, TreeHerder]:
Manual testing, it would be good to get QA verification in Beta.

[Risks and why]:
Low(ish) - fairly simple change that maps paper type to size units and uses that for calculations.

[String/UUID change made/needed]:
None

Comment 14

[Gerry Chang [:gchang]]

Comment on attachment 8766708 [details]
Bug 1280159: Let paper type drive page size units on Windows.

This patch fixes a regression. Take it in 48 beta 6 and aurora.

Comment 15

[Gerry Chang [:gchang]]

This needs QE to test in beta.

Comment 16

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/65cb053a14bc

Comment 17

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-beta/rev/f574e40a75b5

Comment 18

[Iulia Cristescu, QA [:JuliaC] (away, please needinfo? [e-mail])]

According to Comment 13 (and also based on my investigations), this bug is only affecting Windows platform.
I managed to reproduce this issue on 50.0a1 (2016-06-14).
Also, I investigated it on 
- Windows 10 x64
- Windows 7 x64

The bug is verified fixed on 
- latest Nightly 50.0a1 (2016-07-10)
- latest Aurora 49.0a2 (2016-07-11)
- 48.0b6 build 1 (20160706215822)

I will set the flags accordingly.