732565 - layout/reftests/percent-overflow-sizing/hscroll* has 17 tests that fails on android when we turn browser.tabs.remote=false

Status: RESOLVED WONTFIX

(Core :: Layout, defect)

Description

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

On android xul, we run with e10s and our reftest harness will setup the xul window to have the remote=true attribute.  For native fennec we have the preference to turn on e10s set, but we don't read it.  This means that reftest is still thinking we are remote, but in reality we are not.  

In trying to turn this off, I found that we have 17 tests which passed but we have marked as fails-if(android)

we need to figure out why this fails on android with e10s enabled and either fix it or figure out the appropriate manifest condition to not run it.

The log file with the e10s enabled android xul is here:
http://people.mozilla.org/~jmaher/android_reftests/percent-overflow-sizing.e10s.log

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

It seems to be a difference of the tests having color #008000 and the references having #008200 (at least in all of the 5 or so I looked at).

Comment 2

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

these pass on native android without browser.tabs.remote set.

Comment 3

[Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)]

In bug #1072417 we removed the browser.tabs.remote,
so can this bug be closed or we should leave it open as it's other case?

Comment 4

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

I don't believe the problem has been fixed, we still have e10s, browser.tabs.remote was how we turned on e10s 2 years ago.

Comment 5

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

if we want to close this, please remove the manifest annotations:
http://dxr.mozilla.org/mozilla-central/source/layout/reftests/percent-overflow-sizing/reftest.list#3

The whole purpose of this bug was that we failed in e10s back on the first round of e10s.  Now e10s has changed including the prefs we cannot just close bugs out without doing cleanup (i.e. removing the annotations)

Comment 6

[Geoff Brown [:gbrown]]

Test manifests updated in bug 1257948.