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

Status: RESOLVED FIXED

(Firefox OS Graveyard :: AudioChannel, defect)

Description

[Gregor Wagner [:gwagner]]

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.

Comment 1

[Randy Lin [:rlin] - Be Mozillian on 2015/01/26]

hmm, it seems happen on debug build, "B2G ICS Emulator Debug"
I'll fix it later.

Comment 2

[Gregor Wagner [:gwagner]]

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.

Comment 3

[Randy Lin [:rlin] - Be Mozillian on 2015/01/26]

Hi Gregor. 
Is it possible to push on your try server to verify the solution?

Comment 4

[Randy Lin [:rlin] - Be Mozillian on 2015/01/26]

Attachment: patch v1

Change another way to get visibility change (refer from fmradio.)

Comment 5

[Randy Lin [:rlin] - Be Mozillian on 2015/01/26]

Attachment: patch v1

update header.

Comment 6

[Gregor Wagner [:gwagner]]

(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.

Comment 7

[Randy Lin [:rlin] - Be Mozillian on 2015/01/26]

Hi Gregor,
I don't know how to push to the pine branch, 
Can I just change the try server location?

Comment 8

[Gregor Wagner [:gwagner]]

(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

Comment 9

[Randy Lin [:rlin] - Be Mozillian on 2015/01/26]

got green on the mochitest-4 
https://tbpl.mozilla.org/?tree=Pine&showall=1&rev=7fbaf6b2155b

Comment 10

[Randy Lin [:rlin] - Be Mozillian on 2015/01/26]

Attachment: check-in patch

carry reviewer, check-in patch

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/b2g-inbound/rev/5e6f5a0ef9cd

Comment 12

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/5e6f5a0ef9cd