MigrationUtils.jsm import should use app modules resource url

RESOLVED FIXED in Firefox 14

Status

()

Product:
Firefox
Component:
Migration
RESOLVED FIXED
Reported:
5 years ago
Modified:
4 years ago

People

(Reporter: mak, Assigned: mak)

Tracking

Version:
unspecified
Target:
Firefox 14
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

patch v1.0
1.71 KB, patch
fryn
: review+
Details | Diff | Splinter Review
(Assignee)

Description

•
5 years ago 
We should correct instances of resource://gre/modules/MigrationUtils.jsm to resource:///modules/MigrationUtils.jsm
(Assignee)

Comment 1

•
5 years ago 
for reference:
http://mxr.mozilla.org/mozilla-central/search?string=gre%2Fmodules%2FMigrationUtils.jsm&tree=mozilla-central
(Assignee)

Comment 2

•
5 years ago 
let's just fix this.
Assignee: nobody → mak77
(Assignee)

Comment 3

•
5 years ago 
Created attachment 616690 [details] [diff] [review]
patch v1.0
 (Assignee)

Updated

•
5 years ago
Attachment #616690 - Flags: review?(fryn)

Comment 4

•
5 years ago 
Comment on attachment 616690 [details] [diff] [review]
patch v1.0

Looks good to me.
Attachment #616690 - Flags: review?(fryn) → review+
 (Assignee)

Comment 5

•
5 years ago 
https://hg.mozilla.org/integration/mozilla-inbound/rev/df8d8d06a7bd
Target Milestone: --- → Firefox 14

Comment 6

•
5 years ago 
https://hg.mozilla.org/mozilla-central/rev/df8d8d06a7bd
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.