1221159 - re-enable a few working tests for e10s

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Tracy Walker [:tracy]]

I've found a few test cases in /browser/base/content/test/general/ that are working in e10s:

browser_canonizeURL.js (Bug 1094510)
browser_contextSearchTabPosition.js (bug 967013 and bug 1094761)
browser_datachoices_notification.js (bug 1113930)
browser_popup_blocker.js (bug 1081925 and bug 112552)

try run shows they should be good to be re-enable on all platforms: https://treeherder.mozilla.org/#/jobs?repo=try&revision=7a76da23e34e

Comment 1

[Tracy Walker [:tracy]]

Attachment: browser-ini-patch

Patch attached with r=jimm

Comment 2

[Jim Mathies [:jimm]]

Comment on attachment 8682552 [details] [diff] [review]
browser-ini-patch

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

::: browser/base/content/test/general/browser.ini
@@ -357,5 @@
>  [browser_plainTextLinks.js]
>  [browser_popupUI.js]
>  skip-if = buildapp == 'mulet'
>  [browser_popup_blocker.js]
> -skip-if = (os == 'linux') || (e10s && debug) # Frequent bug 1081925 and bug 1125520 failures

this was disabled under linux generally (the os == 'linux' check), did you intend to turn this back on for that platform?

Comment 3

[Tracy Walker [:tracy]]

(In reply to Jim Mathies [:jimm] from comment #2)
> Comment on attachment 8682552 [details] [diff] [review]
> browser-ini-patch
> 
> Review of attachment 8682552 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: browser/base/content/test/general/browser.ini
> @@ -357,5 @@
> >  [browser_plainTextLinks.js]
> >  [browser_popupUI.js]
> >  skip-if = buildapp == 'mulet'
> >  [browser_popup_blocker.js]
> > -skip-if = (os == 'linux') || (e10s && debug) # Frequent bug 1081925 and bug 1125520 failures
> 
> this was disabled under linux generally (the os == 'linux' check), did you
> intend to turn this back on for that platform?

Yes, I ran it through try with it enabled on Linux. It appeared to be fine in the try run.

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/7195286a81bd

Comment 5

[Nigel Babu [:nigelb]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a0c456c0161fd5a289fbd65ba95b7ab3f10f1f93
Backed out changeset 7195286a81bd (bug 1221159) for e10s bc7 bustage

Comment 6

[Tracy Walker [:tracy]]

In the try run I thought the bc7 bustage was not associated with my patch. It seems bug 1217332 may be blocking this.

I'll try to sort out which case enabled in my patch, if any, is causing that later test case, browser_syncui.js, to fail on Mac.

Comment 7

[Phil Ringnalda (:philor)]

This has nothing whatsoever to do with that. You were backed out for the Linux e10s bc7 bustage in browser_canonizeURL.js, where it hits the network only with e10s, just like the comment disabling it said, like https://treeherder.mozilla.org/logviewer.html#?job_id=16836406&repo=mozilla-inbound. Your try run didn't even run Linux e10s bc7, and thus didn't run browser_canonizeURL.js.

Comment 8

[Tracy Walker [:tracy]]

My try run did run Linux e10s bc7. It did not, however, run any test for Linux x64.  I'll make sure browser_canonizeURL.js remains disabled for linux in a new patch.

Comment 9

[Tracy Walker [:tracy]]

Attachment: general-broswer-ini.patch

circling back around here with an updated patch.  try run with the new set of test cases to re-enable is clean: https://treeherder.mozilla.org/#/jobs?repo=try&revision=6e9ba8b38969

Comment 10

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/c4de716628d2

Comment 11

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c4de716628d2