Closed
Bug 944130
Opened 12 years ago
Closed 12 years ago
test_speakermanager.html: ASSERTION: background state is only maintained on outer windows
Categories
(Firefox OS Graveyard :: AudioChannel, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gwagner, Assigned: rlin)
References
Details
Attachments
(1 file, 2 obsolete files)
1.36 KB,
patch
|
Details | Diff | Splinter Review |
We have a new mochitest failure on debug-emaulators since yesterday:
https://tbpl.mozilla.org/php/getParsedLog.php?id=31184129&tree=Pine&full=1#error1
14:00:50 INFO - 11-27 21:32:40.410 671 671 I Gonk : NFC: OnConnectError
14:00:50 INFO - 11-27 21:32:40.410 671 671 I Gonk : NFC: OnDisconnect
14:00:50 INFO - 11-27 21:32:40.660 717 717 I Gecko : [Child 717] ###!!! ASSERTION: background state is only maintained on outer windows: 'IsOuterWindow()', file ../../dist/include/nsPIDOMWindow.h, line 105
14:00:50 INFO - 11-27 21:32:40.689 40 794 W AudioPolicyManagerBase: setForceUse() invalid config 1 for FOR_MEDIA
14:00:50 INFO - 11-27 21:32:40.849 717 717 I GeckoDump: 13898 INFO TEST-PASS | /tests/dom/speakermanager/tests/test_speakermanager.html | speaker should be true
14:00:50 INFO - 11-27 21:32:40.889 717 717 I GeckoDump: 13899 INFO TEST-PASS | /tests/dom/speakermanager/tests/test_speakermanager.html | speaker should be false
14:00:50 INFO - 11-27 21:32:41.160 717 717 I GeckoDump: 13900 INFO TEST-INFO | MEMORY STAT vsize after test: 116252672
14:00:50 INFO - 11-27 21:32:41.220 717 717 I GeckoDump: 13901 INFO TEST-INFO | MEMORY STAT residentFast after test: 65892352
14:00:50 INFO - 11-27 21:32:41.270 717 717 I GeckoDump: 13902 INFO TEST-INFO | MEMORY STAT heapAllocated after test: 24230288
14:00:50 INFO - 11-27 21:32:41.420 671 677 I GonkDBus: Socket connect errno=2
14:00:50 INFO - 11-27 21:32:41.420 671 671 I Gonk : NFC: OnConnectError
14:00:50 INFO - 11-27 21:32:41.420 671 671 I Gonk : NFC: OnDisconnect
14:00:50 INFO - 11-27 21:32:41.810 717 717 I GeckoDump: 13903 INFO TEST-END | /tests/dom/speakermanager/tests/test_speakermanager.html | finished in 3981ms
14:00:50 INFO - 11-27 21:32:42.140 717 717 I Gecko : ++DOMWINDOW == 16 (0x43cb8b10) [pid = 717] [serial = 811] [outer = 0x43cb81b0]
14:00:50 INFO - 11-27 21:32:42.420 671 671 I Gonk : NFC: OnDisconnect
14:00:50 INFO - 11-27 21:32:42.779 717 717 I GeckoDump: 13904 ERROR TEST-UNEXPECTED-FAIL | /tests/dom/speakermanager/tests/test_speakermanager.html | Assertion count 1 is greater than expected range 0-0 assertions.
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → rlin
Assignee | ||
Comment 1•12 years ago
|
||
hmm, it seems happen on debug build, "B2G ICS Emulator Debug"
I'll fix it later.
Flags: needinfo?(anygregor)
Assignee | ||
Updated•12 years ago
|
Flags: needinfo?(anygregor)
Reporter | ||
Comment 2•12 years ago
|
||
We are about to unhide those tests on all tbpl trees. There are only 2 assertions left and this is one of them. Please fix it asap.
Assignee | ||
Comment 3•12 years ago
|
||
Hi Gregor.
Is it possible to push on your try server to verify the solution?
Assignee | ||
Comment 4•12 years ago
|
||
Change another way to get visibility change (refer from fmradio.)
Attachment #8340733 -
Flags: review?(amarchesini)
Assignee | ||
Comment 5•12 years ago
|
||
update header.
Attachment #8340733 -
Attachment is obsolete: true
Attachment #8340733 -
Flags: review?(amarchesini)
Assignee | ||
Updated•12 years ago
|
Attachment #8340735 -
Flags: review?(amarchesini)
Reporter | ||
Comment 6•12 years ago
|
||
(In reply to Randy Lin [:rlin] from comment #3)
> Hi Gregor.
> Is it possible to push on your try server to verify the solution?
Sure, feel free to use the pine branch. I merged yesterday but something seems to be broken right now. I will find debug tomorrow.
Assignee | ||
Comment 7•12 years ago
|
||
Hi Gregor,
I don't know how to push to the pine branch,
Can I just change the try server location?
Reporter | ||
Comment 8•12 years ago
|
||
(In reply to Randy Lin [:rlin] from comment #7)
> Hi Gregor,
> I don't know how to push to the pine branch,
> Can I just change the try server location?
I pushed your diff: https://tbpl.mozilla.org/?tree=Pine&showall=1&rev=7fbaf6b2155b
Assignee | ||
Comment 9•12 years ago
|
||
got green on the mochitest-4
https://tbpl.mozilla.org/?tree=Pine&showall=1&rev=7fbaf6b2155b
Updated•12 years ago
|
Attachment #8340735 -
Flags: review?(amarchesini) → review+
Assignee | ||
Comment 10•12 years ago
|
||
carry reviewer, check-in patch
Assignee | ||
Updated•12 years ago
|
Keywords: checkin-needed
Updated•12 years ago
|
Attachment #8340735 -
Attachment is obsolete: true
Comment 11•12 years ago
|
||
Keywords: checkin-needed
Comment 12•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•