1117224 - Create group message manager to only load content.js framescript for tab <browser> elements

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[:Margaret Leibovic]

From bug 1111142 comment 7:

>+    window.messageManager.loadFrameScript("chrome://browser/content/content.js", true);

You originally tried using window.getGroupMessageManager("browsers").loadFrameScript(...), but that was failing. I think that's because we weren't adding message="true" and groupmessagemanager="browsers" to the <browser>s we create in Tab.createBrowser

See:
http://mxr.mozilla.org/mozilla-central/source/browser/base/content/tabbrowser.xml#1582

Might be good to try that again (in a followup or here) to keep the messages from polluting other <browser> objects we might create.

Comment 1

[:Margaret Leibovic]

Attachment: MozReview Request: bz://1117224/margaret

Comment 2

[:Margaret Leibovic]

/r/1979 - Bug 1117224 - Create group message manager to only load content.js framescript for tab <browser> elements. r=mfinkle

Pull down this commit:

hg pull review -r 4d41808464f86d6fe2762f875e3dcf886a02bd99

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Any idea what "message=true" does?

Comment 4

[Mark Finkle (:mfinkle) (use needinfo?)]

https://reviewboard.mozilla.org/r/1977/#review1255

I know you tested to make sure this is all we need :)

Comment 5

[:Margaret Leibovic]

(In reply to Mark Finkle (:mfinkle) from comment #3)
> Any idea what "message=true" does?

I looked at hg blame, and saw this was added in an earlier changeset, so I don't think it's related. Digging back, it landed in 2007, before e10s:
http://hg.mozilla.org/mozilla-central/rev/973de3d4e4f6

It looks like it's used (or was used) for showing messages in the browser.

And yes, I did test :)

Comment 6

[:Margaret Leibovic]

https://hg.mozilla.org/integration/fx-team/rev/0e6ad992ae07

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/0e6ad992ae07

Comment 9

[:Margaret Leibovic]

Attachment: MozReview Request: Bug 1117224 - Create group message manager to only load content.js framescript for tab <browser> elements. r=mfinkle

Comment 10

[Ian Nartowicz]

Any chance that the "browsers" message manager could be exposed as a window property, since it would seem to be preferable in nearly every case to use this rather than window.messageManager?  Or at least document it?  Here?:
https://developer.mozilla.org/en-US/Firefox/Multiprocess_Firefox/Message_Manager/Message_manager_overview