Closed
Bug 778027
Opened 12 years ago
Closed 12 years ago
[Settings API] addObserver & onsettingchange quirks
Categories
(Core :: DOM: Device Interfaces, defect)
Tracking
()
People
(Reporter: timdream, Assigned: gwagner)
Details
Attachments
(1 file)
(deleted),
patch
|
sicking
:
review+
|
Details | Diff | Splinter Review |
Test case 1: http://pastie.org/4340018
Expected:
I/GeckoDump( 3172): ==== set
I/GeckoDump( 3172): ==== observer 1
I/GeckoDump( 3172): ==== observer 2
Actual: each observer callback is being fired twice
I/GeckoDump( 3172): ==== set
I/GeckoDump( 3172): ==== observer 1
I/GeckoDump( 3172): ==== observer 2
I/GeckoDump( 3172): ==== observer 1
I/GeckoDump( 3172): ==== observer 2
Test case 2: http://pastie.org/4340028
Expected:
I/GeckoDump( 3253): ==== set
I/GeckoDump( 3253): ==== observer 1
I/GeckoDump( 3253): ==== observer 2
Actual: observer 1 does not fire, observer 2 is being called twice.
I/GeckoDump( 3253): ==== set
I/GeckoDump( 3253): ==== observer 2
I/GeckoDump( 3253): ==== observer 2
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → anygregor
blocking-basecamp: --- → ?
Reporter | ||
Comment 1•12 years ago
|
||
Test case 3: http://pastie.org/4340633
Expected:
====foo.bar1
====foo1
====foo.bar
====foo
Actual:
====foo.bar
====foo
====foo.bar
====foo
blocking-basecamp: ? → ---
Reporter | ||
Updated•12 years ago
|
blocking-basecamp: --- → ?
Reporter | ||
Updated•12 years ago
|
Summary: [Settings API] addObserver quirks → [Settings API] addObserver & onsettingchange quirks
Reporter | ||
Comment 2•12 years ago
|
||
The way to run these test case on the phone would be replace system/index.html with the content, and start up B2G.
This issue is track on Gaia here
https://github.com/mozilla-b2g/gaia/issues/2891
Reporter | ||
Comment 3•12 years ago
|
||
Test case 4: this runs as expected -- I am reverting Gaia back to this pattern.
http://pastie.org/4342263
Attachment #647232 -
Flags: review?(jonas) → review+
Assignee | ||
Comment 5•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/bbb183d33d46
To put multiple values with a single set isn't implemented. You should use a single lock and multiple set functions instead.
Comment 6•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
Comment 8•12 years ago
|
||
Is Bug 779381 still an issue after this one is solved?
You need to log in
before you can comment on or make changes to this bug.
Description
•