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)

defect
Not set
minor

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: tchung, Assigned: mfinkle)

Details

Attachments

(1 file)

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
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: --- → ?
Attached patch patchSplinter Review
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)
I see no errors with the patch
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+
OS: Maemo → All
Hardware: ARM → All
pushed with Ci fix:
http://hg.mozilla.org/mobile-browser/rev/f9576f2cf2f8
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
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
tracking-fennec: ? → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: