Make innerWidth/innerHeight not round on Nightly.
Categories
(Core :: DOM: CSS Object Model, defect, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox84 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
(Blocks 1 open bug)
Details
(Keywords: perf-alert)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
See the discussion in https://github.com/w3c/csswg-drafts/issues/5260.
Assignee | ||
Comment 1•4 years ago
|
||
On Nightly, for now. This allows websites to get the real viewport size.
The rounded size has caused problems in the past (see bug 1648298 / bug
1648265), and changing it would be ideal.
I told the CSSWG that we can experiment with this on Nightly to
alleviate (or prove) compat concerns, in which case we'd need to use a
different solution, see https://github.com/w3c/csswg-drafts/issues/5260.
Updated•4 years ago
|
Comment 9•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ed3ef7c26759
https://hg.mozilla.org/mozilla-central/rev/b78f57f4c43c
https://hg.mozilla.org/mozilla-central/rev/4745a76dd993
https://hg.mozilla.org/mozilla-central/rev/25a59a08c4d9
https://hg.mozilla.org/mozilla-central/rev/1a9a2a27f508
https://hg.mozilla.org/mozilla-central/rev/5a71034a6449
https://hg.mozilla.org/mozilla-central/rev/7a8e1e203b11
Updated•4 years ago
|
Comment 10•4 years ago
|
||
I would suggest we back this out for breaking youtube.
Comment 11•4 years ago
|
||
(In reply to Tom Schuster [:evilpie] from comment #10)
I would suggest we back this out for breaking youtube.
As per the discussion in #developers, a fix has been landed here https://hg.mozilla.org/integration/autoland/rev/b9dd8e54c8ae449638635758a6571285655a4d0c
Comment 12•4 years ago
|
||
== Change summary for alert #27811 (as of Mon, 23 Nov 2020 06:50:03 GMT) ==
Improvements:
Ratio | Suite | Test | Platform | Options | Absolute values (old vs new) |
---|---|---|---|---|---|
5% | bing | SpeedIndex | android-hw-p2-8-0-android-aarch64-shippable | warm webrender | 791.62 -> 750.67 |
3% | bing | LastVisualChange | android-hw-p2-8-0-android-aarch64-shippable | cold webrender | 5,193.75 -> 5,020.25 |
For up to date results, see: https://treeherder.mozilla.org/perfherder/alerts?id=27811
Assignee | ||
Comment 13•4 years ago
|
||
(In reply to Florin Strugariu [:Bebe] (needinfo me) from comment #12)
Do you know if this regressed again after bug 1677390?
Comment 14•4 years ago
|
||
now that you ask about it it looks like it has regressed again:
Assignee | ||
Comment 15•4 years ago
|
||
Yeah, oh well... That's expected given bug 1677390, thanks for checking!
Description
•