Closed Bug 1250464 Opened 8 years ago Closed 8 years ago

Remove code that depends on the dom.mozApps.auto_confirm_install pref in AppsUtils.jsm

Categories

(Core Graveyard :: DOM: Apps, defect)

defect
Not set
normal

Tracking

(firefox47 affected, firefox48 fixed)

RESOLVED FIXED
mozilla48
Tracking Status
firefox47 --- affected
firefox48 --- fixed

People

(Reporter: marco, Assigned: marco)

References

Details

Attachments

(1 file)

After bug 1238079, this code will be unnecessary:

    // Before bug 910473, this is a temporary workaround to get correct path
    // from child process in mochitest.
    let prefName = "dom.mozApps.auto_confirm_install";
    if (Services.prefs.prefHasUserValue(prefName) &&
        Services.prefs.getBoolPref(prefName)) {
      return { "path": app.basePath + "/" + app.id,
               "isCoreApp": isCoreApp };
    }
I'll try to do this once bug 1238079 lands.
Assignee: nobody → mcastelluccio
Status: NEW → ASSIGNED
Attachment #8728438 - Flags: review?(myk)
Attachment #8728438 - Flags: review?(myk) → review+
https://hg.mozilla.org/mozilla-central/rev/01dd32892033
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: