Chromium Code Reviews| Index: chrome/android/java/src/org/chromium/chrome/browser/password_manager/AccountChooserDialog.java |
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/password_manager/AccountChooserDialog.java b/chrome/android/java/src/org/chromium/chrome/browser/password_manager/AccountChooserDialog.java |
| index b8cb261ec40d9bb82157e9607faca3ca77a563ad..7654ac9997d18f131a8d20e185ef3e6004fb7893 100644 |
| --- a/chrome/android/java/src/org/chromium/chrome/browser/password_manager/AccountChooserDialog.java |
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/password_manager/AccountChooserDialog.java |
| @@ -46,6 +46,7 @@ public class AccountChooserDialog |
| private final String mTitle; |
| private final int mTitleLinkStart; |
| private final int mTitleLinkEnd; |
| + private final String mOrigin; |
| private ArrayAdapter<Credential> mAdapter; |
| /** |
| @@ -56,7 +57,8 @@ public class AccountChooserDialog |
| private AlertDialog mDialog; |
| private AccountChooserDialog(Context context, long nativeAccountChooserDialog, |
| - Credential[] credentials, String title, int titleLinkStart, int titleLinkEnd) { |
| + Credential[] credentials, String title, int titleLinkStart, int titleLinkEnd, |
| + String origin) { |
| mNativeAccountChooserDialog = nativeAccountChooserDialog; |
| mContext = context; |
| mCredentials = credentials.clone(); |
| @@ -64,6 +66,7 @@ public class AccountChooserDialog |
| mTitle = title; |
| mTitleLinkStart = titleLinkStart; |
| mTitleLinkEnd = titleLinkEnd; |
| + mOrigin = origin; |
| } |
| /** |
| @@ -72,15 +75,17 @@ public class AccountChooserDialog |
| * @param title Title message for the dialog, which can contain Smart Lock branding. |
| * @param titleLinkStart Start of a link in case title contains Smart Lock branding. |
| * @param titleLinkEnd End of a link in case title contains Smart Lock branding. |
| + * @param origin Address of the web page, where dialog was triggered. |
| */ |
| @CalledByNative |
| private static AccountChooserDialog createAccountChooser(WindowAndroid windowAndroid, |
| long nativeAccountChooserDialog, Credential[] credentials, String title, |
| - int titleLinkStart, int titleLinkEnd) { |
| + int titleLinkStart, int titleLinkEnd, String origin) { |
| Activity activity = windowAndroid.getActivity().get(); |
| if (activity == null) return null; |
| - AccountChooserDialog chooser = new AccountChooserDialog(activity, |
| - nativeAccountChooserDialog, credentials, title, titleLinkStart, titleLinkEnd); |
| + AccountChooserDialog chooser = |
| + new AccountChooserDialog(activity, nativeAccountChooserDialog, credentials, title, |
| + titleLinkStart, titleLinkEnd, origin); |
| chooser.show(activity.getFragmentManager(), null); |
| return chooser; |
| } |
| @@ -129,6 +134,8 @@ public class AccountChooserDialog |
| public Dialog onCreateDialog(Bundle savedInstanceState) { |
| View titleView = |
| LayoutInflater.from(mContext).inflate(R.layout.account_chooser_dialog_title, null); |
| + TextView origin = (TextView) titleView.findViewById(R.id.origin); |
| + origin.setText(mOrigin); |
| TextView titleMessageText = (TextView) titleView.findViewById(R.id.title); |
| // TODO(melandory): add support for showing site origin in the title. |
|
gone
2016/01/25 18:36:28
Is your TODO still relevant?
melandory
2016/01/26 09:46:19
Done.
|
| if (mTitleLinkStart != 0 && mTitleLinkEnd != 0) { |