Have the Notification Id Generator produce readable results

Have the Notification Id Generator produce readable results

Previously it included some binary values that could not always be
cleanly presented. The length of a generated Id will be ~70
characters.

BUG=

Committed: https://crrev.com/d07b5c45b92aa392fc61be91307b5153b63ac1bf
Cr-Commit-Position: refs/heads/master@{#392654}

[Peter Beverloo, 2016-05-10 15:31:42]

peter@chromium.org changed reviewers:
+ mvanouwerkerk@chromium.org

[Peter Beverloo, 2016-05-10 15:31:43]

+mvanouwerkerk, this ticks off the ID Generator bits of the larger Mojo CL

[Michael van Ouwerkerk, 2016-05-10 16:57:15]

lgtm with nit

https://codereview.chromium.org/1961943004/diff/1/content/browser/notificatio...
File content/browser/notifications/notification_id_generator.cc (right):

https://codereview.chromium.org/1961943004/diff/1/content/browser/notificatio...
content/browser/notifications/notification_id_generator.cc:81: stream <<
base::IntToString(tag.size() ? 1 : 0);
Not sure you need the ternary operators in this file. Let's see if these are
already real bool values.

[Peter Beverloo, 2016-05-10 17:03:33]

Done, thanks!

[Peter Beverloo, 2016-05-10 17:03:55]

The CQ bit was checked by peter@chromium.org

[Peter Beverloo, 2016-05-10 17:03:55]

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

[commit-bot: I haz the power, 2016-05-10 17:04:49]

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

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

Committed patchset #2 (id:20001)

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

Description was changed from

==========
Have the Notification Id Generator produce readable results

Previously it included some binary values that could not always be
cleanly presented. The length of a generated Id will be ~70
characters.

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

to

==========
Have the Notification Id Generator produce readable results

Previously it included some binary values that could not always be
cleanly presented. The length of a generated Id will be ~70
characters.

BUG=

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

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

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