Closed
Bug 583369
Opened 14 years ago
Closed 14 years ago
Unable to save new string values in about:config settings
Categories
(Firefox for Android Graveyard :: General, defect)
Firefox for Android Graveyard
General
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: tchung, Assigned: mfinkle)
Details
Attachments
(1 file)
1.02 KB,
patch
|
mbrubeck
:
review+
|
Details | Diff | Splinter Review |
When trying to edit an existing type=string values, there's no way to save the changes. Typing in the new string, then clicking away will only revert that value back to its original. Theres no dynamic updates shown. Also observe that the "Reset" button never gets enabled and remains grayed out Repro: 1) install fennec nightly on maemo 20100730 2) open about:config, and search for a type=string value (eg. services.sync.log.logger.engine.passwords = "Debug") 3) change the value of that string so something else (eg "Trace") 4) Verify there's no save button, Reset button remains disabled, and clicking away will only revert your value back to the original Expected: - Dynamic updating of value changes for string type configs Actual: - does not save new string value
Comment 1•14 years ago
|
||
Confirmed with the latest trunk build. In the error console, I get these js errors, every time, I type something of the word "Trace": Error: CC is not defined Source: chrome://browser.content/bindings/setting.xml line nr: 323
tracking-fennec: --- → ?
Assignee | ||
Comment 2•14 years ago
|
||
Cc and Ci are not available when running in content. They are borrowed from browser.js when used in chrome.
Assignee: nobody → mark.finkle
Attachment #464118 -
Flags: review?(mbrubeck)
Assignee | ||
Comment 3•14 years ago
|
||
I see no errors with the patch
Comment 4•14 years ago
|
||
Comment on attachment 464118 [details] [diff] [review] patch There are a few other uses of Ci in the file which should be fixed too. r=mbrubeck with that change.
Attachment #464118 -
Flags: review?(mbrubeck) → review+
Updated•14 years ago
|
OS: Maemo → All
Hardware: ARM → All
Assignee | ||
Comment 5•14 years ago
|
||
pushed with Ci fix: http://hg.mozilla.org/mobile-browser/rev/f9576f2cf2f8
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 6•14 years ago
|
||
Verified fix on Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv:2.0b4pre) Gecko/20100810 Namoroka/4.04pre Fennec/2.0a1pre
Status: RESOLVED → VERIFIED
Updated•10 years ago
|
tracking-fennec: ? → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•