Closed Bug 561839 Opened 14 years ago Closed 14 years ago

import PlacesUtils.jsm instead of utils.js

Categories

(Cloud Services :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mak, Assigned: Mardak)

References

Details

Attachments

(1 file)

Bug 560104 introduced PlacesUtils.jsm, the old utils.js file is now a mock that imports PlacesUtils.jsm, for backwards compatibility, but it will put an error in the console when imported (still will work fine).

Weave should use this code path for backwards compatibility.

Try {
  Components.utils.import("resource://gre/modules/PlacesUtils.jsm");
} catch (ex) {
  Components.utils.import("resource://gre/modules/utils.js");
}
Target Milestone: --- → 1.3
Attached patch v1Splinter Review
Assignee: nobody → edilee
Status: NEW → ASSIGNED
Attachment #443219 - Flags: review?(mconnor)
Attachment #443219 - Flags: review?(mconnor) → review+
http://hg.mozilla.org/labs/weave/rev/c3385ba3fecc
Try the new PlacesUtils file before the old utils.js.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: 1.3 → 1.3b4
> *** Bug 566838 has been marked as a duplicate of this bug. ***
So why didn't my Weave Sync auto update from Addons.mozilla.org I wonder. Manually checking for updates didn't work either.
Because we don't push betas to AMO.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: