HomePhabricator

Regression: correctly log the click action total interaction time

Authored by bmansurov on Sep 28 2016, 8:17 PM.

Description

Regression: correctly log the click action total interaction time

Per T143051, hovercards was asked to be closed after the user clicks on
the hovercard (or the link that showed it). Closing the popup before
logging the click event causes logData to be lost. This patch logs
the data first and then clears it.

The regression was introduced in Ifd6f75c2a53d8d7b5ef9fd3f232f85b55eea24c8.

Also do not attempt closing a popup when it may not be open.

Bug: T146934
Change-Id: I02febc83036130bcea0a769114c9126cb481bafe