Resolution doesn't get applied after one touch pinch
Categories
(Core :: Panning and Zooming, defect, P3)
Tracking
()
People
(Reporter: kats, Assigned: kats)
References
Details
Attachments
(3 files)
Comment 1•6 years ago
|
||
Comment 2•6 years ago
|
||
Comment 3•6 years ago
|
||
Assignee | ||
Comment 4•6 years ago
|
||
Bug 1111333 - the original for reference
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Comment 8•6 years ago
|
||
The patch for this is simple enough. Time to write some tests! rolls up sleeves
Assignee | ||
Comment 9•6 years ago
|
||
Assignee | ||
Comment 10•6 years ago
|
||
Assignee | ||
Comment 11•6 years ago
|
||
Assignee | ||
Comment 12•6 years ago
|
||
Sometimes we can get empty transactions after a scrollframe is
layerized. In such cases the isLayerized check would incorrectly detect
the scrollframe as not being layerized because it would only look at the
data for the empty transaction.
Depends on D23494
Assignee | ||
Comment 13•6 years ago
|
||
The helper_basic_onetouchpinch.html is basically a copy of
helper_basic_zoom.html with a few things changed (most importantly, the
touch event sequence).
Depends on D23495
Comment 14•6 years ago
|
||
Comment 15•6 years ago
|
||
Backed out for memory leaks.
backout: https://hg.mozilla.org/integration/autoland/rev/8f2a6dbf221ca51fcb80f1a94eea41d377a0c91c
failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=234318839&repo=autoland&lineNumber=3087
Comment 16•6 years ago
|
||
Huh. That seems to be a legitimate bug in nsDOMWindowUtils::SetDisplayPortForElement
, introduced in this commit:
Comment 17•6 years ago
|
||
(In reply to Botond Ballo [:botond] from comment #16)
Huh. That seems to be a legitimate bug in
nsDOMWindowUtils::SetDisplayPortForElement
, introduced in [this
commit](https://hg.mozilla.org/mozilla-central/rev/
484e07a716f1aac33b4fd9ee246c7eab8b504523):
-> bug 1535862.
Assignee | ||
Comment 18•6 years ago
|
||
Thanks for the quick diagnosis and fix! I'll reland these patches on Monday.
Comment 19•6 years ago
|
||
Comment 20•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/aededeb1f164
https://hg.mozilla.org/mozilla-central/rev/729056f6dc85
https://hg.mozilla.org/mozilla-central/rev/cb58de9731cb
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Description
•