Status

()

defect
P5
normal
7 years ago
a month ago

People

(Reporter: jruderman, Unassigned)

Tracking

(Blocks 1 bug, {testcase})

Trunk
x86_64
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
1. Load the testcase.
2. Focus the button.
3. While moving the mouse(?), press spacebar.

Result:
  An empty black tooltip appears where the mouse cursor was last seen.

My fuzzer noticed this bug because it triggers a chrome JS error:
  JavaScript error: chrome://browser/content/browser.js, line 6553: tipElement is null
(Reporter)

Comment 1

7 years ago
Posted file testcase
We should probably cancel the existing tooltip timer when moving to fullscreen, and/or when
locking the mouse.
Assignee: nobody → bugs
Actually, I'm not sure where is the best way to cancel the tooltip, since it should probably
happen whenever we change the fullscreen element.
Assignee: bugs → nobody
Component: DOM: Events → DOM
QA Contact: events → general
But I believe adding something like
static void CancelTooltip()
  {
    if (mInstance) {
      nsRefPtr<nsXULTooltipListener> instance = mInstance;
      instance->HideTooltip();
      instance->KillTooltipTimer();
    }
  }
to nsXULTooltipListener and call it in right place could work.
Or, Neil, do you have better ideas. It is a bit ugly to #include nsXULTooltipListener.h 
in content/. Perhaps we should add CancelTooltip to PopupManager?

Comment 6

7 years ago
(In reply to Olli Pettay [:smaug] from comment #2)
> We should probably cancel the existing tooltip timer when moving to
> fullscreen, and/or when locking the mouse.

Are there events that fire when when either of these happen?

Do we want all popups to close when they happen, or just tooltips?
I guess also <select> should be closed.
https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.
Priority: -- → P5
Component: DOM → DOM: Core & HTML
Product: Core → Core
You need to log in before you can comment on or make changes to this bug.