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

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

The non-zero hide timeout makes it possible to hover over popover
and copy some useful information from it.

BUG=599592
R=dgozman, pfeldman

Committed: https://crrev.com/d217d6f3d6fd76f78b153ee1de31557a353168e8
Cr-Commit-Position: refs/heads/master@{#385048}

[lushnikov, 2016-04-04 21:37:44]

pta

[lushnikov, 2016-04-04 21:37:49]

The CQ bit was checked by lushnikov@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-04 21:38:42]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1861443002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1861443002/1

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

lgtm

https://codereview.chromium.org/1861443002/diff/1/third_party/WebKit/Source/d...
File
third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeOutline.js
(right):

https://codereview.chromium.org/1861443002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeOutline.js:76:
this._popoverHelper.setTimeout(0, 100);
Isn't 100 too short a timeout? Make it 1000 to accommodate everyone?

[lushnikov, 2016-04-04 22:35:19]

https://codereview.chromium.org/1861443002/diff/1/third_party/WebKit/Source/d...
File
third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeOutline.js
(right):

https://codereview.chromium.org/1861443002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeOutline.js:76:
this._popoverHelper.setTimeout(0, 100);
On 2016/04/04 21:49:02, dgozman wrote:
> Isn't 100 too short a timeout? Make it 1000 to accommodate everyone?

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.

[commit-bot: I haz the power, 2016-04-04 23:14:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-04 23:14:35]

Dry run: This issue passed the CQ dry run.

[lushnikov, 2016-04-04 23:14:43]

The CQ bit was checked by lushnikov@chromium.org

[commit-bot: I haz the power, 2016-04-04 23:15:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1861443002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1861443002/1

[commit-bot: I haz the power, 2016-04-04 23:23:13]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-04 23:24:41]

Description was changed from

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

The non-zero hide timeout makes it possible to hover over popover
and copy some useful information from it.

BUG=599592
R=dgozman, pfeldman
==========

to

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

The non-zero hide timeout makes it possible to hover over popover
and copy some useful information from it.

BUG=599592
R=dgozman, pfeldman

Committed: https://crrev.com/d217d6f3d6fd76f78b153ee1de31557a353168e8
Cr-Commit-Position: refs/heads/master@{#385048}
==========

[commit-bot: I haz the power, 2016-04-04 23:24:42]

Patchset 1 (id:??) landed as
https://crrev.com/d217d6f3d6fd76f78b153ee1de31557a353168e8
Cr-Commit-Position: refs/heads/master@{#385048}