779335 - Text zoom level on the opener page affects the width of a window.open window

Status: NEW

(Core :: DOM: Core & HTML, defect, P5)

Description

[John Morrison [:jrgm]]

This came up in https://github.com/mozilla/browserid/issues/2008#issuecomment-7412230 but seems like a generic issue.

Steps to reproduce:

1. Go to https://bugzilla.mozilla.org/

2. Do View->Zoom->Zoom-out once

3. Click the sign-in button from browserid

Expected: the dialog box opens at the requested width parameter of 700

Actual: the dialog box opens with width 627 (which approximately the same ratio to 700 as the text zoom factor on the opener page bugzilla.mozilla.org).

Tested with 17.0a1 (2012-07-24) on OSX 10.7

Comment 1

[John Morrison [:jrgm]]

Note: when the dialog opens at the reduced width it has the layout shown in the top image in https://github.com/mozilla/browserid/issues/2008#issuecomment-7122021 which is layout intended for mobile devices. When it opens in the requested width of 700 it has the layout shown in the second image.

Comment 2

[Alice0775 White]

Regression window:
Good:
http://hg.mozilla.org/mozilla-central/rev/77070aa1f917
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b6pre) Gecko/20100909 Firefox/4.0b6pre ID:20100909221909
Bad;
http://hg.mozilla.org/mozilla-central/rev/008a0b8578db
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b6pre) Gecko/20100909 Firefox/4.0b6pre ID:20100909231905
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=77070aa1f917&tochange=008a0b8578db

Triggered by: bug 594140

Comment 3

[Boris Zbarsky [:bzbarsky]]

Yes, this behavior is the behavior bug 594140 was supposed to implement.  The width/height passed to window.open are treated as being in the opener's CSS pixels....

Comment 4

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Maybe we should propagate the zoom of the original document to the opened window?

Comment 5

[Boris Zbarsky [:bzbarsky]]

Another option is to only do the zoom thing for same-origin opens (as based on the url passed to window.open).

Comment 6

[John Morrison [:jrgm]]

Thanks Boris, Roc, for taking the time to have a look.

Comment 7

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.