Closed
Bug 734016
Opened 13 years ago
Closed 12 years ago
navigator.mozApps.install on Success handler
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
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
Reporter | ||
Updated•13 years ago
|
Assignee: nobody → fabrice
Comment 1•13 years ago
|
||
(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"?
Comment 2•13 years ago
|
||
(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
Updated•13 years ago
|
Component: DOM → DOM: Device Interfaces
QA Contact: general → device-interfaces
Comment 3•13 years ago
|
||
Don't mean to bikeshed, but the webapps API is hardly about "Device Interfaces"...
Comment 4•13 years ago
|
||
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
Reporter | ||
Comment 5•13 years ago
|
||
2)
Reporter | ||
Comment 6•13 years ago
|
||
+ 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.
Updated•13 years ago
|
Component: DOM: Device Interfaces → DOM: Mozilla Extensions
QA Contact: device-interfaces → general
Comment 7•12 years ago
|
||
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
Assignee | ||
Updated•12 years ago
|
Component: DOM: Mozilla Extensions → DOM
Assignee | ||
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•