Closed
Bug 1239170
Opened 9 years ago
Closed 5 years ago
[e10s] XUL textboxes on win7 are a different size under e10s, leading to a group of reftest failures
Categories
(Core :: DOM: Editor, defect, P4)
Core
DOM: Editor
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: jgriffin, Unassigned)
References
Details
A group of reftests fails on Windows 7 under e10s mode, because the textboxes are slightly different size than expected.
See: https://treeherder.mozilla.org/#/jobs?repo=try&revision=b6d533747a51&selectedJob=15357298
reftest analyzer link: http://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=http://archive.mozilla.org/pub/firefox/try-builds/jgriffin@mozilla.com-b6d533747a51337f9d5a327f65e08e224b3af18d/try-win32/try_win7-ix_test-reftest-bm111-tests1-windows-build1903.txt.gz&only_show_unexpected=1
These will be disabled on that config so we can get the reftest job running there.
Updated•9 years ago
|
Flags: needinfo?(mrbkap)
![]() |
||
Updated•9 years ago
|
Flags: needinfo?(mrbkap) → needinfo?(jmathies)
![]() |
||
Comment 1•9 years ago
|
||
I haven't had a chance to dig into this much. It's clear there are slight differences in the overall layout in the content process here, but they are pretty minor.
Flags: needinfo?(jmathies)
![]() |
||
Comment 2•9 years ago
|
||
- prevents us from running a few editor tests.
- falls in the polish bucket.
tests:
https://docs.google.com/spreadsheets/d/1YQ6HHcg14iPtA5hvNklnCrinfPmoxXyC_iqAczX9U2Y/edit?pref=2&pli=1#gid=68132823
Priority: -- → P4
![]() |
||
Updated•9 years ago
|
tracking-e10s:
--- → +
Comment 3•9 years ago
|
||
Ehsan, are you OK not running these tests? The thing they're testing is actually passing but they're failing for unrelated (and unimportant) reasons.
Flags: needinfo?(ehsan)
Comment 4•9 years ago
|
||
(In reply to Blake Kaplan (:mrbkap) (please use needinfo!) from comment #3)
> Ehsan, are you OK not running these tests? The thing they're testing is
> actually passing but they're failing for unrelated (and unimportant) reasons.
Sure. These tests are testing a XUL control, so we don't need to run them anyway, right?
Flags: needinfo?(ehsan)
Comment 5•9 years ago
|
||
Hmm, how can we disable these tests on e10s? It seems like there's nothing similar to skip-if=e10s for reftests...
Comment 6•9 years ago
|
||
Is it browserIsRemote?
Comment 7•9 years ago
|
||
As far as I can tell, all of the failing editor tests in comment 0 are already disabled on Windows e10s in bug 1239025.
Comment 8•9 years ago
|
||
Comment 9•5 years ago
|
||
Textboxes are going to be gone and bug 1586018 is getting rid of these tests.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•