Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(94)

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 10824224: Add CNS function to clean up port matching requester IP address. (Closed)

Created:
8 years, 4 months ago by shadi
Modified:
8 years, 4 months ago
Reviewers:
DaleCurtis
CC:
chromium-reviews, feature-media-reviews_chromium.org, pam+watch_chromium.org
Base URL:
http://git.chromium.org/chromium/src.git@cns_apache
Visibility:
Public.

Description

Add CNS function to clean up port matching requester IP address. BUG=141537 TEST=unit tests Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=150705

Patch Set 1 : #

Total comments: 2

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+48 lines, -2 lines) Patch
M media/tools/constrained_network_server/cns.py View 1 3 chunks +17 lines, -2 lines 0 comments Download
M media/tools/constrained_network_server/cns_test.py View 1 chunk +31 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
shadi
DPTAL :-)
8 years, 4 months ago (2012-08-09 01:07:55 UTC) #1
DaleCurtis
lgtm https://chromiumcodereview.appspot.com/10824224/diff/2001/media/tools/constrained_network_server/cns.py File media/tools/constrained_network_server/cns.py (right): https://chromiumcodereview.appspot.com/10824224/diff/2001/media/tools/constrained_network_server/cns.py#newcode183 media/tools/constrained_network_server/cns.py:183: """Cleans up all the ports allocated using the ...
8 years, 4 months ago (2012-08-09 01:12:34 UTC) #2
shadi
https://chromiumcodereview.appspot.com/10824224/diff/2001/media/tools/constrained_network_server/cns.py File media/tools/constrained_network_server/cns.py (right): https://chromiumcodereview.appspot.com/10824224/diff/2001/media/tools/constrained_network_server/cns.py#newcode183 media/tools/constrained_network_server/cns.py:183: """Cleans up all the ports allocated using the reques ...
8 years, 4 months ago (2012-08-09 01:17:10 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/shadi@chromium.org/10824224/2002
8 years, 4 months ago (2012-08-09 01:17:56 UTC) #4
commit-bot: I haz the power
8 years, 4 months ago (2012-08-09 03:27:14 UTC) #5 
Change committed as 150705

Powered by Google App Engine
This is Rietveld 408576698