Clean up custom tab toolbar ellisizing and animation

Clean up custom tab toolbar ellisizing and animation

This CL uses android:ellipsize to let the title bar show '...' if it is
too long.

This CL also fixes a bug that if updateSecurityIcon() is called twice
quickly, we double show the urlbar animation.

BUG=506326
Committed: https://crrev.com/444738fe2024c5df902b43d7266f1ba4d241b809
Cr-Commit-Position: refs/heads/master@{#337901}

[Ian Wen, 2015-07-08 18:37:59]

ianwen@chromium.org changed reviewers:
+ yusufo@chromium.org

[Ian Wen, 2015-07-08 18:37:59]

ptal. :) RTL patch will be uploaded right away.

[Yusuf, 2015-07-08 18:52:13]

lgtm after we add the padding back in

https://codereview.chromium.org/1226833010/diff/1/chrome/android/java/res/lay...
File chrome/android/java/res/layout/custom_tabs_toolbar.xml (left):

https://codereview.chromium.org/1226833010/diff/1/chrome/android/java/res/lay...
chrome/android/java/res/layout/custom_tabs_toolbar.xml:35:
android:paddingEnd="@dimen/toolbar_edge_padding"
we actually still want the padding at the end.

[Ian Wen, 2015-07-08 18:52:50]

https://codereview.chromium.org/1226833010/diff/1/chrome/android/java/res/lay...
File chrome/android/java/res/layout/custom_tabs_toolbar.xml (left):

https://codereview.chromium.org/1226833010/diff/1/chrome/android/java/res/lay...
chrome/android/java/res/layout/custom_tabs_toolbar.xml:35:
android:paddingEnd="@dimen/toolbar_edge_padding"
On 2015/07/08 18:52:13, Yusuf wrote:
> we actually still want the padding at the end.

Done.

[Ian Wen, 2015-07-08 18:52:55]

The CQ bit was checked by ianwen@chromium.org

[commit-bot: I haz the power, 2015-07-08 18:54:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1226833010/20001

[commit-bot: I haz the power, 2015-07-08 18:57:58]

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

[commit-bot: I haz the power, 2015-07-08 18:57:58]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

(exceeded global retry quota)

[Ian Wen, 2015-07-08 19:56:15]

The CQ bit was checked by ianwen@chromium.org

[Ian Wen, 2015-07-08 19:56:15]

The patchset sent to the CQ was uploaded after l-g-t-m from yusufo@chromium.org
Link to the patchset: https://codereview.chromium.org/1226833010/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2015-07-08 19:57:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1226833010/40001

[commit-bot: I haz the power, 2015-07-08 20:43:29]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-07-08 20:44:28]

Patchset 3 (id:??) landed as
https://crrev.com/444738fe2024c5df902b43d7266f1ba4d241b809
Cr-Commit-Position: refs/heads/master@{#337901}