Closed Bug 1455114 Opened 2 years ago Closed 2 years ago

TPS can't be imported as an XPI

Categories

(Firefox :: Sync, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
Firefox 61
Tracking Status
firefox61 --- fixed

People

(Reporter: tcsc, Assigned: tcsc)

Details

Attachments

(1 file)

TPS's resource URI's still fail to load (e.g. bug 1449248).

Reported by davehunt in #sync, and after some debugging in #fx-team, it turns out the resource URI needs to end with a trailing slash? Who knows.

I'm fixing an instance of unnecessary whitespace in this file also, since it's bugged me for a while.

Patch incoming
Comment on attachment 8969065 [details]
Bug 1455114 - Fix resource://tps/* failing to load when TPS is used as an XPI

https://reviewboard.mozilla.org/r/237752/#review243548

Hooray for artisanal module loading! :-)
Attachment #8969065 - Flags: review?(kit) → review+
Pushed by tchiovoloni@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/136bae04fe6f
Fix resource://tps/* failing to load when TPS is used as an XPI r=kitcambridge
https://hg.mozilla.org/mozilla-central/rev/136bae04fe6f
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
You need to log in before you can comment on or make changes to this bug.