Closed
Bug 1438328
Opened 6 years ago
Closed 6 years ago
Port |Bug 1438270 - Remove nsIDOMDocumentXBL| to C-C
Categories
(Thunderbird :: General, enhancement)
Thunderbird
General
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 60.0
People
(Reporter: jorgk-bmo, Assigned: jorgk-bmo)
References
Details
Attachments
(2 files)
|
3.75 KB,
patch
|
aceman
:
review+
|
Details | Diff | Splinter Review |
|
5.36 KB,
patch
|
aceman
:
review+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #1438270 +++ https://searchfox.org/comm-central/search?q=loadBindingDocument&case=true&path=mail%2F And https://searchfox.org/comm-central/search?q=loadBindingDocument&case=true&path=suite in SeaMonkey. These calls need to be removed. Also see bug 1438270 comment #7. Richard, Aceman, you know anything about this?
Flags: needinfo?(richard.marti)
Flags: needinfo?(acelists)
|
||
Comment 1•6 years ago
|
||
> These calls need to be removed. To be clear, they do not need to be removed just to get bug 1438270 landed. They continue to work from JS. But I'd like to remove them (as a separate fix from bug 1438270), so would like to understand whether they're really needed in comm-central code.
|
Assignee | |
Comment 2•6 years ago
|
||
Attachment #8951067 -
Flags: review?(acelists)
|
||
Comment 3•6 years ago
|
||
I know nothing about this. And it seems you found the needed solution.
Flags: needinfo?(richard.marti)
|
|
Assignee | |
Comment 4•6 years ago
|
||
|
|
Assignee | |
Comment 5•6 years ago
|
||
I need to remove dom_xbl.xpt from the package manifests now, or else we're busted. As per comment #1, no need to land the other patch just yet.
Keywords: leave-open
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/9979f414b1af Port bug 1438270 to TB/IB/SM: remove dom_xbl.xpt from package manifests. rs=bustage-fix
Comment on attachment 8951067 [details] [diff] [review] 1438328-remove-loadBindingDocument.patch Review of attachment 8951067 [details] [diff] [review]: ----------------------------------------------------------------- OK, let's try this. I haven't seen any visible problems while loading messages and also all tests pass: https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=c0185d8a574c55e144fc3cb43d91fcaa985df743
Attachment #8951067 -
Flags: review?(acelists) → review+
Attachment #8951189 -
Flags: review+
|
|
Assignee | |
Updated•6 years ago
|
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/8f403ab50774 Port bug 1438270 to C-C: Remove use of nsIDOMDocumentXBL.loadBindingDocument(). r=aceman
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•6 years ago
|
Target Milestone: --- → Thunderbird 60.0
You need to log in
before you can comment on or make changes to this bug.
Description
•