DevTools: [Elements] set non-zero hide timeout timer for image popover

third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeOutline.js

 /*
  * Copyright (C) 2007, 2008 Apple Inc.  All rights reserved.
  * Copyright (C) 2008 Matt Lilek <webkit@mattlilek.com>
  * Copyright (C) 2009 Joseph Pecoraro
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
@@ skipping 55 matching lines @@
     this._includeRootDOMNode = !omitRootDOMNode;
     this._selectEnabled = selectEnabled;
     /** @type {?WebInspector.DOMNode} */
     this._rootDOMNode = null;
     /** @type {?WebInspector.DOMNode} */
     this._selectedDOMNode = null;
 
     this._visible = false;
 
     this._popoverHelper = new WebInspector.PopoverHelper(this._element, this._getPopoverAnchor.bind(this), this._showPopover.bind(this));
-    this._popoverHelper.setTimeout(0);
+    this._popoverHelper.setTimeout(0, 100);

[dgozman, 2016/04/04 21:49:02]

Isn't 100 too short a timeout? Make it 1000 to accommodate everyone?

[lushnikov, 2016/04/04 22:35:18]

You only need it to hover over the popover.

The "showPopover" timeout is set to zero, which means you get this popover
instantly as you hover over the link. The 1000 will make them "stick" - and it
doesn't feel good, given you hover over links
accidentally 90% of the time.

 
     /** @type {!Map<!WebInspector.DOMNode, !WebInspector.ElementsTreeOutline.UpdateRecord>} */
     this._updateRecords = new Map();
     /** @type {!Set<!WebInspector.ElementsTreeElement>} */
     this._treeElementsBeingUpdated = new Set();
 
     this._domModel.addEventListener(WebInspector.DOMModel.Events.MarkersChanged, this._markersChanged, this);
 }
 
 /** @typedef {{node: !WebInspector.DOMNode, isCut: boolean}} */
@@ skipping 1697 matching lines @@
             if (node) {
                 this.treeOutline._selectedDOMNode = node;
                 this.treeOutline._selectedNodeChanged();
             }
         }
         return true;
     },
 
     __proto__: TreeElement.prototype
 }