In find_unmapped_histograms.py, remove comments from histogram names

In find_unmapped_histograms.py, remove comments from histogram names

While scanning the source code for histogram matches, remove C++-style
and C-style comments from histogram names. This reduces the number of
false-negatives that the script produces.

Before / after diff:

  WARNING: components/data_reduction_proxy/core/browser/data_reduction_proxy_compression_stats.cc contains non-literal histogram name <uma>
  WARNING: cc/raster/one_copy_raster_buffer_provider.cc contains non-literal histogram name <base::StringPrintf("Renderer4.%s.PartialRasterPercentageSaved.OneCopy">
- WARNING: content/browser/renderer_host/p2p/socket_host_udp.cc contains non-literal histogram name <"WebRTC.SystemSendPacketDuration_UDP" /* name */>
- WARNING: ui/events/blink/input_handler_proxy.cc contains non-literal histogram name <// So named for historical reasons.
-             "Event.Latency.RendererImpl.GestureScroll2">
  WARNING: chrome/browser/browser_shutdown.cc contains non-literal histogram name <time.c_str(>
  WARNING: chrome/browser/browser_shutdown.cc contains non-literal histogram name <time_per.c_str(>
--- 20,23 ----

R=isherman@chromium.org
BUG=635726
Committed: https://crrev.com/986ac05bfafd2f0016f282473324011326b52947
Cr-Commit-Position: refs/heads/master@{#411567}

[Ilya Sherman, 2016-08-10 06:19:04]

LGTM, thanks =)

[Ilya Sherman, 2016-08-10 06:19:09]

The CQ bit was checked by isherman@chromium.org

[Ilya Sherman, 2016-08-10 06:19:12]

The CQ bit was unchecked by isherman@chromium.org

[Adam Rice, 2016-08-12 06:35:08]

The CQ bit was checked by ricea@chromium.org

[Adam Rice, 2016-08-12 06:35:09]

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

[commit-bot: I haz the power, 2016-08-12 06:35:25]

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-08-12 07:15:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-12 07:18:50]

Description was changed from

==========
In find_unmapped_histograms.py, remove comments from histogram names

While scanning the source code for histogram matches, remove C++-style
and C-style comments from histogram names. This reduces the number of
false-negatives that the script produces.

Before / after diff:

  WARNING:
components/data_reduction_proxy/core/browser/data_reduction_proxy_compression_stats.cc
contains non-literal histogram name <uma>
  WARNING: cc/raster/one_copy_raster_buffer_provider.cc contains non-literal
histogram name
<base::StringPrintf("Renderer4.%s.PartialRasterPercentageSaved.OneCopy">
- WARNING: content/browser/renderer_host/p2p/socket_host_udp.cc contains
non-literal histogram name <"WebRTC.SystemSendPacketDuration_UDP" /* name */>
- WARNING: ui/events/blink/input_handler_proxy.cc contains non-literal histogram
name <// So named for historical reasons.
-             "Event.Latency.RendererImpl.GestureScroll2">
  WARNING: chrome/browser/browser_shutdown.cc contains non-literal histogram
name <time.c_str(>
  WARNING: chrome/browser/browser_shutdown.cc contains non-literal histogram
name <time_per.c_str(>
--- 20,23 ----


R=isherman@chromium.org
BUG=635726
==========

to

==========
In find_unmapped_histograms.py, remove comments from histogram names

While scanning the source code for histogram matches, remove C++-style
and C-style comments from histogram names. This reduces the number of
false-negatives that the script produces.

Before / after diff:

  WARNING:
components/data_reduction_proxy/core/browser/data_reduction_proxy_compression_stats.cc
contains non-literal histogram name <uma>
  WARNING: cc/raster/one_copy_raster_buffer_provider.cc contains non-literal
histogram name
<base::StringPrintf("Renderer4.%s.PartialRasterPercentageSaved.OneCopy">
- WARNING: content/browser/renderer_host/p2p/socket_host_udp.cc contains
non-literal histogram name <"WebRTC.SystemSendPacketDuration_UDP" /* name */>
- WARNING: ui/events/blink/input_handler_proxy.cc contains non-literal histogram
name <// So named for historical reasons.
-             "Event.Latency.RendererImpl.GestureScroll2">
  WARNING: chrome/browser/browser_shutdown.cc contains non-literal histogram
name <time.c_str(>
  WARNING: chrome/browser/browser_shutdown.cc contains non-literal histogram
name <time_per.c_str(>
--- 20,23 ----

R=isherman@chromium.org
BUG=635726

Committed: https://crrev.com/986ac05bfafd2f0016f282473324011326b52947
Cr-Commit-Position: refs/heads/master@{#411567}
==========

[commit-bot: I haz the power, 2016-08-12 07:18:51]

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