MozAfterPaint might fire too early on Mac, meaning browser-delayed-startup-finished fires too early on Mac, meaning I can't show dialog
Categories
(Core :: Widget: Cocoa, defect, P3)
Tracking
()
People
(Reporter: mkaply, Assigned: mkaply)
References
Details
(Whiteboard: tpi:+)
Attachments
(4 files, 1 obsolete file)
Assignee | ||
Comment 1•11 years ago
|
||
Assignee | ||
Comment 2•11 years ago
|
||
Comment 3•11 years ago
|
||
Updated•11 years ago
|
Assignee | ||
Comment 4•11 years ago
|
||
Assignee | ||
Comment 5•11 years ago
|
||
Assignee | ||
Comment 6•11 years ago
|
||
Comment 7•11 years ago
|
||
Comment 8•11 years ago
|
||
Updated•11 years ago
|
Assignee | ||
Comment 9•11 years ago
|
||
Comment 10•11 years ago
|
||
Assignee | ||
Comment 11•11 years ago
|
||
Comment 12•11 years ago
|
||
Comment 13•11 years ago
|
||
Assignee | ||
Comment 14•11 years ago
|
||
Comment 15•11 years ago
|
||
Updated•11 years ago
|
Comment 17•11 years ago
|
||
Assignee | ||
Comment 18•11 years ago
|
||
Comment 19•11 years ago
|
||
Comment 21•11 years ago
|
||
Comment 22•11 years ago
|
||
Comment 23•11 years ago
|
||
Comment 24•11 years ago
|
||
Comment 25•11 years ago
|
||
Assignee | ||
Comment 26•11 years ago
|
||
Comment 27•11 years ago
|
||
Comment 28•11 years ago
|
||
Comment 29•11 years ago
|
||
Comment 30•11 years ago
|
||
Comment 31•11 years ago
|
||
Comment 32•11 years ago
|
||
Comment 33•11 years ago
|
||
Comment 34•11 years ago
|
||
Comment 35•11 years ago
|
||
Comment 36•11 years ago
|
||
Updated•11 years ago
|
Updated•11 years ago
|
Comment 37•11 years ago
|
||
Updated•11 years ago
|
Comment 38•11 years ago
|
||
Comment 39•11 years ago
|
||
Comment 40•10 years ago
|
||
Updated•10 years ago
|
Comment 41•9 years ago
|
||
Updated•9 years ago
|
Updated•8 years ago
|
Comment 42•8 years ago
|
||
Comment 43•8 years ago
|
||
Updated•8 years ago
|
Comment 44•8 years ago
|
||
Updated•7 years ago
|
Updated•7 years ago
|
Comment 45•6 years ago
|
||
Assignee | ||
Comment 47•6 years ago
|
||
Assignee | ||
Comment 48•5 years ago
|
||
Here's the talos runs:
I think this shows it had no effect.
Comment 49•5 years ago
|
||
(In reply to Mike Kaply [:mkaply] from comment #48)
I think this shows it had no effect.
Hi! I'm not sure what information is being asked for from me. Does the patch work, and you're wondering whether or not its acceptable to land given the Talos numbers, or does the patch not work and you're wondering why, or something else entirely?
Assignee | ||
Comment 50•5 years ago
|
||
Sorry, forgot context.
Many ages ago in the patch:
Thanks, mkaply. Seems okay, but I'm wary of Talos regression sensitivity here. Can you do a try run and show a comparison against central?
and I failed to followthrough.
So this is me posting Talos and based on my limited understanding, I think it has no effect?
Comment 51•5 years ago
|
||
Looks to me like we're in the noise. No strong signals, so let's go for it.
Comment 52•5 years ago
|
||
Comment 53•5 years ago
|
||
Backed out for assertion failures on nsContentUtils::IsSafeToRunScript()
Backout link: https://hg.mozilla.org/integration/autoland/rev/e3c371dd1b931efa8dcaf90c7ba30bedf028654b
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=299073689&repo=autoland&lineNumber=121766
There were also :
Crash test : https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=299072443&repo=autoland&lineNumber=110039
and
Mochitest crash a11y: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=299076279&repo=autoland&lineNumber=3109
Assignee | ||
Comment 54•5 years ago
|
||
I think I'm going to just end this bug. It really only applied to legacy extensions, and I don't know what behavior would be affected now.
Updated•5 years ago
|
Description
•