1304648 - With multiple monitors, dppx resolution of a new window pop-up is wrong

Status: RESOLVED WORKSFORME

(Core :: Layout, defect)

Description

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Attachment: dppx-popup.png

STR:

1. An MBPR, with an external monitor and extended desktop, have Firefox window opened on the external monitor.
2. Open some website require Google log-in or OAuth window etc.
3. Observe the window that got opened on the MBPR screen.

Expected:

1. Content is scaled correctly

Actual:

1. Content is scaled incorrectly at 1x on a dppx != 1x screen.

Note:

My MPBR is set to scaled at 1680x1050 ("More Space" in the Displays preferences) and the external monitor is on 1920x1080.

Comment 1

[Jonathan Kew [:jfkthame]]

Could you give a specific URL where you're seeing this issue? I just tried triggering a Google login window on such a site, but didn't run into this problem.

Comment 2

[Tim Guan-tin Chien [:timdream] (please needinfo)]

This specific dialog pops up when do the following:

1. Open a Google Spreadsheet doc
2. Tools -> Script editor...
3. Write some Google App Script JS that would need a permission, like

function foo() {
  SpreadsheetApp.getActiveSpreadsheet().getSheets()[0]..getRange(1, 1).setValue("Hello World");
}

4. Save the script by click the save icon, or,
5. Run the script by select the function "foo" on the toolbar and click the run icon.

(I can't remember which step would trigger the permission popup, but I am sure one of these would if you run the script for the first time)

These are very long steps and it's entirely possible they somehow changed the script to make the problem go away. If you couldn't reproduce this issue with these steps, maybe we should just try to write up a test case ....

Comment 3

[Jonathan Kew [:jfkthame]]

I was able to sort-of-partly-reproduce this with the steps above using FF49 -- the permission popup didn't have mis-scaled content, like in your screenshot, but the window was much bigger than I'd expect; looks like it was 2x-sized, with its content displayed OK, just centered in the big window.

With current Nightly, however, the sizing looks better (and the content is still fine). So I think this may have been fixed by one of the dpi-relateed patches we've landed in the past month or two. Could you check whether you can still reproduce with a newer version?

Comment 4

[Tim Guan-tin Chien [:timdream] (please needinfo)]

I cannot reproduce this bug with latest Nightly. WFM, would need to identify the patch if we want to uplift things.