Closed
Bug 854138
Opened 12 years ago
Closed 12 years ago
Fix blank Sound and Display settings screens (JavaScript error: app://settings.gaiamobile.org/js/settings.js, line 294: ranges[i].refresh is not a function)
Categories
(Firefox OS Graveyard :: Gaia::Settings, defect)
Tracking
(b2g-v1.1hd fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
b2g-v1.1hd | --- | fixed |
People
(Reporter: jwatt, Assigned: jwatt)
References
Details
Attachments
(1 file)
(deleted),
patch
|
kaze
:
review+
|
Details | Diff | Splinter Review |
The Sound and Display settings screens in B2G builds that are built against mozilla-central are broken (blank) and render the phone unusable until rebooted if the user tries to view them. The console contains this error:
JavaScript error: app://settings.gaiamobile.org/js/settings.js, line 294: ranges[i].refresh is not a function
The issue is that the polyfill code for <input type=range> adds a refresh() method to the element that replaces the <input>, and some code calls that without checking if the element is a native implementation or the polyfill.
The pref to enable native <input type=range> was flipped on in bug 841948.
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #728661 -
Flags: review?(mwu)
Assignee | ||
Comment 2•12 years ago
|
||
I'm making the code work in the case where the native <input type=range> is both enabled and disabled (rather than removing the polyfill) since the gaia code has to work with both older and newer gecko trees for now.
Comment 3•12 years ago
|
||
Comment on attachment 728661 [details] [diff] [review]
patch
Forwarding review to someone who might know the code.
Attachment #728661 -
Flags: review?(mwu) → review?(kaze)
Assignee | ||
Comment 4•12 years ago
|
||
Pull request is here: https://github.com/mozilla-b2g/gaia/pull/8784
Comment 5•12 years ago
|
||
Comment on attachment 728661 [details] [diff] [review]
patch
Review of attachment 728661 [details] [diff] [review]:
-----------------------------------------------------------------
https://github.com/mozilla-b2g/gaia/commit/1eb7430d65e7ae1c355b758fbc18ec3f00c5e57e
Attachment #728661 -
Flags: review?(kaze) → review+
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 6•11 years ago
|
||
Need this one for v1.1.0hd (related to https://bugzilla.mozilla.org/show_bug.cgi?id=854469)
status-b2g-v1.1hd:
--- → ?
Comment 7•11 years ago
|
||
Please, see comment 6 ^^ : https://bugzilla.mozilla.org/show_bug.cgi?id=854138#c6
Flags: needinfo?(wchang)
Comment 8•11 years ago
|
||
Updated•11 years ago
|
Flags: needinfo?(wchang)
You need to log in
before you can comment on or make changes to this bug.
Description
•