Closed Bug 1060621 Opened 5 years ago Closed 5 years ago
Scope should CC m Location and m Navigator
It doesn't, but that seems odd. Why doesn't it? Should at least be documented.
Comment on attachment 8482785 [details] [diff] [review] cc.patch r=me; add the testcase?
Attachment #8482785 - Flags: review?(bzbarsky) → review+
Do we want this on branches? Should be rather safe patch.
Might not be a bad idea. I guess this has been a problem for a while....
Assignee: nobody → amarchesini
OS: Mac OS X → All
Hardware: x86 → All
Attachment #8482785 - Attachment is obsolete: true
Summary: Should WorkerScope CC mLocation and mNavigator? → WorkerScope should CC mLocation and mNavigator
Can you also put in a test for an expando on location?
(In reply to Andrew McCreight [:mccr8] from comment #6) > Can you also put in a test for an expando on location? ah sorry... I didn't see your comment. I'll do in the other patch about WorkerNavigator.location.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
Comment on attachment 8482815 [details] [diff] [review] cc.patch Approval Request Comment [Feature/regressing bug #]: We need this to land bug 1062920 without leaking. [User impact if declined]: We won't take bug 1062920 [Describe test coverage new/current, TBPL]: Worker code is well tested in automation. [Risks and why]: Low risk [String/UUID change made/needed]: N/A
You need to log in before you can comment on or make changes to this bug.