Keep top controls visible if SHOW is called right after HIDE.

Keep top controls visible if SHOW is called right after HIDE.

Currently, we early exit if the visibility amount matches our
desired ending amount.  But when you set HIDDEN as the current,
it will start an animation, and a immediate call to SHOWN will
see that the ending condition is met and return, but does not
clear the animation.

Now, clear the animation if the current value matches our
desired ending value to ensure it doesn't move.

BUG=639702
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

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

[Ted C, 2016-09-27 00:07:35]

Description was changed from

==========
Keep top controls visible if SHOW is called right after HIDE.

Currently, we early exit if the visibility amount matches our
desired ending amount.  But when you set HIDDEN as the current,
it will start an animation, and a immediate call to SHOWN will
see that the ending condition is met and return, but does not
clear the animation.

Now, clear the animation if the current value matches our
desired ending value to ensure it doesn't move.

BUG=639702
==========

to

==========
Keep top controls visible if SHOW is called right after HIDE.

Currently, we early exit if the visibility amount matches our
desired ending amount.  But when you set HIDDEN as the current,
it will start an animation, and a immediate call to SHOWN will
see that the ending condition is met and return, but does not
clear the animation.

Now, clear the animation if the current value matches our
desired ending value to ensure it doesn't move.

BUG=639702
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[Ted C, 2016-09-27 00:08:07]

tedchoc@chromium.org changed reviewers:
+ aelias@chromium.org

[Ted C, 2016-09-27 00:08:08]

PTAL

[aelias_OOO_until_Jul13, 2016-09-27 04:25:47]

lgtm

[Ted C, 2016-09-27 04:29:59]

The CQ bit was checked by tedchoc@chromium.org

[commit-bot: I haz the power, 2016-09-27 04:30:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-27 06:11:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-27 06:14:57]

Description was changed from

==========
Keep top controls visible if SHOW is called right after HIDE.

Currently, we early exit if the visibility amount matches our
desired ending amount.  But when you set HIDDEN as the current,
it will start an animation, and a immediate call to SHOWN will
see that the ending condition is met and return, but does not
clear the animation.

Now, clear the animation if the current value matches our
desired ending value to ensure it doesn't move.

BUG=639702
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
Keep top controls visible if SHOW is called right after HIDE.

Currently, we early exit if the visibility amount matches our
desired ending amount.  But when you set HIDDEN as the current,
it will start an animation, and a immediate call to SHOWN will
see that the ending condition is met and return, but does not
clear the animation.

Now, clear the animation if the current value matches our
desired ending value to ensure it doesn't move.

BUG=639702
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

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

[commit-bot: I haz the power, 2016-09-27 06:14:58]

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