1302352 - browser_context_menu_iframe.js is skipped in e10s

Status: RESOLVED FIXED

(Toolkit :: Password Manager, defect)

Description

[Evan Tseng [:evanxd]]

The "browser_context_menu_iframe.js" test is skiped[1] on e10s environment. Let's re-enable it.

[1]: http://searchfox.org/mozilla-central/source/toolkit/components/passwordmgr/test/browser/browser.ini#56-57

Comment 1

[Evan Tseng [:evanxd]]

Pushed a new try[1]: Re-enable the test to debug.

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

Comment 2

[Evan Tseng [:evanxd]]

Re-trigger "browser_context_menu_iframe.js" test for multiple times to reproduce the intermittent failure.

Comment 3

[Evan Tseng [:evanxd]]

The "browser_context_menu_iframe.js" test is passed for 50 times continuously. Looks like we can re-enable it.

Comment 4

[Evan Tseng [:evanxd]]

Attachment: bug-1302352.patch

Hi Matthew,

Let's re-enable the test. It is passed for 50 times continuously on the try[1].

Thanks for the review.

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

Comment 5

[Evan Tseng [:evanxd]]

Pushed a try[1] for updating commit message.

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

Comment 6

[Matthew N. [:MattN]]

I think it was just skipped because it was assumed that it should be since it was copied from browser_context_menu.js (which is disabled in e10s).

Comment 7

[Matthew N. [:MattN]]

Thanks

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/fda5c90a13da
Re-enable browser_context_menu_iframe.js in e10s environment. r=MattN

Comment 9

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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