IOSChromeUpdatePasswordInfoBarDelegate for update password UI

IOSChromeUpdatePasswordInfoBarDelegate for update password UI

Adds IOSChromeUpdatePasswordInfoBarDelegate to serve as
infobar delegate for the update password infobar. This delegate
provides account data to the controller, indicating which account
to display and whether there are multiple accounts possible
for the pending password update.

The controller created in Create will be changed to a custom
controller for update passwords when the controller is available.

BUG=622244
Committed: https://crrev.com/aad66ecb0cfcee57820f640e859f96d8fe198125
Cr-Commit-Position: refs/heads/master@{#403762}

[Jackie Quinn, 2016-06-30 13:05:53]

Description was changed from

==========
Add infobar controller and delegate for iOS update passwords UI.

BUG=622244
==========

to

==========
IOSChromeUpdatePasswordInfoBarDelegate for update password UI

Adds IOSChromeUpdatePasswordInfoBarDelegate to serve as
infobar delegate for the update password infobar. This delegate
provides account data to the controller, indicating which account
to display and whether there are multiple accounts possible
for the pending password update.

The controller created in Create will be changed to a custom
controller for update passwords when the controller is available. 

BUG=622244
==========

[Jackie Quinn, 2016-06-30 13:05:53]

jyquinn@chromium.org changed reviewers:
+ dvadym@chromium.org, vabr@chromium.org

[vabr (Chromium), 2016-06-30 13:25:13]

LGTM, just nits.
Thanks!
Vaclav

https://codereview.chromium.org/2108643002/diff/20001/ios/chrome/browser/pass...
File ios/chrome/browser/passwords/ios_chrome_update_password_infobar_delegate.mm
(right):

https://codereview.chromium.org/2108643002/diff/20001/ios/chrome/browser/pass...
ios/chrome/browser/passwords/ios_chrome_update_password_infobar_delegate.mm:1:
// Copyright 2015 The Chromium Authors. All rights reserved.
2015 -> 2016

https://codereview.chromium.org/2108643002/diff/20001/ios/chrome/browser/pass...
ios/chrome/browser/passwords/ios_chrome_update_password_infobar_delegate.mm:35:
// Replace ConfirmInfoBarController with update password controller class.
nit: If this is a TODO, then please prefix it with TODO(crbug.com/622244)

[Jackie Quinn, 2016-06-30 13:54:32]

https://codereview.chromium.org/2108643002/diff/20001/ios/chrome/browser/pass...
File ios/chrome/browser/passwords/ios_chrome_update_password_infobar_delegate.mm
(right):

https://codereview.chromium.org/2108643002/diff/20001/ios/chrome/browser/pass...
ios/chrome/browser/passwords/ios_chrome_update_password_infobar_delegate.mm:1:
// Copyright 2015 The Chromium Authors. All rights reserved.
On 2016/06/30 13:25:13, vabr (Chromium) wrote:
> 2015 -> 2016

Done.

https://codereview.chromium.org/2108643002/diff/20001/ios/chrome/browser/pass...
ios/chrome/browser/passwords/ios_chrome_update_password_infobar_delegate.mm:35:
// Replace ConfirmInfoBarController with update password controller class.
On 2016/06/30 13:25:13, vabr (Chromium) wrote:
> nit: If this is a TODO, then please prefix it with TODO(crbug.com/622244)

Done.

[vabr (Chromium), 2016-06-30 14:29:04]

lgtm

[dvadym, 2016-06-30 14:38:42]

lgtm, as far as I see the logic here is the same as for other platforms.

[Jackie Quinn, 2016-07-05 09:11:32]

The CQ bit was checked by jyquinn@chromium.org

[Jackie Quinn, 2016-07-05 09:11:33]

The patchset sent to the CQ was uploaded after l-g-t-m from vabr@chromium.org,
dvadym@chromium.org
Link to the patchset: https://codereview.chromium.org/2108643002/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2016-07-05 09:11:44]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-05 10:20:45]

Description was changed from

==========
IOSChromeUpdatePasswordInfoBarDelegate for update password UI

Adds IOSChromeUpdatePasswordInfoBarDelegate to serve as
infobar delegate for the update password infobar. This delegate
provides account data to the controller, indicating which account
to display and whether there are multiple accounts possible
for the pending password update.

The controller created in Create will be changed to a custom
controller for update passwords when the controller is available. 

BUG=622244
==========

to

==========
IOSChromeUpdatePasswordInfoBarDelegate for update password UI

Adds IOSChromeUpdatePasswordInfoBarDelegate to serve as
infobar delegate for the update password infobar. This delegate
provides account data to the controller, indicating which account
to display and whether there are multiple accounts possible
for the pending password update.

The controller created in Create will be changed to a custom
controller for update passwords when the controller is available. 

BUG=622244
==========

[commit-bot: I haz the power, 2016-07-05 10:20:46]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-07-05 10:22:19]

Description was changed from

==========
IOSChromeUpdatePasswordInfoBarDelegate for update password UI

Adds IOSChromeUpdatePasswordInfoBarDelegate to serve as
infobar delegate for the update password infobar. This delegate
provides account data to the controller, indicating which account
to display and whether there are multiple accounts possible
for the pending password update.

The controller created in Create will be changed to a custom
controller for update passwords when the controller is available. 

BUG=622244
==========

to

==========
IOSChromeUpdatePasswordInfoBarDelegate for update password UI

Adds IOSChromeUpdatePasswordInfoBarDelegate to serve as
infobar delegate for the update password infobar. This delegate
provides account data to the controller, indicating which account
to display and whether there are multiple accounts possible
for the pending password update.

The controller created in Create will be changed to a custom
controller for update passwords when the controller is available.

BUG=622244

Committed: https://crrev.com/aad66ecb0cfcee57820f640e859f96d8fe198125
Cr-Commit-Position: refs/heads/master@{#403762}
==========

[commit-bot: I haz the power, 2016-07-05 10:22:20]

Patchset 4 (id:??) landed as
https://crrev.com/aad66ecb0cfcee57820f640e859f96d8fe198125
Cr-Commit-Position: refs/heads/master@{#403762}