Remove Password Authentication code that is no longer used.

Remove Password Authentication code that is no longer used.

BUG=421254
Committed: https://crrev.com/de35ea60c964deef8fa238814e608389e9e13111
Cr-Commit-Position: refs/heads/master@{#298768}

[aurimas (slooooooooow), 2014-10-08 17:46:08]

aurimas@chromium.org changed reviewers:
+ tedchoc@chromium.org

[aurimas (slooooooooow), 2014-10-08 17:46:09]

Hey Ted,

PTAL.

Aurimas

[Ted C, 2014-10-08 18:14:16]

lgtm

[aurimas (slooooooooow), 2014-10-08 18:17:13]

aurimas@chromium.org changed reviewers:
+ isherman@chromium.org

[aurimas (slooooooooow), 2014-10-08 18:17:14]

Hey Ilya,

Please take a look at this change.

Thanks,
Aurimas

[Ilya Sherman, 2014-10-08 18:58:22]

https://chromiumcodereview.appspot.com/641753002/diff/1/chrome/browser/passwo...
File chrome/browser/password_manager/save_password_infobar_delegate.cc (left):

https://chromiumcodereview.appspot.com/641753002/diff/1/chrome/browser/passwo...
chrome/browser/password_manager/save_password_infobar_delegate.cc:80:
form_to_save_->SetUseAdditionalPasswordAuthentication(
It looks like SetUseAdditionalPasswordAuthentication() can be removed from the
PasswordFormManager class as well.  Indeed, there are a bunch more changes
linked in [ https://code.google.com/p/chromium/issues/detail?id=341492 ].  Can
more of this code be removed?  Are you planning to send separate CLs for that
work?

[aurimas (slooooooooow), 2014-10-08 21:05:21]

On 2014/10/08 18:58:22, Ilya Sherman wrote:
>
https://chromiumcodereview.appspot.com/641753002/diff/1/chrome/browser/passwo...
> File chrome/browser/password_manager/save_password_infobar_delegate.cc (left):
> 
>
https://chromiumcodereview.appspot.com/641753002/diff/1/chrome/browser/passwo...
> chrome/browser/password_manager/save_password_infobar_delegate.cc:80:
> form_to_save_->SetUseAdditionalPasswordAuthentication(
> It looks like SetUseAdditionalPasswordAuthentication() can be removed from the
> PasswordFormManager class as well.  Indeed, there are a bunch more changes
> linked in [ https://code.google.com/p/chromium/issues/detail?id=341492 ].  Can
> more of this code be removed?  Are you planning to send separate CLs for that
> work?

Yes, I am planning to make a second CL that will remove the rest of
https://code.google.com/p/chromium/issues/detail?id=341492 and 
https://codereview.chromium.org/139253004 changes.

[Ilya Sherman, 2014-10-08 21:06:04]

Okay.  This CL LGTM, then.

[aurimas (slooooooooow), 2014-10-08 21:07:50]

The CQ bit was checked by aurimas@chromium.org

[commit-bot: I haz the power, 2014-10-08 21:16:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/641753002/1

[commit-bot: I haz the power, 2014-10-09 04:38:01]

Committed patchset #1 (id:1) as 5c5627ba5e354e9dc03e4fd5caef4ee11f18e96d

[commit-bot: I haz the power, 2014-10-09 04:38:55]

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