Remove Test DISABLED_CongestionAvoidanceSending

Remove Test DISABLED_CongestionAvoidanceSending

Remove the test as the TODO comment in pacing_sender_test.cc by ianswett.

BUG=
R=asanka@chromium.org

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

[asanka, 2015-02-17 18:49:22]

asanka@chromium.org changed reviewers:
+ rtenneti@chromium.org
- asanka@chromium.org

[asanka, 2015-02-17 18:50:01]

asanka@chromium.org changed reviewers:
+ asanka@chromium.org

[asanka, 2015-02-17 18:50:01]

asanka -> rtenneti

Why remove? Can this test be fixed or is it not necessary?

[ramant (doing other things), 2015-02-17 18:52:53]

https://codereview.chromium.org/928913002/diff/1/net/quic/congestion_control/...
File net/quic/congestion_control/pacing_sender_test.cc (left):

https://codereview.chromium.org/928913002/diff/1/net/quic/congestion_control/...
net/quic/congestion_control/pacing_sender_test.cc:197: // TODO(ianswett): Remove
this test.
Please don't remove this code. It is shared with the internal source tree.

[asanka, 2015-02-18 17:17:38]

https://codereview.chromium.org/928913002/diff/1/net/quic/congestion_control/...
File net/quic/congestion_control/pacing_sender_test.cc (left):

https://codereview.chromium.org/928913002/diff/1/net/quic/congestion_control/...
net/quic/congestion_control/pacing_sender_test.cc:197: // TODO(ianswett): Remove
this test.
On 2015/02/17 at 18:52:53, ramant wrote:
> Please don't remove this code. It is shared with the internal source tree.

Can you summarize the current state of this test? Is there a bug to point at?

I think while we are looking at this, we should at least update the comment
since it's clearly misleading.

[ramant (doing other things), 2015-02-23 21:53:07]

lgtm

https://codereview.chromium.org/928913002/diff/1/net/quic/congestion_control/...
File net/quic/congestion_control/pacing_sender_test.cc (left):

https://codereview.chromium.org/928913002/diff/1/net/quic/congestion_control/...
net/quic/congestion_control/pacing_sender_test.cc:197: // TODO(ianswett): Remove
this test.
On 2015/02/18 17:17:37, asanka wrote:
> On 2015/02/17 at 18:52:53, ramant wrote:
> > Please don't remove this code. It is shared with the internal source tree.
> 
> Can you summarize the current state of this test? Is there a bug to point at?
> 
> I think while we are looking at this, we should at least update the comment
> since it's clearly misleading.

Checked with Ian. He said this test can be removed. Will merge this change into
the internal source tree.

[zhuoyu.qian, 2015-02-26 07:31:24]

The CQ bit was checked by zhuoyu.qian@samsung.com

[commit-bot: I haz the power, 2015-02-26 07:31:39]

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

[commit-bot: I haz the power, 2015-02-26 08:08:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-02-26 08:08:47]

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