1415115 - PERMA-FAIL | accessible/tests/mochitest/events/docload/test_docload_root.html | Test timed out.

Status: RESOLVED INVALID

(Core :: Disability Access APIs, defect)

Description

[Gary Chen [:xeonchen]]

This ONLY happens after applying patches of bug 1373708, but it didn't seem to happen several weeks ago, so I'm tracing regression range.

[1] https://treeherder.mozilla.org/logviewer.html#?job_id=142668716&repo=try&lineNumber=2620

Comment 1

[Gary Chen [:xeonchen]]

Attachment: http.log

program hangs after line #9896
> INFO Invoke the 'close dialog' test { scenario #0: expected 'focus' event;  }

Hi Yura, I saw your patch of bug 638313, do you have any idea if there's any assumption in the code that may hang hang if some resources are opened asynchronously?

In this case I can see two files:

- omni.ja!/chrome/toolkit/res/loading-image.png
- omni.ja!/chrome/toolkit/res/broken-image.png

are async-opened before #9896 and loaded after.

Comment 2

[Gary Chen [:xeonchen]]

By the way, this happens after bug 638313 is landed.

Comment 3

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: 1415115.patch

I want to see if explicit focus on window helps things on linux.

Comment 4

[alexander :surkov (:asurkov)]

Comment on attachment 8925985 [details] [diff] [review]
1415115.patch

Review of attachment 8925985 [details] [diff] [review]:
-----------------------------------------------------------------

I don't have any problems with the approach, I guess it should do a trick, but you could have enabled logging to see if the focus event is indeed not fired or, if it's fired, then where it goes to. At least you could have a better understanding what happens here.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/35a375648ee0
forcing test window focus on dialog close. r=surkov

Comment 6

[Yura Zenevich [:yzen] (please use "needinfo")]

Hi Gary, adding 'leave-open' to make sure it fixes your issue. Could you try running the test on your side with the rebased vesrion once it lands on nightly?

Comment 7

[Gary Chen [:xeonchen]]

Unfortunately, it doesn't fix this issue :(

[1] https://treeherder.mozilla.org/#/jobs?repo=try&revision=f4e7ac17b308bf06be9a915439b29f7ad7321bd3&selectedJob=142992408

Comment 8

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/35a375648ee0

Comment 9

[Yura Zenevich [:yzen] (please use "needinfo")]

(In reply to Gary Chen [:xeonchen] (needinfo plz) from comment #7)
> Unfortunately, it doesn't fix this issue :(
> 
> [1]
> https://treeherder.mozilla.org/#/
> jobs?repo=try&revision=f4e7ac17b308bf06be9a915439b29f7ad7321bd3&selectedJob=1
> 42992408

Alex, do you think we can fire a reorder event for cases like this - when child dialog window is closed?

Comment 10

[Gary Chen [:xeonchen]]

Don't know if this helps, but I found that removing all tests except test_docload_root.html will make it green.
Then I add them back one by one[1-5], only |test_docload_busy.html| or |test_docload_embedded.html| break the test.

[1] aria     https://treeherder.mozilla.org/#/jobs?repo=try&revision=8615ef1ae5aa5402a47ea7df100b49f58500665e
[2] busy     https://treeherder.mozilla.org/#/jobs?repo=try&revision=d0a5d2a2ea049c515d6ddd2531dd417ca3b47ca0
[3] embedded https://treeherder.mozilla.org/#/jobs?repo=try&revision=b4ca128635fb26e34a52bc9917659b0a430c544f
[4] iframe   https://treeherder.mozilla.org/#/jobs?repo=try&revision=595103f0b7da51145d7826df1167964f0567a497
[5] shutdown https://treeherder.mozilla.org/#/jobs?repo=try&revision=b3637784f620cbcbfc4811141a11dd7d07d8050d

Comment 11

[alexander :surkov (:asurkov)]

(In reply to Yura Zenevich [:yzen] from comment #9)
> (In reply to Gary Chen [:xeonchen] (needinfo plz) from comment #7)
> > Unfortunately, it doesn't fix this issue :(
> > 
> > [1]
> > https://treeherder.mozilla.org/#/
> > jobs?repo=try&revision=f4e7ac17b308bf06be9a915439b29f7ad7321bd3&selectedJob=1
> > 42992408
> 
> Alex, do you think we can fire a reorder event for cases like this - when
> child dialog window is closed?

reorder event on application accessible or? do you know what happens with the focus? it has to go somewhere, no?

Comment 12

[alexander :surkov (:asurkov)]

Yura, ping, see ^

Comment 13

[Yura Zenevich [:yzen] (please use "needinfo")]

Hi Gary, I'm trying to run the tests  on a local ubuntu machine with patches from bug 1373708 applied and I can't seem to get any failures (even when running with --verify flag). Do I have to do anything specific/additional to see permafail locally?

Comment 14

[Gary Chen [:xeonchen]]

(In reply to Yura Zenevich [:yzen] from comment #13)
> Hi Gary, I'm trying to run the tests  on a local ubuntu machine with patches
> from bug 1373708 applied and I can't seem to get any failures (even when
> running with --verify flag). Do I have to do anything specific/additional to
> see permafail locally?

I can't reproduce this on my platform, either.
It's somehow like a treeherder-only issue.

But if you want to reproduce locally, you have to run packaged version of gecko by:

$ mach package
$ mach mochitest --appname dist <other arguments>

Comment 15

[Gary Chen [:xeonchen]]

It's green on yesterday's m-c!

https://treeherder.mozilla.org/#/jobs?repo=try&revision=aa043f9cf4d1128ad8f6ff73009a0c76b6a0bb78

So let's close this.

Comment 16

[Yura Zenevich [:yzen] (please use "needinfo")]

(In reply to Gary Chen [:xeonchen] (needinfo plz) from comment #15)
> It's green on yesterday's m-c!
> 
> https://treeherder.mozilla.org/#/
> jobs?repo=try&revision=aa043f9cf4d1128ad8f6ff73009a0c76b6a0bb78
> 
> So let's close this.

hmm, fascinating, :)

Comment 17

[alexander :surkov (:asurkov)]

(In reply to Yura Zenevich [:yzen] from comment #16)
> (In reply to Gary Chen [:xeonchen] (needinfo plz) from comment #15)
> > It's green on yesterday's m-c!
> > 
> > https://treeherder.mozilla.org/#/
> > jobs?repo=try&revision=aa043f9cf4d1128ad8f6ff73009a0c76b6a0bb78
> > 
> > So let's close this.
> 
> hmm, fascinating, :)

yeah, a bit scary though

Comment 18

[Firefox Bug Husbandry Bot]

Removing leave-open keyword from resolved bugs, per :sylvestre.