Closed Bug 1792993 Opened 3 years ago Closed 3 years ago

Log additional error message in console when AddonManager.installSitePermsAddonFromWebpage throws

Categories

(Core :: DOM: Core & HTML, task)

task

Tracking

()

RESOLVED FIXED
107 Branch
Tracking Status
firefox107 --- fixed

People

(Reporter: nchevobbe, Assigned: nchevobbe)

References

Details

Attachments

(1 file)

With https://phabricator.services.mozilla.com/D156192 and https://phabricator.services.mozilla.com/D151477 , requestMIDIAccess could be rejected for different reasons:

  • the user rejected the addon prompt
  • requestMIDIAccess was called from a plain-IP page
  • requestMIDIAccess was called from a remote origin iframe
  • requestMIDIAccess was called from a public etld page

at the moment, all of those cases reject with the same message: "WebMIDI requires a site permission add-on to activate"

This is good when the user caused the rejection, not so much when the AddonManager caused it.

It would be nice to tweak the error message to expose the actual reason of the rejection so developers could act on that

We might also mention the media.navigator.permission.disabled pref so developers could workaround those restriction in development phases if they don't have a proper secure environment

Blocks: 1790620
Depends on: 1778959
Assignee: nobody → nchevobbe
Blocks: 1790621
No longer blocks: 1790620
Attachment #9298358 - Attachment description: Bug 1792993 - Include exception message from Addonmanager.installSitePermsAddonFromWebPage in requestMIDIAccess reject message. r=pbz. → Bug 1792993 - Log additional error message in console when Addonmanager.installSitePermsAddonFromWebPage throws. r=pbz.
Summary: Include error information from AddonManager.installSitePermsAddonFromWebpage in requestMIDIAccess reject error message → Lod additional error message in console when AddonManager.installSitePermsAddonFromWebpage throws
Status: NEW → ASSIGNED
Summary: Lod additional error message in console when AddonManager.installSitePermsAddonFromWebpage throws → Log additional error message in console when AddonManager.installSitePermsAddonFromWebpage throws
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6cb45fde952c Log additional error message in console when Addonmanager.installSitePermsAddonFromWebPage throws. r=pbz.
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 107 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: