1150151 - Get e10s browser-chrome and reftests green on Linux debug

Status: RESOLVED FIXED

(Testing :: Mochitest, defect)

Description

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

We're very close to being able to do this. Just need to disable a couple tests and tweak the leak threshold for reftests.

Comment 1

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

Attachment: disable failing tests

This patch disable some tests. One is a pre-existing timeout that happens more often in debug. Then there are docshell leaks in a few social tests. Finally, there's bug 1150036, which is kinda weird.

Comment 2

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

Attachment: add crash annotation

Adds an intentional crash annotation.

Comment 3

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

Attachment: set reftest leak threshold

Bumps up the reftest leak threshold for content processes. I made it the same as the one we use for mochitests.

Comment 4

[Dave Townsend [:mossop]]

Comment on attachment 8586903 [details] [diff] [review]
disable failing tests

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

Awesome!

Comment 5

[Andrew McCreight [:mccr8]]

Comment on attachment 8586904 [details] [diff] [review]
add crash annotation

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

Hooray!

Comment 6

[Andrew McCreight [:mccr8]]

Are you actually seeing any leaks?  I looked at the C and R tests in this try run and it didn't look like there were any:
  https://treeherder.mozilla.org/#/jobs?repo=try&revision=e3dbb0b63fb2
Though I'm not sure how that's possible.

Comment 7

[Andrew McCreight [:mccr8]]

Oh right, that's an older run without Reftests being run in e10s.  I'll look at the newer one:
  https://treeherder.mozilla.org/#/jobs?repo=try&revision=49e74e9ac0fd

Comment 8

[Andrew McCreight [:mccr8]]

Comment on attachment 8586905 [details] [diff] [review]
set reftest leak threshold

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

The leaks look the same as for Mochitests, so I suppose it makes sense to make the threshold the same.

Comment 9

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/a7c811369af6
remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/2402d1757afe
remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/4d7284fd06b4

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/a7c811369af6
https://hg.mozilla.org/mozilla-central/rev/2402d1757afe
https://hg.mozilla.org/mozilla-central/rev/4d7284fd06b4