Closed Bug 734016 Opened 13 years ago Closed 12 years ago

navigator.mozApps.install on Success handler

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: onecyrenus, Unassigned)

References

Details

found a few inconsistencies with the html / js and the moz-central implementation. 1) navigator.mozApps.install(); Expected: Error: install missing required url argument Got: Error: Not enough arguments [mozIDOMApplicationRegistry.install] <-- is this a good enough error message ? 2) navigator.mozApps.install receipts: null in onsuccess: ←- in html / js receipts: [] in onsuccess: ← in moz-central 3) navigator.mozApps.install in onSuccess installTime = undefined in moz-central
Assignee: nobody → fabrice
Blocks: 697006
(In reply to dclarke@mozilla.com from comment #0) > 1) > navigator.mozApps.install(); > Expected: > Error: install missing required url argument > Got: > Error: Not enough arguments [mozIDOMApplicationRegistry.install] <-- is > this a good enough error message ? Welcome to XPConnect. All of our DOM APIs throw "helpful" error messages like that (try it with, say, window.addEventListener()). Also, I think you mean "onsuccess" and not "onSuccess"?
(Also, moving to Core :: DOM since navigator.mozApps is part of the DOM, not Firefox.)
Assignee: fabrice → nobody
Component: General → DOM
Product: Firefox → Core
QA Contact: general → general
Component: DOM → DOM: Device Interfaces
QA Contact: general → device-interfaces
Don't mean to bikeshed, but the webapps API is hardly about "Device Interfaces"...
David, 1) what philikon said. This kind of error doesn't even reach the API implementation (which is not bad per se). 2) I'm happy to change the behavior on m-c, but is there a spec that defines this behavior? 3) that's a bug. Can you file it on its own?
OS: Mac OS X → All
Hardware: x86 → All
2)
+ Adding anant to the thread to answer where the spec is defined. + Filing a bug for issue 3. + Updating the test, just wanted to make sure you were aware that the language was different. But I believe it would be good to standardize the text across each platform.
Component: DOM: Device Interfaces → DOM: Mozilla Extensions
QA Contact: device-interfaces → general
Blocks: 746465
Submitted Bug 763072 to address 2 (specifically that the polyfill/html should copy moz-central). The rest of the bug appears to have been addressed elsewhere.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → INVALID
Component: DOM: Mozilla Extensions → DOM
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.