Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe] Reporter
	Comment 1 • 10 years ago

Repurposing... browser/devtools/layoutview/test/* should be re-enabled in browser/devtools/layoutview/test/browser.ini when they are E10S ready.

Another reason that these tests were disabled is because the mouse pointer on test boxes can produce unexpected mouseover events in tests, causing oranges depending on the test box mouse pointer.

To solve this issue we should set EventUtils.disableNonTestMouseEvents(true) in head.js for all devtools tests.

We are blocked by bug 1029451 because disableNonTestMouseEvents() incorrectly blocks synthetic mouse events but this shouldn't prevent us getting these tests ready for E10S.

	Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe] Reporter
	Comment 2 • 10 years ago

Fixed browser_layoutview.js

	Chris Peterson [:cpeterson]
	Comment 4 • 10 years ago

Moving DevTools test bugs to e10s milestone M7 (i.e. not blocking e10s merge to Aurora).

	Eddy Bruel [:ejpbruel]
	Comment 5 • 10 years ago

Hey Mike, what's the status on this bug? Is there anything we can do to unblock you?

	Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe] Reporter
	Comment 6 • 10 years ago

(In reply to Eddy Bruel [:ejpbruel] from comment #5)
> Hey Mike, what's the status on this bug? Is there anything we can do to
> unblock you?

Sure, you could fix bug 1029451.

If the mouse pointer on a test box happens to be over one of the layout view regions it causes a mouseover event causing a fail... this can be fixed with EventUtils.disableNonTestMouseEvents(true).

When we set EventUtils.disableNonTestMouseEvents(true) it erroneously prevents us from programmatically triggering mouseover events in tests, breaking the test. This is because synthetic click and mouseover events have the event.isSynthesized property set to false (bug 1029451).

Because of this issue, the risk of a layout test failing currently depends on the mouse pointer position on the test box, which is less than ideal.

	Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe] Reporter
	Comment 7 • 9 years ago

tromey fixed this on the platform side so we just need to disable none test mouse events from the JS side now.

	Nicolas Chevobbe [:nchevobbe]
	Comment 8 • 8 years ago

Inspector's Bugs Triage - Filter on CLIMBING SHOES