Closed Bug 965594 Opened 8 years ago Closed 8 years ago

JavaScript strict warning: file:///[path]/obj/dist/bin/components/Weave.js, line 84: setting a property that has only a getter

Categories

(Firefox :: Sync, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 30

People

(Reporter: dholbert, Assigned: markh)

References

Details

(Whiteboard: [qa-])

Noticed this go by in my terminal-spew, when starting up mochitests:
{
JavaScript strict warning: file:///[path]/obj/dist/bin/components/Weave.js, line 84: setting a property that has only a getter
}

The line of code is:
> 84       this.fxAccountsEnabled = false;
http://mxr.mozilla.org/mozilla-central/source/services/sync/Weave.js?rev=974df2f038ae#84
and it was added in bug 959548.

I think the warning is saying that the assignment there is ineffective (?)
The warning is actually saying it is naughty :)  But it works.

This will be fixed when part 2 of bug 959222 lands.
Depends on: 959222
Whiteboard: [qa-]
Assignee: nobody → mhammond
Status: NEW → ASSIGNED
Component: Firefox Sync: UI → Sync
Product: Mozilla Services → Firefox
Fixed some time ago - we no longer set this.fxAccountsEnabled
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 30
You need to log in before you can comment on or make changes to this bug.