HomePhabricator

Regression: Correctly log total interaction time for `dismiss` event

Authored by bmansurov on Sep 28 2016, 7:40 PM.

Description

Regression: Correctly log total interaction time for dismiss event

Per T143051, hovercards was asked to be closed after the user clicks on
the hovercard (or the link that showed it). Doing so before logging the
dismiss event caused logData to be lost. This patch logs the data first
and the clears it.

The regression was introduced in I3f3c4780cc31dc8d84cdd76df2c77fa45fbea882

Bug: T146927
Change-Id: Ie965d43b04962b2dbe2d15caa4c14bf62d1e39ea

Details