Some cleanup of the histogram guide. Mainly formatting fixes.

Some cleanup of the histogram guide. Mainly formatting fixes.
Can see current guide at:
https://chromium.googlesource.com/chromium/src.git/+/HEAD/tools/metrics/histograms/README.md#Revising-Histograms

Largest change is removal of the guide on exact linear histogram, I think it is less important to give guidance now there is an explicit macro.

BUG=

Committed: https://crrev.com/6dfcb893913ccd2bd6cf4f9e07a4e156aaa6cb27
Cr-Commit-Position: refs/heads/master@{#422779}

[rkaplow, 2016-10-03 21:54:04]

Description was changed from

==========
Some cleanup of the histogram guide. Mainly formatting fixes.
Can see current guide at:
https://chromium.googlesource.com/chromium/src.git/+/HEAD/tools/metrics/histo...

Largest change is removal of the guide on exact linear histogram, I think it is
less important to give guidance now there is an explicit macro.


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

to

==========
Some cleanup of the histogram guide. Mainly formatting fixes.
Can see current guide at:
https://chromium.googlesource.com/chromium/src.git/+/HEAD/tools/metrics/histo...

Largest change is removal of the guide on exact linear histogram, I think it is
less important to give guidance now there is an explicit macro.


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

[rkaplow, 2016-10-03 21:54:04]

rkaplow@chromium.org changed reviewers:
+ mpearson@chromium.org

[Mark P, 2016-10-03 22:00:48]

lgtm with comments

Thanks for doing this.

--mark

https://codereview.chromium.org/2389813003/diff/1/tools/metrics/histograms/RE...
File tools/metrics/histograms/README.md (right):

https://codereview.chromium.org/2389813003/diff/1/tools/metrics/histograms/RE...
tools/metrics/histograms/README.md:88: for larger numbers.  The macros default
to 50 buckets which is appropriate for
some default to 100; that's why I just wishy-washy words.  Not sure if this is
important.

https://codereview.chromium.org/2389813003/diff/1/tools/metrics/histograms/RE...
tools/metrics/histograms/README.md:97: Test your histograms using
*chrome://histograms*.  Make sure they're being
*chrome://histograms*
will this make a link?

[rkaplow, 2016-10-04 13:55:27]

The CQ bit was checked by rkaplow@chromium.org

[rkaplow, 2016-10-04 13:55:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
mpearson@chromium.org
Link to the patchset: https://codereview.chromium.org/2389813003/#ps40001
(title: "remove ws")

[commit-bot: I haz the power, 2016-10-04 13:55:49]

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-10-04 14:04:45]

Description was changed from

==========
Some cleanup of the histogram guide. Mainly formatting fixes.
Can see current guide at:
https://chromium.googlesource.com/chromium/src.git/+/HEAD/tools/metrics/histo...

Largest change is removal of the guide on exact linear histogram, I think it is
less important to give guidance now there is an explicit macro.


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

to

==========
Some cleanup of the histogram guide. Mainly formatting fixes.
Can see current guide at:
https://chromium.googlesource.com/chromium/src.git/+/HEAD/tools/metrics/histo...

Largest change is removal of the guide on exact linear histogram, I think it is
less important to give guidance now there is an explicit macro.


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

[commit-bot: I haz the power, 2016-10-04 14:04:46]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-04 14:07:21]

Description was changed from

==========
Some cleanup of the histogram guide. Mainly formatting fixes.
Can see current guide at:
https://chromium.googlesource.com/chromium/src.git/+/HEAD/tools/metrics/histo...

Largest change is removal of the guide on exact linear histogram, I think it is
less important to give guidance now there is an explicit macro.


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

to

==========
Some cleanup of the histogram guide. Mainly formatting fixes.
Can see current guide at:
https://chromium.googlesource.com/chromium/src.git/+/HEAD/tools/metrics/histo...

Largest change is removal of the guide on exact linear histogram, I think it is
less important to give guidance now there is an explicit macro.

BUG=

Committed: https://crrev.com/6dfcb893913ccd2bd6cf4f9e07a4e156aaa6cb27
Cr-Commit-Position: refs/heads/master@{#422779}
==========

[commit-bot: I haz the power, 2016-10-04 14:07:23]

Patchset 3 (id:??) landed as
https://crrev.com/6dfcb893913ccd2bd6cf4f9e07a4e156aaa6cb27
Cr-Commit-Position: refs/heads/master@{#422779}

[rkaplow, 2016-10-04 15:40:33]

https://codereview.chromium.org/2389813003/diff/1/tools/metrics/histograms/RE...
File tools/metrics/histograms/README.md (right):

https://codereview.chromium.org/2389813003/diff/1/tools/metrics/histograms/RE...
tools/metrics/histograms/README.md:88: for larger numbers.  The macros default
to 50 buckets which is appropriate for
On 2016/10/03 22:00:47, Mark P wrote:
> some default to 100; that's why I just wishy-washy words.  Not sure if this is
> important.

ok, just added a note.

https://codereview.chromium.org/2389813003/diff/1/tools/metrics/histograms/RE...
tools/metrics/histograms/README.md:97: Test your histograms using
*chrome://histograms*.  Make sure they're being
On 2016/10/03 22:00:47, Mark P wrote:
> *chrome://histograms*
> will this make a link?

No - this just turns it italic. I'm not sure how to do a non http link, I
couldn't find any examples, and it may not even be possible. I figured  just
italic would be good enough.

[rkaplow, 2016-10-04 19:47:44]

Description was changed from

==========
Some cleanup of the histogram guide. Mainly formatting fixes.
Can see current guide at:
https://chromium.googlesource.com/chromium/src.git/+/HEAD/tools/metrics/histo...

Largest change is removal of the guide on exact linear histogram, I think it is
less important to give guidance now there is an explicit macro.

BUG=

Committed: https://crrev.com/6dfcb893913ccd2bd6cf4f9e07a4e156aaa6cb27
Cr-Commit-Position: refs/heads/master@{#422779}
==========

to

==========
Some cleanup of the histogram guide. Mainly formatting fixes.
Can see current guide at:
https://chromium.googlesource.com/chromium/src.git/+/HEAD/tools/metrics/histo...

Largest change is removal of the guide on exact linear histogram, I think it is
less important to give guidance now there is an explicit macro.

Also now adding myself and mpearson@ as owners

BUG=

Committed: https://crrev.com/6dfcb893913ccd2bd6cf4f9e07a4e156aaa6cb27
Cr-Commit-Position: refs/heads/master@{#422779}
==========

[rkaplow, 2016-10-04 19:50:30]

Description was changed from

==========
Some cleanup of the histogram guide. Mainly formatting fixes.
Can see current guide at:
https://chromium.googlesource.com/chromium/src.git/+/HEAD/tools/metrics/histo...

Largest change is removal of the guide on exact linear histogram, I think it is
less important to give guidance now there is an explicit macro.

Also now adding myself and mpearson@ as owners

BUG=

Committed: https://crrev.com/6dfcb893913ccd2bd6cf4f9e07a4e156aaa6cb27
Cr-Commit-Position: refs/heads/master@{#422779}
==========

to

==========
Some cleanup of the histogram guide. Mainly formatting fixes.
Can see current guide at:
https://chromium.googlesource.com/chromium/src.git/+/HEAD/tools/metrics/histo...

Largest change is removal of the guide on exact linear histogram, I think it is
less important to give guidance now there is an explicit macro.

BUG=

Committed: https://crrev.com/6dfcb893913ccd2bd6cf4f9e07a4e156aaa6cb27
Cr-Commit-Position: refs/heads/master@{#422779}
==========