The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e9194fafbe858338e049ff946d2f844c20f7c944

commit e9194fafbe858338e049ff946d2f844c20f7c944
Author: ftang <ftang@chromium.org>
Date: Tue May 10 05:28:50 2016

Call SetTranslateEnable(true) in the 2016Q2 UI even it is blocked language or blocked URL so Omnibar will display the translate icon

BUG= 610376 

Review-Url: https://codereview.chromium.org/1966443003
Cr-Commit-Position: refs/heads/master@{#392550}

[modify] https://crrev.com/e9194fafbe858338e049ff946d2f844c20f7c944/components/translate/core/browser/translate_manager.cc

verified in "Version 52.0.2735.0 canary (64-bit)"

juliecattiau@google.com said UX team want us to rollback this design.

Julie- could you confirm in this bug you want this design to be rollback.

I confirm that we want to roll this change back.

Rationale: after users chose to "Never translate" a language or a page, the state in which users are needs to be reflected in the Translate bubble UI. The next time users open the bubble after choosing to "Never translate" by clicking on the omnibox icon, we would ideally show a simplified view of the bubble that informs users they are in a "Never translate" state and gives them the option to undo. For example: "You're not translating German. [Undo] / [Translate Once]". Since this is not something we can easily implement short term, we have decided to rollback the change described in this bug and to not keep the translate icon present in the omnibox after taking a "Never Translate" action for now.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/898979d8cb1874e230621ebf0e4dd0b8208c0894

commit 898979d8cb1874e230621ebf0e4dd0b8208c0894
Author: ftang <ftang@chromium.org>
Date: Wed Jul 13 19:47:52 2016

remove the omnibar icon changes based on UX feedback

BUG= 610376 

Review-Url: https://codereview.chromium.org/2130623002
Cr-Commit-Position: refs/heads/master@{#405235}

[modify] https://crrev.com/898979d8cb1874e230621ebf0e4dd0b8208c0894/components/translate/core/browser/translate_manager.cc
[modify] https://crrev.com/898979d8cb1874e230621ebf0e4dd0b8208c0894/components/translate/core/browser/translate_ui_delegate.cc
[modify] https://crrev.com/898979d8cb1874e230621ebf0e4dd0b8208c0894/components/translate/core/browser/translate_ui_delegate_unittest.cc

Your change meets the bar and is auto-approved for M53 (branch: 2785)

Please merge your change to M53 branch 2785 ASAP (latest by 4:00 PM PST on Monday, 07/18) in order to make it to M53 dev release next week before Beta promotion.

groby- could you merge it into M53 for me?

This is already merged to M53 branch 2785 - https://chromium.googlesource.com/chromium/src.git/+/81fde70ae27332255d909c9e77493671cccdf0e5. So applying "merge-merged-2785" label and removing "Merge-Approved-53" label. Please correct me if I'm missing anything here.

Thank you groby@ for M53 merges.

Verified the merge on the latest M-53(53.0.2785.21) on Windows-7 and Linux Ubuntu 14.04 as per the manual steps in C#0. This is working as intended. Attached is the screen-cast of the same.

Deleted: 610376.mp4 1.8 MB

	610376.mp4 1.8 MB View Download