Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(17)

Issue 27001: Bug 906324

Can't Edit
Can't Publish+Mail
Start Review
Created:
7 years, 5 months ago by acain
Modified:
7 years, 4 months ago
Reviewers:
ekr-webrtc
Visibility:
Public.

Description

Bug 906324

First cut at adding single-agent ICE test app (C++) 
and Python test wrapper.

-- Moved existing mtransport/test/ unit test code into
   mtransport/test/unit/ dir

-- Created new mtransport/test/apps dir for new files
   (suggest better place?)

-- I'm hoping to refactor the IceTestPeer class and share
   it between the new ice_peer and also ice_unittest 

-- The STUN and TURN options have not been exercised yet

-- The offerer/answerer option is not used (except for
   passing it to the NrIceCtx ctor, which doesn't use it)

-- The new test_ice_peer.py script gets copied to the
   mtransport/test/apps/ directory.  (suggest better 
   location?)  I run it from there.

Patch Set 1 #

Patch Set 2 : IceTestPeer refactor, ice_peer improvements #

Unified diffs Side-by-side diffs Delta from patch set Stats Patch
A media/mtransport/test/IceTestPeer.h View 1 1 chunk +146 lines, -0 lines 0 comments Download
A media/mtransport/test/IceTestPeerImpl.h View 1 1 chunk +354 lines, -0 lines 0 comments Download
M media/mtransport/test/Makefile.in View 1 1 chunk +9 lines, -0 lines 0 comments Download
A media/mtransport/test/ice_peer.cpp View 1 1 chunk +408 lines, -0 lines 0 comments Download
M media/mtransport/test/ice_unittest.cpp View 1 8 chunks +21 lines, -358 lines 0 comments Download
M media/mtransport/test/moz.build View 1 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 1

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 905+:3555e821a5cd+