Tweak CryptauthDeviceManager to store fetched BeaconSeed data in Base64URL encoding, and retrieve i…

Tweak CryptauthDeviceManager to store fetched BeaconSeed data in Base64URL encoding, and retrieve it as raw data. This is necessary because CryptauthDeviceManager assumed that BeacondSeed data from the server was Base64 encoded, when in reality it was raw.

BUG=678145
Committed: https://crrev.com/ac31f879c80a4bbb2a1b089da4ea073d87d5060b
Cr-Commit-Position: refs/heads/master@{#441450}

[Ryan Hansberry, 2017-01-04 02:14:11]

Description was changed from

==========
Tweak CryptauthDeviceManager to store fetched BeaconSeed data in Base64URL
encoding, and retrieve it as raw data. This is necessary because
CryptauthDeviceManager assumed that BeacondSeed data from the server was Base64
encoded, when in reality it was raw.

BUG=
==========

to

==========
Tweak CryptauthDeviceManager to store fetched BeaconSeed data in Base64URL
encoding, and retrieve it as raw data. This is necessary because
CryptauthDeviceManager assumed that BeacondSeed data from the server was Base64
encoded, when in reality it was raw.

BUG=678145
==========

[Ryan Hansberry, 2017-01-04 02:14:11]

hansberry@chromium.org changed reviewers:
+ khorimoto@chromium.org

[Kyle Horimoto, 2017-01-04 18:42:29]

lgtm

https://codereview.chromium.org/2611913002/diff/1/components/cryptauth/crypta...
File components/cryptauth/cryptauth_device_manager.cc (right):

https://codereview.chromium.org/2611913002/diff/1/components/cryptauth/crypta...
components/cryptauth/cryptauth_device_manager.cc:71: // Note that the beacon
seeds' data is stored in Base64Url form because
nit: |BeaconSeed|s'

[Ryan Hansberry, 2017-01-04 18:56:30]

The CQ bit was checked by hansberry@chromium.org

[Ryan Hansberry, 2017-01-04 18:56:31]

The patchset sent to the CQ was uploaded after l-g-t-m from
khorimoto@chromium.org
Link to the patchset: https://codereview.chromium.org/2611913002/#ps20001
(title: "Fixing a comment.")

[commit-bot: I haz the power, 2017-01-04 18:56:48]

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, 2017-01-04 20:16:07]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483556190870540,
"parent_rev": "a1604b5a53bc15dbf90f322ace7dead54564ef1e", "commit_rev":
"f7f4f7b4de3d381db88f7b2ace190caea45cd638"}

[commit-bot: I haz the power, 2017-01-04 20:16:32]

Description was changed from

==========
Tweak CryptauthDeviceManager to store fetched BeaconSeed data in Base64URL
encoding, and retrieve it as raw data. This is necessary because
CryptauthDeviceManager assumed that BeacondSeed data from the server was Base64
encoded, when in reality it was raw.

BUG=678145
==========

to

==========
Tweak CryptauthDeviceManager to store fetched BeaconSeed data in Base64URL
encoding, and retrieve it as raw data. This is necessary because
CryptauthDeviceManager assumed that BeacondSeed data from the server was Base64
encoded, when in reality it was raw.

BUG=678145

Review-Url: https://codereview.chromium.org/2611913002
==========

[commit-bot: I haz the power, 2017-01-04 20:16:33]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2017-01-04 20:19:34]

Description was changed from

==========
Tweak CryptauthDeviceManager to store fetched BeaconSeed data in Base64URL
encoding, and retrieve it as raw data. This is necessary because
CryptauthDeviceManager assumed that BeacondSeed data from the server was Base64
encoded, when in reality it was raw.

BUG=678145

Review-Url: https://codereview.chromium.org/2611913002
==========

to

==========
Tweak CryptauthDeviceManager to store fetched BeaconSeed data in Base64URL
encoding, and retrieve it as raw data. This is necessary because
CryptauthDeviceManager assumed that BeacondSeed data from the server was Base64
encoded, when in reality it was raw.

BUG=678145

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

[commit-bot: I haz the power, 2017-01-04 20:19:36]

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