Save federated credential's signon_realm in the correct format.

Save federated credential's signon_realm in the correct format.

It should be "federation://<origin>/<federation provider>". After https://codereview.chromium.org/1946563003 it became just "<origin>" like for normal credentials. Therefore, password and federated credentials started to overwrite each other in the password storage.

BUG=611035
Committed: https://crrev.com/032bf70d261ff99daeae51e28583fd23e2458415
Cr-Commit-Position: refs/heads/master@{#392994}

[vasilii, 2016-05-11 17:01:18]

vasilii@chromium.org changed reviewers:
+ cfroussios@chromium.org

[vasilii, 2016-05-11 17:01:18]

Hi Christos,

please have a look.

[vasilii, 2016-05-11 17:12:10]

Description was changed from

==========
Serialize federated credential's signon_realm in the correct format.

It should be "federation://<origin>/<federation provider>".

BUG=611035
==========

to

==========
Save federated credential's signon_realm in the correct format.

It should be "federation://<origin>/<federation provider>". After
https://codereview.chromium.org/1946563003 it became just "<origin>" like for
normal credentials. Therefore, password and federated credentials started to
overwrite each other in the password storage.

BUG=611035
==========

[cfroussios, 2016-05-11 17:21:19]

On 2016/05/11 17:01:18, vasilii wrote:
> Hi Christos,
> 
> please have a look.

lgtm

[vasilii, 2016-05-11 17:23:53]

The CQ bit was checked by vasilii@chromium.org

[vasilii, 2016-05-11 17:23:54]

The patchset sent to the CQ was uploaded after l-g-t-m from
cfroussios@chromium.org
Link to the patchset: https://codereview.chromium.org/1965413003/#ps20001
(title: "fix more tests")

[commit-bot: I haz the power, 2016-05-11 17:24:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1965413003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1965413003/20001

[commit-bot: I haz the power, 2016-05-11 18:16:12]

Description was changed from

==========
Save federated credential's signon_realm in the correct format.

It should be "federation://<origin>/<federation provider>". After
https://codereview.chromium.org/1946563003 it became just "<origin>" like for
normal credentials. Therefore, password and federated credentials started to
overwrite each other in the password storage.

BUG=611035
==========

to

==========
Save federated credential's signon_realm in the correct format.

It should be "federation://<origin>/<federation provider>". After
https://codereview.chromium.org/1946563003 it became just "<origin>" like for
normal credentials. Therefore, password and federated credentials started to
overwrite each other in the password storage.

BUG=611035
==========

[commit-bot: I haz the power, 2016-05-11 18:16:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-11 18:18:23]

Description was changed from

==========
Save federated credential's signon_realm in the correct format.

It should be "federation://<origin>/<federation provider>". After
https://codereview.chromium.org/1946563003 it became just "<origin>" like for
normal credentials. Therefore, password and federated credentials started to
overwrite each other in the password storage.

BUG=611035
==========

to

==========
Save federated credential's signon_realm in the correct format.

It should be "federation://<origin>/<federation provider>". After
https://codereview.chromium.org/1946563003 it became just "<origin>" like for
normal credentials. Therefore, password and federated credentials started to
overwrite each other in the password storage.

BUG=611035

Committed: https://crrev.com/032bf70d261ff99daeae51e28583fd23e2458415
Cr-Commit-Position: refs/heads/master@{#392994}
==========

[commit-bot: I haz the power, 2016-05-11 18:18:25]

Patchset 2 (id:??) landed as
https://crrev.com/032bf70d261ff99daeae51e28583fd23e2458415
Cr-Commit-Position: refs/heads/master@{#392994}