Chromium Code Reviews
DescriptionReset HoverHighlightView state before re-populating
When a HoverHighlightView needs to be repopulated, other than removing
its children, the pointers it has to its children should be nulled out,
too. Otherwise, it would try to reuse them causing crashes. This CL adds
a Reset() function to HoverHighlightView that properly resets the state
of HoverHighlightView before re-populating it.
BUG=718411
TEST=none
Review-Url: https://codereview.chromium.org/2865533002
Cr-Commit-Position: refs/heads/master@{#469480}
(cherry picked from commit e5f363be8e46b5e650988d7393390940bd25ff67)
Review-Url: https://codereview.chromium.org/2863783002 .
Cr-Commit-Position: refs/branch-heads/3086@{#5}
Cr-Branched-From: 4159f36e93c398b9816b734a14e97220e83fc912-refs/heads/master@{#468266}
Committed: https://chromium.googlesource.com/chromium/src/+/5d20b0cede94b6473d61950c4014b125717804b6
Patch Set 1 #
Messages
Total messages: 2 (1 generated)
|
||||||||||||||||||||||||||||||||||||||||||||||