Commit d80f5035 authored by Jeffrey Cohen's avatar Jeffrey Cohen Committed by Commit Bot

[ShareButtonInToolbar] add explicit end animation state to explicit show

The share button was getting stuck in a intermediate step where it was
being marked as VISIBLE, but with an alpha of zero and an offset.
This makes explicit the assumption that setting the button to visible
will find it in its default state.

Bug: 1084488
Change-Id: I2ac9797cfdc254c255e31d41bc82a96603bb3f93
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218707Reviewed-by: default avatarPatrick Noland <pnoland@chromium.org>
Reviewed-by: default avatarMatthew Jones <mdjones@chromium.org>
Commit-Queue: Jeffrey Cohen <jeffreycohen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#773252}
parent 07683471
......@@ -2600,6 +2600,7 @@ public class ToolbarPhone extends ToolbarLayout implements Invalidator.Client, O
runShowOptionalButtonAnimation();
} else {
mOptionalButton.setVisibility(View.VISIBLE);
onOptionalButtonAnimationEnd();
}
} else {
mOptionalButton.setVisibility(View.VISIBLE);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment