1310706 - Re-enable tests from dom/browser-element/mochitest/mochitest-oop.ini

Status: RESOLVED WONTFIX

(Core :: DOM: Content Processes, defect, P3)

Description

[:gerard-majax]

Some tests were disabled in bug 1266035, they would need to be re-enabled.

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

is there any desire to fix these tests?

Comment 2

[Ryan VanderMeulen (PTO, back July 27)]

These tests are already skipped on purpose on Android and e10s, so it seems highly unlikely we're ever going to invest time in re-enabling these for non-e10s.

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

I would prefer to remove the tests from the tree if that is the case, we have them perma skipped:
https://searchfox.org/mozilla-central/search?q=1310706&case=true&regexp=false&path=

any concerns with that?

Comment 4

[Ryan VanderMeulen (PTO, back July 27)]

I think that's a question for a DOM peer :)

Comment 5

[Olli Pettay [:smaug][[e-mail]]]

Hmm, does devtools still use mozbrowser? if so, it should have some tests.

Comment 6

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

(In reply to Olli Pettay [:smaug] from comment #5)
> Hmm, does devtools still use mozbrowser? if so, it should have some tests.

Yes, DevTools creates mozbrowser frames as part of Responsive Design Mode.  We also use some mozbrowser events and APIs, though currently it's a fairly limited set.

We only use remote mozbrowser frames.  So, we'd ideally have at least the OOP tests running on e10s platforms.

I'll do some investigation to see how many obstacles there are here.

Comment 7

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

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

Comment 8

[Artur Iunusov]

https://bugzilla.mozilla.org/show_bug.cgi?id=1770944

browser-element was completely removed since these times