Remove unnecessary comment in metrics code

Remove unnecessary comment in metrics code

BUG=637322
Committed: https://crrev.com/4582c72d65e363b1fa6be6e7842888fdfc8dfd1e
Cr-Commit-Position: refs/heads/master@{#418315}

[lawrencewu, 2016-09-13 17:32:30]

Description was changed from

==========
Remove unnecessary comment in metrics code

BUG=637322
==========

to

==========
Remove unnecessary comment in metrics code

BUG=637322
==========

[lawrencewu, 2016-09-13 17:32:30]

lawrencewu@chromium.org changed reviewers:
+ asvitkine@chromium.org, rkaplow@chromium.org

[lawrencewu, 2016-09-13 17:33:04]

The CQ bit was checked by lawrencewu@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-13 17:33:39]

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

[lawrencewu, 2016-09-13 17:35:09]

lawrencewu@chromium.org changed reviewers:
- rkaplow@chromium.org

[Alexei Svitkine (slow), 2016-09-13 17:38:36]

https://codereview.chromium.org/2332393002/diff/1/components/metrics/metrics_...
File components/metrics/metrics_state_manager.cc (right):

https://codereview.chromium.org/2332393002/diff/1/components/metrics/metrics_...
components/metrics/metrics_state_manager.cc:264: // retrieval failed. If not,
return nullptr and generate a new GUID.
"and generate a new GUID" part of the comment is confusing, because that's not
being done in this function.

I suggest expanding the comment to give more context, like ". This will result
in a new GUID being generated and backed up by the calling code FooBar()."

[lawrencewu, 2016-09-13 17:43:13]

The CQ bit was checked by lawrencewu@chromium.org to run a CQ dry run

[lawrencewu, 2016-09-13 17:43:23]

updated the patchset

[commit-bot: I haz the power, 2016-09-13 17:43:40]

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

[Alexei Svitkine (slow), 2016-09-13 17:45:05]

lgtm

[lawrencewu, 2016-09-13 17:48:39]

The CQ bit was unchecked by lawrencewu@chromium.org

[lawrencewu, 2016-09-13 17:48:42]

The CQ bit was checked by lawrencewu@chromium.org

[commit-bot: I haz the power, 2016-09-13 17:48:56]

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

[lawrencewu, 2016-09-13 17:48:56]

The CQ bit was checked by lawrencewu@chromium.org

[lawrencewu, 2016-09-13 17:48:57]

The CQ bit was unchecked by lawrencewu@chromium.org

[lawrencewu, 2016-09-13 17:49:48]

The CQ bit was checked by lawrencewu@chromium.org

[commit-bot: I haz the power, 2016-09-13 17:50:15]

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-09-13 18:42:33]

Description was changed from

==========
Remove unnecessary comment in metrics code

BUG=637322
==========

to

==========
Remove unnecessary comment in metrics code

BUG=637322
==========

[commit-bot: I haz the power, 2016-09-13 18:42:34]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-13 18:45:30]

Description was changed from

==========
Remove unnecessary comment in metrics code

BUG=637322
==========

to

==========
Remove unnecessary comment in metrics code

BUG=637322

Committed: https://crrev.com/4582c72d65e363b1fa6be6e7842888fdfc8dfd1e
Cr-Commit-Position: refs/heads/master@{#418315}
==========

[commit-bot: I haz the power, 2016-09-13 18:45:31]

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