Status

()

enhancement
RESOLVED FIXED
Last year
Last year

People

(Reporter: adrian17, Assigned: adrian17)

Tracking

(Blocks 1 bug)

unspecified
mozilla60
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox60 fixed)

Details

Attachments

(1 attachment)

No description provided.
Assignee

Updated

Last year
Assignee: nobody → adrian.wielgosik
Blocks: 1387169
Status: NEW → ASSIGNED
Comment hidden (mozreview-request)

Comment 2

Last year
mozreview-review
Comment on attachment 8951020 [details]
Bug 1438270 - Remove nsIDOMDocumentXBL.

https://reviewboard.mozilla.org/r/220274/#review226242

r=me.

::: dom/base/nsDocument.cpp:6216
(Diff revision 1)
> -                                   rv);
> -  return rv.StealNSResult();
> -}
> -
>  void
>  nsIDocument::LoadBindingDocument(const nsAString& aURI,

This is unused in mozilla-central.  Do you mind filing a Thunderbird bug about checking whether their uses are really needed?
Attachment #8951020 - Flags: review?(bzbarsky) → review+
Jorg, you're going to need comm-central manifest changes here.  Also, see comment 2?
Flags: needinfo?(jorgk)

Comment 4

Last year
I watch changes to package manifests by default ;-) Which Thunderbird uses do you see?
Flags: needinfo?(jorgk) → needinfo?(bzbarsky)

Comment 6

Last year
I know nothing about it. What does it do? Is there a replacement? Zero documentation in nsIDOMDocumentXBL.idl.
It preloads the XBL binding.  But these are chrome:// bindings, so they get sync-loaded anyway when needed.  Hence there doesn't seem to be much need to preload them.

I wasn't expecting you to deal with those loadBindings calls personally, just to pass on to whoever might know about that end of the woods, assuming there is anyone.  And ideally in a separate bug, not here.

Updated

Last year
Depends on: 1438328

Comment 8

Last year
(In reply to Boris Zbarsky [:bz] (no decent commit message means r-) from comment #7)
> just to pass on to whoever might know about that end of the woods, assuming
> there is anyone.
:-(

> And ideally in a separate bug, not here.
Done.

Comment 10

Last year
bugherder
https://hg.mozilla.org/mozilla-central/rev/84611d1652f2
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in before you can comment on or make changes to this bug.