Fix a couple issues with anchored popups uncovered by bug 1665476.
Categories
(Thunderbird :: General, defect)
Tracking
(thunderbird_esr78 wontfix, thunderbird82 wontfix)
People
(Reporter: emilio, Assigned: emilio)
References
Details
Attachments
(2 files)
Assignee | ||
Comment 1•4 years ago
|
||
The patch in bug 1666497 (which makes regular <window>-based XUL
documents with other the XHTML-based ones) uncovers an issue in the
anchored popup tests, which starts timing out.
The underlying issue is that it changes the reflow order from popups in
XUL-only documents with respect to their anchors. Note that Firefox
already uses the "broken" code, because we moved to xhtml and
browser.xhtml doesn't use a XUL <window> anymore.
Since we reflow now the popup after the anchor rather than before, we
don't need to rely on CheckForAnchorChange() to deal with changes. This
means that we need to notify on the positioning changes that happen
during reflow though.
We should be able to simplify a bit the CheckForAnchorChange() stuff
now, but it also takes care of hiding popups and such so I don't plan to
do it on this bug to minimize risk.
Assignee | ||
Comment 2•4 years ago
|
||
Updated•4 years ago
|
Comment 4•4 years ago
|
||
Backed out changeset 1b76f477b19c (Bug 1666497) for unexpected reflow failures. CLOSED TREE
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=317426479&repo=autoland&lineNumber=2335
Backout: https://hg.mozilla.org/integration/autoland/rev/ec8f5ee6f607fd5e0c76bf3bd76dc0ca00a64328
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 6•4 years ago
|
||
Before my patch we were skipping some popuppositioned events, so it's
not unexpected that this count can go up as a result.
Updated•4 years ago
|
Comment 7•4 years ago
|
||
Backed out for bc failure on browser_appmenu.js
Backout link: https://hg.mozilla.org/integration/autoland/rev/da52c9e0a94a5726d383f1f486a9ce1eba6199e9
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=317653140&repo=autoland&lineNumber=1529
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Comment 10•4 years ago
|
||
Comment 11•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/741543bb9d41
https://hg.mozilla.org/mozilla-central/rev/cdec58fb4132
Updated•4 years ago
|
Updated•4 years ago
|
Description
•