Closed Bug 660739 Opened 10 years ago Closed 10 years ago
ERROR TEST-UNEXPECTED-FAIL test
_idcheck .xul | check id: messengercompose .xul#content Area Context Set
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1306807420.1306810850.22456.gz 10451 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/suite/common/tests/chrome/test_idcheck.xul | check id: messengercompose.xul#contentAreaContextSet Ian, you forgot to fix the test ;-)
No. I didn't forget, the test is correct. I just have to wait for the patch to land from bug 657161, then I can remove the popupset line from sidebarOverlay.xul (which I did forget to include in that patch), so I'll put the patch on here.
Assignee: nobody → iann_bugzilla
Status: NEW → ASSIGNED
Once all windows that use the sidebar overlay have the contentAreaContextSet popupset the overlay won't need it - test will then pass again.
Correct patch this time
Comment on attachment 536690 [details] [diff] [review] Remove popupset [Checked in: Comment 6] This could do with a comment somewhere mentioning the dependency.
Attachment #536690 - Flags: review?(neil) → review+
Comment on attachment 536690 [details] [diff] [review] Remove popupset [Checked in: Comment 6] With additional comment: http://hg.mozilla.org/comm-central/rev/3435704c16d9
Attachment #536690 - Attachment description: Remove popupset → Remove popupset [Checked in: Comment 6]
You need to log in before you can comment on or make changes to this bug.