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

RESOLVED FIXED in mozilla11

Status

()

Core
XUL
RESOLVED FIXED
6 years ago
7 months ago

People

(Reporter: masayuki, Assigned: masayuki)

Tracking

(Depends on: 1 bug)

Trunk
mozilla11
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments, 1 obsolete attachment)

Created attachment 575110 [details] [diff] [review]
Patch

I have no idea for writing tests of tooltip...
Created attachment 576859 [details] [diff] [review]
Patch
Attachment #575110 - Attachment is obsolete: true
Attachment #576859 - Flags: review?(enndeakin)
Attachment #576859 - Flags: review?(bugs)

Comment 2

6 years ago
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+

Comment 3

6 years ago
>-        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'.

Updated

6 years ago
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
Depends on: 705982
https://hg.mozilla.org/mozilla-central/rev/b2a4048facc0
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Created attachment 577897 [details] [diff] [review]
test

Okay, I succeeded to write the test.
Attachment #577897 - Flags: review?(bugs)

Comment 7

6 years ago
Comment on attachment 577897 [details] [diff] [review]
test

Make sure to push this to try before landing.
Attachment #577897 - Flags: review?(bugs) → review+
of course, done.
https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=78e5abd73ab1
https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=f013b5718b97
inbound:
https://hg.mozilla.org/integration/mozilla-inbound/rev/50e07a58ee89

Updated

6 years ago
Depends on: 706743
https://hg.mozilla.org/mozilla-central/rev/50e07a58ee89
Depends on: 728541

Updated

a year ago
Depends on: 1206726
You need to log in before you can comment on or make changes to this bug.