ftang - zkoch confimed that we can merge it into M53. Can you please submit the CL and request-merge-53?

Thanks,

Barak

https://codereview.chromium.org/2167843005/ is on the way.

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

commit 149a3740a58de22cd0ebbd0fe962a9f116c2e776
Author: ftang <ftang@chromium.org>
Date: Fri Jul 22 22:09:07 2016

switch translate element loading domain

BUG= 630068 

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

[modify] https://crrev.com/149a3740a58de22cd0ebbd0fe962a9f116c2e776/components/translate/core/browser/translate_script.cc

Before we approve merge to M53, Could you please confirm whether this change is baked/verified in Canary and safe to merge?

Also is this change applicable to all OS or any specific OS?

To all OS. 

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

 ftang@, could you please merge this to M53 branch 2785 ASAP. Thank you.

Please merge your change to M53 branch 2785 before 5:00 PM PDT today (Monday) so we can pick up for last M53 Dev release tomorrow. Thank you.

groby- could you help to merge this to M53 today?

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

commit def9f527d1c220c72e0e9df7cd854eab22962705
Author: Rachel Blum <groby@google.com>
Date: Mon Jul 25 23:58:53 2016

switch translate element loading domain

BUG= 630068 

Review-Url: https://codereview.chromium.org/2167843005
Cr-Commit-Position: refs/heads/master@{#407279}
(cherry picked from commit 149a3740a58de22cd0ebbd0fe962a9f116c2e776)

Review URL: https://codereview.chromium.org/2183623002 .

Cr-Commit-Position: refs/branch-heads/2785@{#348}
Cr-Branched-From: 68623971be0cfc492a2cb0427d7f478e7b214c24-refs/heads/master@{#403382}

[modify] https://crrev.com/def9f527d1c220c72e0e9df7cd854eab22962705/components/translate/core/browser/translate_script.cc

groby@ : Could you please let us know the steps to verify the issue if this can be tested from TE end.

Very simple to verify the change is good. Just verify translate feature is still working. If the change is wrong, the feature will be totally broken. There are no way to see which exact host the chrome load the script so there are no way for TE to observe beside that.

Verified that translate bubble was seen and page got translated to English and vise versaon Windows 7, Mac and Linux with Chrome 53.0.2785.30,below are the languages which I have checked:

1. Chinese(traditional and simplified)(Sina.com)
2. Japanese(www.yahoo.co.jp) 
3. German(http://www.bild.de/wa/ll/bild-de/unangemeldet-42925516.bild.html)
4. Telugu (http://www.eenadu.net/district/main.aspx?dsname=hyderabad)
5. Hindi (http://www.jagran.com/)
6. Korean (Naver.com)
7. Arabic (http://www.aljazeera.net/portal)
8. Hebrew (http://www.haaretz.co.il/)

Steps followed :
1. Install 53.0.2785.30
2. Open Chrome download page "https://www.google.com/chrome/browser/desktop/index.html' or multiple other webpages as stated above
3. navigate to bottom of the page and change the language to see the translate bubble.
4. Select translate to English and verify that the page is getting translated.

Thanks Frank for the update and Prudhvi to verify it.
Adding respective TE-verified labels for the same.