Remove PacketRouterTest fixture

Remove PacketRouterTest fixture

Remove the mostly-unused fixture PacketRouterTest.

BUG=None

Review-Url: https://codereview.webrtc.org/2991093002
Cr-Commit-Position: refs/heads/master@{#19203}
Committed: https://chromium.googlesource.com/external/webrtc/+/b1338fec81b8eea646c114f5fd555a94d85abb4e

[eladalon, 2017-07-31 14:15:35]

eladalon@webrtc.org changed reviewers:
+ danilchap@webrtc.org

[eladalon, 2017-07-31 14:15:36]

PTAL

[danilchap, 2017-07-31 14:18:15]

lgtm

https://codereview.webrtc.org/2991093002/diff/1/webrtc/modules/pacing/packet_...
File webrtc/modules/pacing/packet_router_unittest.cc (right):

https://codereview.webrtc.org/2991093002/diff/1/webrtc/modules/pacing/packet_...
webrtc/modules/pacing/packet_router_unittest.cc:36: }  // namespace
since you've added an unnamed namespace, may be hide own MockWithRemb inside it
too.

[eladalon, 2017-07-31 14:31:34]

eladalon@webrtc.org changed reviewers:
+ stefan@webrtc.org

[eladalon, 2017-07-31 14:31:35]

Stefan, this needs an owner-lgtm.

[stefan-webrtc, 2017-07-31 14:32:34]

rs-lgtm

[eladalon, 2017-07-31 14:33:16]

https://codereview.webrtc.org/2991093002/diff/1/webrtc/modules/pacing/packet_...
File webrtc/modules/pacing/packet_router_unittest.cc (right):

https://codereview.webrtc.org/2991093002/diff/1/webrtc/modules/pacing/packet_...
webrtc/modules/pacing/packet_router_unittest.cc:36: }  // namespace
On 2017/07/31 14:18:15, danilchap wrote:
> since you've added an unnamed namespace, may be hide own MockWithRemb inside
it
> too.

Done.

[eladalon, 2017-08-01 15:46:22]

The CQ bit was checked by eladalon@webrtc.org

[eladalon, 2017-08-01 15:46:23]

The patchset sent to the CQ was uploaded after l-g-t-m from
danilchap@webrtc.org, stefan@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2991093002/#ps40001 (title:
"Rebased")

[commit-bot: I haz the power, 2017-08-01 15:46:27]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-08-01 16:36:20]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1501602382889370,
"parent_rev": "5dfac33dfdd430d60f3ecf4c59d15a62ec1f68ef", "commit_rev":
"b1338fec81b8eea646c114f5fd555a94d85abb4e"}

[commit-bot: I haz the power, 2017-08-01 16:36:25]

Description was changed from

==========
Remove PacketRouterTest fixture

Remove the mostly-unused fixture PacketRouterTest.

BUG=None
==========

to

==========
Remove PacketRouterTest fixture

Remove the mostly-unused fixture PacketRouterTest.

BUG=None

Review-Url: https://codereview.webrtc.org/2991093002
Cr-Commit-Position: refs/heads/master@{#19203}
Committed:
https://chromium.googlesource.com/external/webrtc/+/b1338fec81b8eea646c114f5f...
==========

[commit-bot: I haz the power, 2017-08-01 16:36:26]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/external/webrtc/+/b1338fec81b8eea646c114f5f...