Make traffic shaping work when WPR chooses the server ports.

Make traffic shaping work when WPR chooses the server ports.

WPR may be invoked with ports of 0 to let WPR choose an ephemeral port.
However, we weren't handling this properly and detecting the chosen port
for traffic shaping. This CL fixes that by getting the actual port used.

R=slamm@chromium.org

Committed: https://code.google.com/p/web-page-replay/source/detail?r=540

[tonyg, 2014-02-04 05:10:47]

ptal

[slamm, 2014-02-04 17:36:02]

I like it!

https://codereview.chromium.org/153503002/diff/1/trafficshaper.py
File trafficshaper.py (right):

https://codereview.chromium.org/153503002/diff/1/trafficshaper.py#newcode56
trafficshaper.py:56: ports=None,
Nice simplification.

https://codereview.chromium.org/153503002/diff/1/trafficshaper.py#newcode110
trafficshaper.py:110: ports = ','.join([str(p) for p in self.ports])
A generator works just as well here:

    ports = ','.join(str(p) for p in self.ports)

(Not that it matters.)

[tonyg, 2014-02-04 17:42:21]

> https://codereview.chromium.org/153503002/diff/1/trafficshaper.py#newcode110
> trafficshaper.py:110: ports = ','.join([str(p) for p in self.ports])
> A generator works just as well here:
> 
>     ports = ','.join(str(p) for p in self.ports)

Oh cool, I didn't know that syntax worked without the square braces. Fixed.

Do you want to provide the official lgtm or should I assume the I like it counts
:)

[slamm, 2014-02-04 17:45:30]

I lgtm it!

[tonyg, 2014-02-04 18:08:11]

Committed patchset #2 manually as r540 (presubmit successful).