Closed Bug 1250963 Opened 5 years ago Closed 5 years ago

Stop passing a JSContext to ModifyBusyCount

Categories

(Core :: DOM: Workers, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla47
Tracking Status
firefox47 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

Details

(Whiteboard: btpp-active)

Attachments

(2 files)

It's not needed, afaict.
The only reason NotifyRunnable::Dispatch needs a JSContext is so that it can call
ModifyBusyCount in Pre/PostDispatch.  The only reason that needs a JSContext is
to call Cancel(), which only needs it to call Notify(), which only needs it to
call NotifyPrivate, which only needs it to dispatch a NotifyRunnable.
Attachment #8723120 - Flags: review?(khuey)
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Blocks: 1250975
Whiteboard: btpp-active
There was also an android mochitest failure when these patches were in the tree: https://treeherder.mozilla.org/logviewer.html#?job_id=22434259&repo=mozilla-inbound

Unsure if that'd be fixed as part of the fix for the build bustage or if it's a separate issue.
The issue was bug 1250975 for the static analysis issues.  I'm not sure which one is the Android issue; will look into it.
Flags: needinfo?(bzbarsky)
That was bug 1250975 too.
You need to log in before you can comment on or make changes to this bug.