Chromium Code Reviews
Help | Chromium Project | Sign in
(229)

Issue 11015008: Fix cctests using Sockets to be able to run in parallel (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 6 months ago by Jakob
Modified:
1 year, 6 months ago
Reviewers:
Michael Starzinger
CC:
v8-dev_googlegroups.com
Visibility:
Public.

Description

Fix cctests using Sockets to be able to run in parallel

BUG=v8:945
Committed: https://code.google.com/p/v8/source/detail?r=12636

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -15 lines) Lint Patch
M test/cctest/cctest.h View 2 chunks +8 lines, -0 lines 0 comments 0 errors Download
M test/cctest/cctest.status View 2 chunks +0 lines, -10 lines 0 comments ? errors Download
M test/cctest/test-debug.cc View 2 chunks +4 lines, -4 lines 0 comments 0 errors Download
M test/cctest/test-sockets.cc View 1 chunk +1 line, -1 line 0 comments 0 errors Download
Trybot results:
Commit:

Messages

Total messages: 2
Jakob
This is the approach you suggested. Please take a look.
1 year, 6 months ago #1
Michael Starzinger
1 year, 6 months ago #2
LGTM.

As discussed offline, there would be a possible way to get a free port from the
OS (see comment on linked issue). But that would require us to add another
function to our platform abstraction layer and implement it on all platforms
just for the sake of testing. So we should stick with this simple approach.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6