1692021 - Update notification bar stacking

Status: VERIFIED FIXED

(Firefox :: Messaging System, enhancement, P2)

Description

[Mark Striemer [:mstriemer]]

With the proton changes, notification bars will now always be shown, rather than just showing the highest-priority notification in each of the 3 notification boxes.

All browser-specific notifications will go in one notificationbox (rather than in gHighPriorityNotificationBox and gNotificationBox). This notificationbox will be above the tab-specific notificationbox, and new notifications are added at the top of the notificationbox.

All tab-specific notifications will be shown, with the newest notification at the bottom of the notificationbox.

So with 4 notifications shown (this seems incredibly unlikely but preparing for the worst) with the number being order of appearance:

4) Places locked
1) Set as default
------------------
2) Pop-up blocked
3) Part of page crashed

Comment 1

[Tyler Duzan]

Has this been completed? It appears that not everything is 100%, but that stacking functions from my PM review.

Comment 2

[Mark Striemer [:mstriemer]]

Attachment: Bug 1692021 - Order infobars by appearance rather than priority r?gijs

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b80da7156f20
Order infobars by appearnce rather than priority r=Gijs

Comment 4

[Natalia Csoregi [:nataliaCs]]

Backed out for failures on test_notificationbox.xhtml

backout: https://hg.mozilla.org/integration/autoland/rev/e8ccb5d14184d1088712de5ce8037148649b8fdc

push: https://treeherder.mozilla.org/jobs?repo=autoland&revision=b80da7156f2018ea3c9bc5fb00c847a6e1a0f38f&searchStr=mochitest-chrome&group_state=expanded&selectedTaskRun=Qtog0aLGRq-NVQp-QqOZhg.0

failure log: https://treeherder.mozilla.org/logviewer?job_id=336702332&repo=autoland&lineNumber=2704

[task 2021-04-15T20:29:36.482Z] 20:29:36 INFO - TEST-PASS | toolkit/content/tests/chrome/test_notificationbox.xhtml | removeAllNotifications
[task 2021-04-15T20:29:36.483Z] 20:29:36 INFO - Buffered messages finished
[task 2021-04-15T20:29:36.483Z] 20:29:36 INFO - TEST-UNEXPECTED-FAIL | toolkit/content/tests/chrome/test_notificationbox.xhtml | appendNotification last notification - got [object HTMLElement], expected [object HTMLElement]
[task 2021-04-15T20:29:36.483Z] 20:29:36 INFO - SimpleTest.is@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:500:14
[task 2021-04-15T20:29:36.483Z] 20:29:36 INFO - result@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_notificationbox.xhtml:419:9
[task 2021-04-15T20:29:36.483Z] 20:29:36 INFO - runTimedTests@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_notificationbox.xhtml:605:22
[task 2021-04-15T20:29:36.483Z] 20:29:36 INFO - runTimedTestsWait@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_notificationbox.xhtml:624:18
[task 2021-04-15T20:29:36.483Z] 20:29:36 INFO - Not taking screenshot here: see the one that was previously logged

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7ad69409f8f4
Order infobars by appearance rather than priority r=Gijs

Comment 6

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/7ad69409f8f4

Comment 7

[Peter Magyari (Desktop QA)]

Verified fixed on Windows 10, MacOS 11 and Ubuntu 20.04 using Nightly 89.0a1 (20210418213620)