Intermittent comm/mail/test/browser/message-reader/browser_imageOverflow.js | The message display needs to be less than 400px wide - 832 < 400 -
Categories
(Thunderbird :: Message Reader UI, defect, P5)
Tracking
(thunderbird_esr128 wontfix, thunderbird133 affected)
People
(Reporter: intermittent-bug-filer, Assigned: mkmelin)
Details
(Keywords: intermittent-failure, intermittent-testcase)
Attachments
(1 file)
Filed by: mkmelin [at] iki.fi
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=469935630&repo=comm-central
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/Yq6Tfb0PRo-WndBw0V5p4Q/runs/0/artifacts/public/logs/live_backing.log
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
Comment 6•1 year ago
|
||
Looking at the logs I think all that is probably needed to fix this is to wait slightly longer after the resize event. Maybe an animation frame or similar? Because the second assertion (L#182 "No scrollbars") already has the expected width (and fails because we keep the width we read before the first one).
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
Comment 13•1 year ago
|
||
I added some extra waiting in Bug 1917278 but it seems that wasn't quite enough yet.
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Assignee | ||
Comment 18•1 year ago
|
||
There were a bunch of issues with this test.
Simplified the real code to make it more understandable (and using boolean attributes).
Updated•1 year ago
|
Updated•1 year ago
|
| Assignee | ||
Updated•1 year ago
|
Comment 19•1 year ago
|
||
Pushed by toby@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/fe4882174805
Fix intermittent comm/mail/test/browser/message-reader/browser_imageOverflow.js | The message display needs to be less than 400px wide - 832 < 400. r=tobyp
| Comment hidden (Intermittent Failures Robot) |
| Assignee | ||
Comment 21•1 year ago
|
||
Comment on attachment 9436741 [details]
Bug 1912579 - Fix intermittent comm/mail/test/browser/message-reader/browser_imageOverflow.js | The message display needs to be less than 400px wide - 832 < 400. r=freaktechnik
[Approval Request Comment]
User impact if declined: image overflow handling may not always work correctly
Testing completed (on c-c, etc.): c-c, beta
Risk to taking this patch (and alternatives if risky): fairly safe but not trivial (though issues would be contained to the feature in question)
Comment 22•1 year ago
|
||
Comment on attachment 9436741 [details]
Bug 1912579 - Fix intermittent comm/mail/test/browser/message-reader/browser_imageOverflow.js | The message display needs to be less than 400px wide - 832 < 400. r=freaktechnik
[Triage Comment]
Approved for esr128
Comment 23•1 year ago
|
||
Does this bug need bug 1875325 landed on ESR first? It seems to fix an intermittent issue for a test that doesn't exist on ESR.
| Assignee | ||
Comment 24•1 year ago
|
||
Apparently yes. I guess it might be best not to uplift this as there are dependencies.
| Assignee | ||
Updated•1 year ago
|
Description
•