Closed Bug 767733 Opened 12 years ago Closed 12 years ago

MediaManager must push JS context before calling callbacks.

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED INVALID
Tracking Status
firefox13 --- unaffected
firefox14 --- unaffected

People

(Reporter: jst, Assigned: jst)

Details

(Keywords: regression)

Attachments

(1 file)

      No description provided.
Attachment #636107 - Flags: review?(bugs)
Guessing sg:moderate.
Attachment #636107 - Flags: review?(bugs) → review+
Comment on attachment 636107 [details] [diff] [review]
Use nsCxPusher before making calls to callbacks.

Actually, if these runnables are used only via event loop, 
XPConnect's push/pop should be enough. But if the runnables are used as
scriptrunners, we must have the cxpusher stuff.
These runnables should run straight from the event loop, as they're posted from the a different thread to the main thread. So if we don't need them in that case, I'm fine with marking this invalid.
Keywords: sec-moderate
Whiteboard: [sg:moderate]
Keywords: regression
This bug has kind of stalled out.  Is this still valid?
Shouldn't be valid. (though, I'm not familiar with MediaManager)
Based on my understanding, XPConnect takes care of setting up the correct JS context when passing values from chrome to content. MediaManager uses standard XPConnect techniques to pass a MediaStream to the provided callback, so I think we can safely close this bug.
(In reply to Anant Narayanan [:anant] from comment #6)
> Based on my understanding, XPConnect takes care of setting up the correct JS
> context when passing values from chrome to content.
Nothing to do with this bug

> MediaManager uses
> standard XPConnect techniques to pass a MediaStream to the provided
> callback, so I think we can safely close this bug.
By default when calling a callback one should push JS context to stack, but if it is know the
callback is always async, we can rely on XPConnect to push null to the JS context stack.
Should we untrack or is there any remaining action here for FF16?
Johnny, Olli, ping on comment 8. Should we just close this out?
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → INVALID
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: