Closed
Bug 561839
Opened 14 years ago
Closed 14 years ago
import PlacesUtils.jsm instead of utils.js
Categories
(Cloud Services :: General, defect)
Cloud Services
General
Tracking
(Not tracked)
RESOLVED
FIXED
1.3b4
People
(Reporter: mak, Assigned: Mardak)
References
Details
Attachments
(1 file)
1.27 KB,
patch
|
mconnor
:
review+
|
Details | Diff | Splinter Review |
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"); }
Updated•14 years ago
|
Target Milestone: --- → 1.3
Assignee | ||
Comment 2•14 years ago
|
||
Updated•14 years ago
|
Attachment #443219 -
Flags: review?(mconnor) → review+
Assignee | ||
Comment 3•14 years ago
|
||
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
Comment 5•14 years ago
|
||
> *** 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.
Comment 6•14 years ago
|
||
Because we don't push betas to AMO.
You need to log in
before you can comment on or make changes to this bug.
Description
•