QUIC - Delete Net.QuicStreamFactory.TSVIPCliIsLoaded histogram. We have

QUIC - Delete Net.QuicStreamFactory.TSVIPCliIsLoaded histogram. We have
fixed the crash bug when TSVIPCli DLL is loaded. Cleaning up the code
that is not needed.

BUG=471986
R=rkaplow@chromium.org, rch@chromium.org

Committed: https://crrev.com/5a38f3b112479570bb7ad1efc560ae885ce2dfce
Cr-Commit-Position: refs/heads/master@{#400795}

[Ryan Hamilton, 2016-06-18 15:00:25]

lgtm

[ramant (doing other things), 2016-06-20 16:43:46]

Description was changed from

==========
QUIC - Delete Net.QuicStreamFactory.TSVIPCliIsLoaded histogram. We have
fixed the crash bug when TSVIPCli DLL is loaded. Cleaning up the code
that is not needed.

BUG=471986
R=asvitkine@chromium.org, rch@chromium.org
==========

to

==========
QUIC - Delete Net.QuicStreamFactory.TSVIPCliIsLoaded histogram. We have
fixed the crash bug when TSVIPCli DLL is loaded. Cleaning up the code
that is not needed.

BUG=471986
R=rkaplow@chromium.org, rch@chromium.org
==========

[ramant (doing other things), 2016-06-20 16:43:46]

rtenneti@chromium.org changed reviewers:
+ rkaplow@chromium.org
- asvitkine@chromium.org

[ramant (doing other things), 2016-06-20 16:44:27]

Hi rkaplow@,
  Can you please take a look?

thanks
raman

[rkaplow, 2016-06-20 20:17:49]

https://codereview.chromium.org/2071373002/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/2071373002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:29902: - 
<owner>rtenneti@chromium.org</owner>
can you instead mark it as obsolete

[ramant (doing other things), 2016-06-20 20:26:16]

https://codereview.chromium.org/2071373002/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/2071373002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:29902: - 
<owner>rtenneti@chromium.org</owner>
On 2016/06/20 20:17:49, rkaplow wrote:
> can you instead mark it as obsolete

Done.

[rkaplow, 2016-06-20 20:28:27]

lgtm

[ramant (doing other things), 2016-06-20 20:32:40]

The CQ bit was checked by rtenneti@chromium.org

[ramant (doing other things), 2016-06-20 20:32:41]

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

[commit-bot: I haz the power, 2016-06-20 20:33:23]

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

[commit-bot: I haz the power, 2016-06-20 21:44:12]

Description was changed from

==========
QUIC - Delete Net.QuicStreamFactory.TSVIPCliIsLoaded histogram. We have
fixed the crash bug when TSVIPCli DLL is loaded. Cleaning up the code
that is not needed.

BUG=471986
R=rkaplow@chromium.org, rch@chromium.org
==========

to

==========
QUIC - Delete Net.QuicStreamFactory.TSVIPCliIsLoaded histogram. We have
fixed the crash bug when TSVIPCli DLL is loaded. Cleaning up the code
that is not needed.

BUG=471986
R=rkaplow@chromium.org, rch@chromium.org
==========

[commit-bot: I haz the power, 2016-06-20 21:44:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-20 21:45:34]

Description was changed from

==========
QUIC - Delete Net.QuicStreamFactory.TSVIPCliIsLoaded histogram. We have
fixed the crash bug when TSVIPCli DLL is loaded. Cleaning up the code
that is not needed.

BUG=471986
R=rkaplow@chromium.org, rch@chromium.org
==========

to

==========
QUIC - Delete Net.QuicStreamFactory.TSVIPCliIsLoaded histogram. We have
fixed the crash bug when TSVIPCli DLL is loaded. Cleaning up the code
that is not needed.

BUG=471986
R=rkaplow@chromium.org, rch@chromium.org

Committed: https://crrev.com/5a38f3b112479570bb7ad1efc560ae885ce2dfce
Cr-Commit-Position: refs/heads/master@{#400795}
==========

[commit-bot: I haz the power, 2016-06-20 21:45:35]

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