The default bug view has changed. See this FAQ.

Properly send a NO_PROVIDER error when there is no options to handle a WebActivity

RESOLVED FIXED in mozilla17

Status

()

Core
DOM
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: etienne, Assigned: etienne)

Tracking

Trunk
mozilla17
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 648292 [details] [diff] [review]
Patch proposal

Looks like |ActivitiesDb.find| will always call the success callback with a result where |result.options| is an empty array if no provider was found.

Updating the "no matching activity registered" condition accordingly.
(Assignee)

Updated

5 years ago
Attachment #648292 - Flags: review?(21)
Attachment #648292 - Flags: review?(21) → review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/02cbcf0159fa
Assignee: nobody → etienne
Keywords: checkin-needed
Flags: in-testsuite-
Backed out as part of the mass tree revert due to bustage caused by other landings:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c801b99d726f

Once the tree is open again, this can reland :-)
Keywords: checkin-needed
Re-landed.
https://hg.mozilla.org/integration/mozilla-inbound/rev/91e49679104c
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/91e49679104c
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
Blocks: 715814

Updated

5 years ago
QA Contact: jsmith
Whiteboard: [qa+]

Updated

5 years ago
Keywords: verifyme
Whiteboard: [qa+]

Updated

5 years ago
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.