Closed Bug 703210 Opened 8 years ago Closed 8 years ago

tooltip is not shown if stopPropagation() of mousemove event is called

Categories

(Core :: XUL, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla11

People

(Reporter: masayuki, Assigned: masayuki)

References

(Depends on 1 open bug, )

Details

Attachments

(3 files, 1 obsolete file)

Attached patch Patch (obsolete) — Splinter Review
I have no idea for writing tests of tooltip...
Attached patch PatchSplinter Review
Attachment #575110 - Attachment is obsolete: true
Attachment #576859 - Flags: review?(enndeakin)
Attachment #576859 - Flags: review?(bugs)
Comment on attachment 576859 [details] [diff] [review]
Patch

The test could be a chrome test using event listener popupshowing
or some such event.
Attachment #576859 - Flags: review?(bugs) → review+
>-        evtTarget = do_QueryInterface(doc);
>-        evtTarget->AddEventListener(NS_LITERAL_STRING("DOMMouseScroll"), 
>+        // Probably, we should listen to untrusted events for hiding tooltips
>+        // on content since tooltips might disturb something of web
>+        // applications.  If we don't specify the aWantsUntrusted of
>+        // AddSystemEventListener(), the event taget set it to TRUE if the
>+        // target is in content.

Not sure about this, but 'taget' should be 'target'.
Attachment #576859 - Flags: review?(enndeakin) → review+
Hmm, I cannot make the new test. Looks like that tooltip's showing event is fired on the XUL document which has the showing tooltip. I don't know if this is an intentional behavior. I think, at least for tests, we should be able to know the tooltip state from tests. I'll file a bug for it.

https://hg.mozilla.org/integration/mozilla-inbound/rev/b2a4048facc0
Whiteboard: [inbound]
Target Milestone: --- → mozilla11
https://hg.mozilla.org/mozilla-central/rev/b2a4048facc0
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Attached patch testSplinter Review
Okay, I succeeded to write the test.
Attachment #577897 - Flags: review?(bugs)
Comment on attachment 577897 [details] [diff] [review]
test

Make sure to push this to try before landing.
Attachment #577897 - Flags: review?(bugs) → review+
Depends on: 706743
Depends on: 728541
Depends on: 1206726
Attachment #9097626 - Attachment description: Bug-1565339 disabled browser_Bug 703210.js on Windows r=#intermittent-reviewers → Bug - 1565339 disable browser_Bug 703210.js on Windows r=#intermittent-reviewers
You need to log in before you can comment on or make changes to this bug.