Closed Bug 1789245 Opened 4 years ago Closed 4 years ago

When "openDevTools" is not passed: installTemporaryAddon: Error: undefined passed where a value is required

Categories

(DevTools :: General, defect)

defect

Tracking

(firefox-esr91 unaffected, firefox-esr102 unaffected, firefox104 unaffected, firefox105 unaffected, firefox106 fixed)

RESOLVED FIXED
106 Branch
Tracking Status
firefox-esr91 --- unaffected
firefox-esr102 --- unaffected
firefox104 --- unaffected
firefox105 --- unaffected
firefox106 --- fixed

People

(Reporter: robwu, Assigned: robwu)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Originally reported at https://github.com/mozilla/web-ext/issues/2499

STR:

  1. Create a directory containing an extension (e.g. by taking an existing extension/xpi file and unzipping it)
  2. Install web-ext if needed.
  3. Run web-ext run -f nightly

Expected:

  • Nightly launches with the extension installed.

Actual:

  • Nightly cannot launch. The following message is displayed by web-ext:

WebExtError: installTemporaryAddon: Error: Error: undefined passed where a value is required

I believe that this is due to a change in https://searchfox.org/mozilla-central/rev/918fd22032de3a0025d6e9f4fcc8b7f625315068/devtools/shared/specs/addon/addons.js#13

If I look at other protocol definitions, then I suppose that prepending nullable: to the boolean would fix this issue.

I'll attach a patch to get this rolling.

Assignee: nobody → rob
Status: NEW → ASSIGNED
Pushed by rob@robwu.nl: https://hg.mozilla.org/integration/autoland/rev/863b5207dfe1 [devtools] Mark openDevTools as nullable for installTemporaryAddon r=devtools-backward-compat-reviewers,ochameau
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 106 Branch
QA Whiteboard: [qa-106b-p2]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: