Closed Bug 1557979 Opened 1 year ago Closed 1 year ago

TEST-UNEXPECTED-FAIL | [snip]mozmill/content-tabs/test-content-tab.js and folder-display/test-right-click-middle-click-messages.js


(Thunderbird :: General, defect)

Not set


(Not tracked)

Thunderbird 69.0


(Reporter: jorgk-bmo, Assigned: jorgk-bmo)


(Whiteboard: [Thunderbird-testfailure: Z all][Thunderbird-disabled-test])


(2 files)

I see this in the error console:
JavaScript error: chrome://messenger/content/nsContextMenu.js, line 84: TypeError: PageMenuParent.buildAndAddToPopup is not a function
JavaScript error: chrome://messenger/content/mailContextMenus.js, line 55: TypeError: gContextMenu is undefined

This comes from bug 1505909 where buildAndAddToPopup was removed:

I'll disable the tests for now since I don't see this being fixed in a flash.

Flags: needinfo?(mkmelin+mozilla)
Flags: needinfo?(geoff)

Just mentioning:
mozmake SOLO_TEST=folder-display/test-right-click-middle-click-messages.js mozmill-one
mozmake SOLO_TEST=content-tabs/test-content-tab.js mozmill-one
show horribly long context menus now.

I've done some pattern matching and with this patch, all but one test pass again. I'll disable the failing one separately.

Assignee: nobody → jorgk
Flags: needinfo?(mkmelin+mozilla)
Attachment #9070810 - Flags: review?(geoff)
Whiteboard: [Thunderbird-testfailure: Z all][Thunderbird-disabled-test]
Pushed by
Port bug 1505909: Replace use of buildAndAddToPopup(). rs=bustage-fix
Disable failing check in test-content-tab.js::test_content_tab_context_menu. rs=bustage-fix

Looking at the test failure:

Somehow that was checking if some "page menu" got added. Looks like it's driven by .hasPageMenu.

I'd like to know what a page menu is, but clearly M-C changed code around this feature here:

Anyway, I have a fix, attaching patch now.

Flags: needinfo?(geoff)
Keywords: leave-open

This does the trick.

Attachment #9070857 - Flags: review?(geoff)

Pushed by
Follow-up: Fix 'page menu' and re-enable check. rs=bustage-fix

Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 69.0
Attachment #9070810 - Flags: review?(geoff) → review+
Comment on attachment 9070857 [details] [diff] [review]

This should be:
this.hasPageMenu = PageMenuParent.addToPopup(menuObject, null, aPopup);

I'll fix it myself if I do a push today.

It seems a page menu is what you get when an HTML page uses the (now deprecated, I believe) <menu> element to add stuff to the context menu.
Attachment #9070857 - Flags: review?(geoff) → review-
Pushed by
Follow-up, take 2: Fix assignment of 'hasPageMenu'. r=darktrojan DONTBUILD
You need to log in before you can comment on or make changes to this bug.