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

commit 664a4cfcc194ee6b0f20069db9bfd116cfe5b32a
Author: mathp <mathp@chromium.org>
Date: Tue Jul 19 22:07:07 2016

[Autofill] Add support for signin promo on Android

Will open the AccountSigninActivity.

BUG= 626383 
TEST=manual

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

[modify] https://crrev.com/664a4cfcc194ee6b0f20069db9bfd116cfe5b32a/chrome/android/java/src/org/chromium/chrome/browser/signin/AccountSigninActivity.java
[modify] https://crrev.com/664a4cfcc194ee6b0f20069db9bfd116cfe5b32a/chrome/android/java/src/org/chromium/chrome/browser/signin/SigninPromoUtil.java
[modify] https://crrev.com/664a4cfcc194ee6b0f20069db9bfd116cfe5b32a/chrome/browser/android/chrome_jni_registrar.cc
[add] https://crrev.com/664a4cfcc194ee6b0f20069db9bfd116cfe5b32a/chrome/browser/android/signin/signin_promo_util_android.cc
[add] https://crrev.com/664a4cfcc194ee6b0f20069db9bfd116cfe5b32a/chrome/browser/android/signin/signin_promo_util_android.h
[modify] https://crrev.com/664a4cfcc194ee6b0f20069db9bfd116cfe5b32a/chrome/browser/signin/signin_promo.cc
[modify] https://crrev.com/664a4cfcc194ee6b0f20069db9bfd116cfe5b32a/chrome/browser/signin/signin_promo.h
[add] https://crrev.com/664a4cfcc194ee6b0f20069db9bfd116cfe5b32a/chrome/browser/signin/signin_promo_util.cc
[add] https://crrev.com/664a4cfcc194ee6b0f20069db9bfd116cfe5b32a/chrome/browser/signin/signin_promo_util.h
[modify] https://crrev.com/664a4cfcc194ee6b0f20069db9bfd116cfe5b32a/chrome/browser/ui/android/autofill/autofill_popup_view_android.cc
[modify] https://crrev.com/664a4cfcc194ee6b0f20069db9bfd116cfe5b32a/chrome/browser/ui/autofill/chrome_autofill_client.cc
[modify] https://crrev.com/664a4cfcc194ee6b0f20069db9bfd116cfe5b32a/chrome/browser/ui/sync/sync_promo_ui.cc
[modify] https://crrev.com/664a4cfcc194ee6b0f20069db9bfd116cfe5b32a/chrome/browser/ui/webui/app_launcher_login_handler.cc
[modify] https://crrev.com/664a4cfcc194ee6b0f20069db9bfd116cfe5b32a/chrome/chrome_browser.gypi

For iOS, here's the current chip system. What should the UI be for the signin promo?

Deleted: Screen Shot 2016-07-21 at 10.58.47 AM.png 103 KB

	Screen Shot 2016-07-21 at 10.58.47 AM.png 103 KB View Download

Hm, the current proposed string is too long to fit into a chip. I think we have two options: 1. ignore iOS for now 2. shorten the string to something that just says, "Sign in to Chrome" or similar. But I don't know how to get the value add in there

Hwi, what's your take?

I prefer option 1 (c#5) both for iOS and Clank. The current sign-in promo proposal was suited for Desktop.

But Android has the Autofill dropdown, and the platform is the highest order of priority in the PRD: https://docs.google.com/document/d/1NuuMn6Z7hxR2wHWMTiovL-K592nlclsLTjYaYODloeQ/edit

What should we do?

Leave out iOS for now.

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

commit f0f7ccc063123d306b5cc5f8c48fccff3547c83a
Author: mathp <mathp@chromium.org>
Date: Thu Jul 21 18:04:01 2016

[Autofill] Remove ifdef for iOS

iOS Autofill uses the different AutofillClientIOS

BUG= 626383 

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

[modify] https://crrev.com/f0f7ccc063123d306b5cc5f8c48fccff3547c83a/chrome/browser/ui/autofill/chrome_autofill_client.cc

OK, work has completed for now on the Android promo and the non-existent iOS promo.