Closed Bug 944130 Opened 6 years ago Closed 6 years ago

test_speakermanager.html: ASSERTION: background state is only maintained on outer windows

Categories

(Firefox OS Graveyard :: AudioChannel, defect)

x86
macOS
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: gwagner, Assigned: rlin)

References

Details

Attachments

(1 file, 2 obsolete files)

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: nobody → rlin
hmm, it seems happen on debug build, "B2G ICS Emulator Debug"
I'll fix it later.
Flags: needinfo?(anygregor)
Flags: needinfo?(anygregor)
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.
Hi Gregor. 
Is it possible to push on your try server to verify the solution?
Attached patch patch v1 (obsolete) — Splinter Review
Change another way to get visibility change (refer from fmradio.)
Attachment #8340733 - Flags: review?(amarchesini)
Attached patch patch v1 (obsolete) — Splinter Review
update header.
Attachment #8340733 - Attachment is obsolete: true
Attachment #8340733 - Flags: review?(amarchesini)
Attachment #8340735 - Flags: review?(amarchesini)
(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.
Hi Gregor,
I don't know how to push to the pine branch, 
Can I just change the try server location?
(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
Attachment #8340735 - Flags: review?(amarchesini) → review+
Attached patch check-in patchSplinter Review
carry reviewer, check-in patch
Attachment #8340735 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/5e6f5a0ef9cd
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.