When run locally I get: SUMMARY-UNEXPECTED-FAIL | c:\mozilla-source\comm-central\comm\mail\test\mozmill\folder-display\test-tooltip-multimessage.js | test-tooltip-multimessage.js::test_tooltips EXCEPTION: tooltip.fillInPageTooltip is not a function at: test-tooltip-multimessage.js line 44 test_tooltips test-tooltip-multimessage.js:44 3 M-C last good: 07aa30516b54bf2644120c51a73cac2f9c M-C first bad: 8b1f1ebed0f0d6c8abc7e201d70d999f92 https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=07aa30516b54bf2644120c51a73cac2f9c&tochange=8b1f1ebed0f0d6c8abc7e201d70d999f92 This comes from Bug 1461798 - Migrate <tooltip> to a C++ implementation. r=smaug which removed that binding :-( https://hg.mozilla.org/mozilla-central/rev/2d5a7c965683#l18.48 Looks like a case for Arshad and Magnus.
Assignee: nobody → jorgk
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/comm-central/rev/e08d005eaad4 temporarily disable MozMill test test-tooltip-multimessage.js. rs=bustage-fix DONTBUILD
Hey, I am not sure what to do. TB has its tooltip which is extending FX tooltip and folderSummary-popup is there which extends TB's tooltip..
It's a bit hard to say, but perhaps you can migrate folderSummary-popup to a CE, then use the bits you need from https://phabricator.services.mozilla.com/D3056 (CE version of <tooltip> from bug 1461798). Emilio, do you see any problems with this, off hand? I don't understand the issues discussed in bug 1461798. We're trying to figure out how to deal with the folderSummary-popup binding https://searchfox.org/comm-central/source/mail/base/content/mailWidgets.xml#2303 It's only used once https://searchfox.org/comm-central/source/mail/base/content/messenger.xul#303 so we don't need cater for more use cases than that.
Flags: needinfo?(mkmelin+mozilla) → needinfo?(emilio)
TBH I'm not that familiar with what ended up happening in that bug, but the issues with using a custom-element were that there are anonymous <tooltip> elements created with C++, so those issues should not apply to this binding.
I don't think this test is useful to keep around. We're testing toolkit functionality, and they already test it quite thoroughly in mozilla/toolkit/components/tooltiptext/tests/.
Comment on attachment 9022558 [details] [diff] [review] bug1496353_tooltip_test.patch OK. I prefer to keep landed patches as attachments, so you can see what's gone on.
Attachment #9022558 - Flags: review?(jorgk) → review+
Pushed by email@example.com: https://hg.mozilla.org/comm-central/rev/25adff603af0 remove multimessage tooltip test. r=jorgk
Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.