New issue
Advanced search Search tips

Issue 644001 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 734085
Owner: ----
Closed: Aug 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Hidpi ratio gets applied a second time to embedded extension options' size

Reported by tobias...@gmail.com, Sep 4 2016

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/53.0.2785.89 Safari/537.36

Steps to reproduce the problem:
1. Have a hidpi ratio set in Windows.
2. Install the attached extension.
3. Open the settings for the extension. The extension has its settings specified as an embedded options page, meaning they are embedded in the extensions manager.
4. Open the developer tools for the embedded options page, by right-clicking on it and choosing "Inspect element". Select the <html> element to look up its size.

What is the expected behavior?
As for the embedded options page's body, a width of 1000px (CSS pixels) is specified, it also should have a width of roughly that size.

What went wrong?
The embedded options gets shown with width of roughly (1000 * hidpi ratio) CSS pixels, i.e. (1000 * (hidpi ratio)^2) device pixels.

For example, in the attached screenshot, the hidpi ratio set is 2.5, so the embedded options get displayed with a width of roughly 2500 CSS pixels, instead of roughly 1000 device pixels.

Somewhere, it seems, the hidpi ratio gets applied a second time to the size of the embedded options.

WebStore page: 

Did this work before? Yes I believe, although I didn't test the attached extension specifically there, in Chrome 52

Chrome version: 53.0.2785.89  Channel: stable
OS Version: 10.0
Flash Version:
 
Screenshot embedded option.png
182 KB View Download
embedded-options-testcase extension.zip
821 bytes Download
"roughly 1000 device pixels" should mean "roughly 1000 CSS pixels".

Comment 2 by bsep@chromium.org, Aug 25 2017

Mergedinto: 734085
Status: Duplicate (was: Unconfirmed)

Sign in to add a comment