Open Bug 586655 Opened 10 years ago Updated 3 years ago

test_bug511075.html fails on Fennec

Categories

(Testing :: Mochitest, defect)

x86
Windows 7
defect
Not set

Tracking

(Not tracked)

REOPENED

People

(Reporter: martijn.martijn, Unassigned)

References

()

Details

(Whiteboard: mobile_unittests)

Attachments

(1 file)

Attached patch testfixSplinter Review
test_bug511075.html fails on Fennec because Fennec doesn't have scrollbars, so big parts of that test are not useful for Fennec.
This patch makes it work for Fennec (makes the one test that is applicable for Fennec work and ignores the other tests for Fennec).

I actually noticed that this test was failing on my machine too (using Firefox), because my scrollbars are 26px wide by default. So I fixed that too, so it works on my machine.
Attachment #465242 - Flags: review?(Olli.Pettay)
Blocks: 511075
Comment on attachment 465242 [details] [diff] [review]
testfix

If this works, great :) r=me
Attachment #465242 - Flags: review?(Olli.Pettay) → review+
I'm battling already for 2 hours getting a tryserver build for this patch :(
Perhaps it's not necessary, although I only tested the patch on Windows and Maemo.
Any news here?
Whiteboard: mobile_unittests
Discovered this in deep triage. Did this ever land mw22? Is this still valid?
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WORKSFORME
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
You need to log in before you can comment on or make changes to this bug.