Reset translation of toolbar_buttons for Url Defocus Animation.

Reset translation of toolbar_buttons for Url Defocus Animation.

The mToolbarButtonsContainer has its translationY modified during scroll so
when the user scrolls on the NTP, it appears to scroll too. However during
the URL focus and defocus animations it should not be touched. Unfortunately
updateNtpTransitionAnimation() is called a few times after the URL focus
animation has been completed while mUrlFocusChangeInProgress is set to false,
causing translationY to be non-zero at the end.  We reset the translationY
here so the mToolbarButtonsContainer is on screen for the defocusing animation.

BUG=623885
Committed: https://crrev.com/56e3c638ef1575429a2ecd5a19ddd069ba42892b
Cr-Commit-Position: refs/heads/master@{#404113}

[PEConn, 2016-07-06 09:34:25]

Description was changed from

==========
Reset translation of toolbar_buttons for Url Defocus Animation.

The mToolbarButtonsContainer has its translationY modified during scroll so when
the user scrolls on the NTP, it appears to scroll too. However during the URL
focus and defocus animations it should not be touched. Unfortunately
updateNtpTransitionAnimation() is called a few times after the URL focus
animation has been completed while mUrlFocusChangeInProgress is set to false,
causing translationY to be non-zero at the end.  We reset the translationY here
so the mToolbarButtonsContainer is on screen for the defocusing animation.

BUG=623885
==========

to

==========
Reset translation of toolbar_buttons for Url Defocus Animation.

The mToolbarButtonsContainer has its translationY modified during scroll so 
when the user scrolls on the NTP, it appears to scroll too. However during 
the URL focus and defocus animations it should not be touched. Unfortunately 
updateNtpTransitionAnimation() is called a few times after the URL focus 
animation has been completed while mUrlFocusChangeInProgress is set to false,
causing translationY to be non-zero at the end.  We reset the translationY 
here so the mToolbarButtonsContainer is on screen for the defocusing animation.

BUG=623885
==========

[PEConn, 2016-07-06 09:36:36]

peconn@chromium.org changed reviewers:
+ bauerb@chromium.org, yusufo@chromium.org

[PEConn, 2016-07-06 09:36:36]

PTAL, the linked bug has before and after videos of the change.

[Bernhard Bauer, 2016-07-06 10:08:24]

LGTM, thanks!

[PEConn, 2016-07-06 11:06:09]

The CQ bit was checked by peconn@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-06 11:06:20]

Dry run: 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-07-06 11:49:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-06 11:49:37]

Dry run: This issue passed the CQ dry run.

[PEConn, 2016-07-07 09:21:45]

The CQ bit was checked by peconn@chromium.org

[commit-bot: I haz the power, 2016-07-07 09:22:12]

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-07-07 09:25:41]

Description was changed from

==========
Reset translation of toolbar_buttons for Url Defocus Animation.

The mToolbarButtonsContainer has its translationY modified during scroll so 
when the user scrolls on the NTP, it appears to scroll too. However during 
the URL focus and defocus animations it should not be touched. Unfortunately 
updateNtpTransitionAnimation() is called a few times after the URL focus 
animation has been completed while mUrlFocusChangeInProgress is set to false,
causing translationY to be non-zero at the end.  We reset the translationY 
here so the mToolbarButtonsContainer is on screen for the defocusing animation.

BUG=623885
==========

to

==========
Reset translation of toolbar_buttons for Url Defocus Animation.

The mToolbarButtonsContainer has its translationY modified during scroll so 
when the user scrolls on the NTP, it appears to scroll too. However during 
the URL focus and defocus animations it should not be touched. Unfortunately 
updateNtpTransitionAnimation() is called a few times after the URL focus 
animation has been completed while mUrlFocusChangeInProgress is set to false,
causing translationY to be non-zero at the end.  We reset the translationY 
here so the mToolbarButtonsContainer is on screen for the defocusing animation.

BUG=623885
==========

[commit-bot: I haz the power, 2016-07-07 09:25:42]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-07 09:25:45]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-07 09:27:13]

Description was changed from

==========
Reset translation of toolbar_buttons for Url Defocus Animation.

The mToolbarButtonsContainer has its translationY modified during scroll so 
when the user scrolls on the NTP, it appears to scroll too. However during 
the URL focus and defocus animations it should not be touched. Unfortunately 
updateNtpTransitionAnimation() is called a few times after the URL focus 
animation has been completed while mUrlFocusChangeInProgress is set to false,
causing translationY to be non-zero at the end.  We reset the translationY 
here so the mToolbarButtonsContainer is on screen for the defocusing animation.

BUG=623885
==========

to

==========
Reset translation of toolbar_buttons for Url Defocus Animation.

The mToolbarButtonsContainer has its translationY modified during scroll so
when the user scrolls on the NTP, it appears to scroll too. However during
the URL focus and defocus animations it should not be touched. Unfortunately
updateNtpTransitionAnimation() is called a few times after the URL focus
animation has been completed while mUrlFocusChangeInProgress is set to false,
causing translationY to be non-zero at the end.  We reset the translationY
here so the mToolbarButtonsContainer is on screen for the defocusing animation.

BUG=623885

Committed: https://crrev.com/56e3c638ef1575429a2ecd5a19ddd069ba42892b
Cr-Commit-Position: refs/heads/master@{#404113}
==========

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

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