Homec4science

Fixing a race condition for hovercards

Authored by Anh Nhan Nguyen <anhnhan@outlook.com> on Apr 6 2013, 01:25.

Description

Fixing a race condition for hovercards

Summary:
Refs T1048 - Take this setup:

D123 D321

Hover with the mouse over one object, and move to another one before it got loaded. You'll have two hovercards. Of these you can't get rid of one anymore. Not through resizing, not through showing another hovercard. (You may have to try a few times). I think it only happens when #2 loads faster than #1. #2 is displayed, then #1 gets loaded and draws.

Or something like that. You could have a race condition where one draws after another and overwrites the current node/anchor, without hiding it. This renders the previous card unhideable, even on resize / mouse far away.

Test Plan:
Did a repro by hovering a lot.

Applied this change. Could not repro anymore.

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Maniphest Tasks: T1048

Differential Revision: https://secure.phabricator.com/D5594

Details

Committed
epriestley <git@epriestley.com>Apr 6 2013, 01:25
Pushed
aubortJan 31 2017, 17:16
Parents
rPH886f3d507eff: Fix typo in PHP string
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHe1b8c9d261cd: Fixing a race condition for hovercards (authored by Anh Nhan Nguyen <anhnhan@outlook.com>).Apr 6 2013, 01:25