[e10s] Re-enable and fix layout/xul/test tests

RESOLVED FIXED in Firefox 40

Status

()

Core
General
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: Neil Deakin, Assigned: Neil Deakin)

Tracking

(Depends on: 1 bug, Blocks: 1 bug)

unspecified
mozilla40
Points:
2
Dependency tree / graph

Firefox Tracking Flags

(e10s+, firefox40 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Created attachment 8593557 [details] [diff] [review]
browser_layoutxul

Three tests are disabled.

This patch reworks them to use BrowserTestUtils functions.

Two real fixes were needed:
 - the browser.js change because normal tooltips from nsXULTooltipListener ignore mouse events while a drag is occurring. This change adds similar support here.
 - need to use system event listeners so that stopPropagation in a child doesn't block the tooltip from appearing.
(Assignee)

Updated

3 years ago
Iteration: --- → 40.2 - 27 Apr
Points: --- → 2
(Assignee)

Comment 1

3 years ago
Comment on attachment 8593557 [details] [diff] [review]
browser_layoutxul

These are all tooltip related tests.
Attachment #8593557 - Flags: review?(wmccloskey)
Comment on attachment 8593557 [details] [diff] [review]
browser_layoutxul

Review of attachment 8593557 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!

::: layout/xul/test/browser_bug685470.js
@@ +1,2 @@
> +add_task(function* () {
> +  const url = "<p id=\"p1\" title=\"tooltip is here\">This paragraph has a tooltip.</p>";

Maybe call this "html" or something instead of url.

::: layout/xul/test/browser_bug703210.js
@@ +15,2 @@
>  
> +  yield BrowserTestUtils.synthesizeMouseAtCenter("#p2", { type: "mousemove" }, browser);

Is this initial mousemove just to ensure that any existing tooltip is hidden? If so, please comment.
Attachment #8593557 - Flags: review?(wmccloskey) → review+

Updated

3 years ago
Blocks: 984139
tracking-e10s: --- → +
https://hg.mozilla.org/mozilla-central/rev/c36ce7aed6d9
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Depends on: 1157276
Depends on: 1157576
You need to log in before you can comment on or make changes to this bug.