Closed Bug 1255036 Opened 4 years ago Closed 4 years ago

Consider removing getNotInstalled

Categories

(Core Graveyard :: DOM: Apps, defect)

defect
Not set

Tracking

(firefox48 fixed)

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: marco, Assigned: marco)

References

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

This function doesn't make sense anymore, it's always returning an empty array because all apps in the registry are launchable now.
What do you think? Can we remove this function from the Apps API?
Flags: needinfo?(myk)
Flags: needinfo?(fabrice)
Yes, I think we can remove it!
Flags: needinfo?(myk)
Sure, go ahead.
Flags: needinfo?(fabrice)
Assignee: nobody → mcastelluccio
Status: NEW → ASSIGNED
myk for dom/apps changes, ehsan for the removal of the function from dom/webidl/Apps.webidl.
Attachment #8728918 - Flags: review?(myk)
Attachment #8728918 - Flags: review?(ehsan)
Keywords: dev-doc-needed
Attachment #8728918 - Flags: review?(ehsan) → review+
Attachment #8728918 - Flags: review?(myk) → review+
https://hg.mozilla.org/mozilla-central/rev/3b9f13405a7e
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
We never ended up documenting this on MDN ;-|

I've added a note to the Firefox 48 release notes, just in case:

https://developer.mozilla.org/en-US/Firefox/Releases/48#Others
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.