Closed
Bug 1496353
Opened 6 years ago
Closed 6 years ago
Port bug 1461798 - tooltip binding removed - TEST-UNEXPECTED-FAIL | [snip]mozmill\folder-display\test-tooltip-multimessage.js | test-tooltip-multimessage.js::test_tooltips
Categories
(Thunderbird :: General, defect)
Thunderbird
General
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 65.0
People
(Reporter: jorgk-bmo, Assigned: mkmelin)
Details
(Whiteboard: [Thunderbird-disabled-test])
Attachments
(2 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
jorgk-bmo
:
review+
|
Details | Diff | Splinter Review |
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.
Flags: needinfo?(mkmelin+mozilla)
Flags: needinfo?(arshdkhn1)
Reporter | ||
Updated•6 years ago
|
Keywords: leave-open
Reporter | ||
Comment 1•6 years ago
|
||
Assignee: nobody → jorgk
Reporter | ||
Updated•6 years ago
|
Whiteboard: [Thunderbird-disabled-test]
Reporter | ||
Updated•6 years ago
|
Assignee: jorgk → nobody
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/e08d005eaad4
temporarily disable MozMill test test-tooltip-multimessage.js. rs=bustage-fix DONTBUILD
Comment 3•6 years ago
|
||
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..
Flags: needinfo?(arshdkhn1)
Reporter | ||
Updated•6 years ago
|
Flags: needinfo?(acelists)
Assignee | ||
Comment 4•6 years ago
|
||
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)
Comment 5•6 years ago
|
||
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.
Flags: needinfo?(emilio)
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → mkmelin+mozilla
Assignee | ||
Comment 6•6 years ago
|
||
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/.
Attachment #9014329 -
Attachment is obsolete: true
Attachment #9022558 -
Flags: review?(jorgk)
Assignee | ||
Updated•6 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•6 years ago
|
Keywords: leave-open
Assignee | ||
Updated•6 years ago
|
Flags: needinfo?(acelists)
Reporter | ||
Comment 7•6 years ago
|
||
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+
Reporter | ||
Updated•6 years ago
|
Attachment #9014329 -
Attachment is obsolete: false
Assignee | ||
Updated•6 years ago
|
Keywords: checkin-needed
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/25adff603af0
remove multimessage tooltip test. r=jorgk
Reporter | ||
Updated•6 years ago
|
Target Milestone: --- → Thunderbird 65.0
You need to log in
before you can comment on or make changes to this bug.
Description
•