Don't show tooltip if hover is a drag

Note: This does not fix the behavior that the text of a tooltip is being selected if the anchor had an active text selection before hovering.

Change-Id: I15135dfbb040ef60af83b00ac14df38c59d522f8
Bug: 409988275
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6448276
Commit-Queue: Nikolay Vitkov <[email protected]>
Reviewed-by: Philip Pfaffe <[email protected]>

Author: Lightning00Blade

front_end/ui/components/tooltips/Tooltip.test.ts

@@ -95,6 +95,16 @@ describe('Tooltip', () => {
     assert.isFalse(container.querySelector('devtools-tooltip')?.open);
   });
 
+  it('should not be activated if dragged', async () => {
+    const container = renderTooltip();
+
+    const button = container.querySelector('button');
+    button?.dispatchEvent(new MouseEvent('mouseenter', {buttons: 1}));
+
+    await checkForPendingActivity();
+    assert.isFalse(container.querySelector('devtools-tooltip')?.open);
+  });
+
   it('should not be activated if un-focused', async () => {
     const container = renderTooltip();
 
@@ -171,7 +181,7 @@ describe('Tooltip', () => {
   });
 
   it('should print a warning if rich tooltip is used with wrong aria label on anchor', () => {
-    const consoleSpy = sinon.spy(console, 'warn');
+    const consoleSpy = sinon.stub(console, 'warn');
     renderTooltip({variant: 'rich'});
     sinon.assert.calledOnce(consoleSpy);
   });

front_end/ui/components/tooltips/Tooltip.ts

@@ -156,7 +156,11 @@ export class Tooltip extends HTMLElement {
     this.#anchorObserver?.disconnect();
   }
 
-  showTooltip = (): void => {
+  showTooltip = (event?: MouseEvent|FocusEvent): void => {
+    // Don't show the tooltip if the mouse is down.
+    if (event && 'buttons' in event && event.buttons) {
+      return;
+    }
     if (this.#timeout) {
       window.clearTimeout(this.#timeout);
     }