Closed
Bug 944130
Opened 9 years ago
Closed 9 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•9 years ago
|
Assignee: nobody → rlin
Assignee | ||
Comment 1•9 years ago
|
||
hmm, it seems happen on debug build, "B2G ICS Emulator Debug" I'll fix it later.
Flags: needinfo?(anygregor)
Assignee | ||
Updated•9 years ago
|
Flags: needinfo?(anygregor)
Reporter | ||
Comment 2•9 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•9 years ago
|
||
Hi Gregor. Is it possible to push on your try server to verify the solution?
Assignee | ||
Comment 4•9 years ago
|
||
Change another way to get visibility change (refer from fmradio.)
Attachment #8340733 -
Flags: review?(amarchesini)
Assignee | ||
Comment 5•9 years ago
|
||
update header.
Attachment #8340733 -
Attachment is obsolete: true
Attachment #8340733 -
Flags: review?(amarchesini)
Assignee | ||
Updated•9 years ago
|
Attachment #8340735 -
Flags: review?(amarchesini)
Reporter | ||
Comment 6•9 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•9 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•9 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•9 years ago
|
||
got green on the mochitest-4 https://tbpl.mozilla.org/?tree=Pine&showall=1&rev=7fbaf6b2155b
Updated•9 years ago
|
Attachment #8340735 -
Flags: review?(amarchesini) → review+
Assignee | ||
Comment 10•9 years ago
|
||
carry reviewer, check-in patch
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Updated•9 years ago
|
Attachment #8340735 -
Attachment is obsolete: true
Comment 11•9 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/5e6f5a0ef9cd
Keywords: checkin-needed
Comment 12•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/5e6f5a0ef9cd
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•