Closed Bug 982191 Opened 6 years ago Closed 6 years ago

The resource://services-common/ alias is broken in Fennec

Categories

(Firefox for Android :: General, defect)

All
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 30

People

(Reporter: lucasr, Assigned: rnewman)

References

Details

Attachments

(1 file)

Which makes Sqlite.jsm fail to lazy load CommonUtils and cause all sorts of weirdness in HomeProvider tests.

Using resource://gre/modules/services-common works fine. Confirmed by trying to import using a remote debugger.
Blocks: 963817
services/common/servicesComponents.manifest
2:resource services-common resource://gre/modules/services-common/
Give this a shot?
This just isn't included in the Android build, see

http://mxr.mozilla.org/mozilla-central/search?string=servicesComponents

You just need to include it in mobile/android/installer/package-manifest.in, like the above examples.  This only registers the single alias above, so it looks fine.
Comment on attachment 8389313 [details] [diff] [review]
Proposed patch. v1

Review of attachment 8389313 [details] [diff] [review]:
-----------------------------------------------------------------

Assuming it works :)
Attachment #8389313 - Flags: review+
Yep, this fixes the issue, thanks.
https://hg.mozilla.org/integration/fx-team/rev/b2b8ba047803

Does this need uplift, Lucas?
Assignee: nobody → rnewman
Status: NEW → ASSIGNED
Target Milestone: --- → Firefox 30
(In reply to Richard Newman [:rnewman] from comment #6)
> https://hg.mozilla.org/integration/fx-team/rev/b2b8ba047803
> 
> Does this need uplift, Lucas?

Nope.
https://hg.mozilla.org/mozilla-central/rev/b2b8ba047803
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.