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

Issue 6080: This makes net_perftests build and run on Linux. (Closed)

Created:
12 years, 2 months ago by please use my chromium address
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

This makes net_perftests build and run on Linux. BUG=3065

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+42 lines, -12 lines) Patch
M net/SConscript View 2 chunks +16 lines, -11 lines 0 comments Download
M net/disk_cache/disk_cache_perftest.cc View 1 4 chunks +26 lines, -1 line 0 comments Download

Messages

Total messages: 8 (0 generated)
please use my chromium address
12 years, 2 months ago (2008-10-02 15:04:40 UTC) #1
Dean McNamee
Good enough for me. I don't know in practice if the fadvise will really have ...
12 years, 2 months ago (2008-10-02 15:10:32 UTC) #2
please use my chromium address
On 2008/10/02 15:10:32, Dean McNamee wrote: > http://codereview.chromium.org/6080/diff/1/3 > File net/disk_cache/disk_cache_perftest.cc (right): > > http://codereview.chromium.org/6080/diff/1/3#newcode75 ...
12 years, 2 months ago (2008-10-02 15:20:40 UTC) #3
sgk
lgtm
12 years, 2 months ago (2008-10-02 20:07:25 UTC) #4
Mark Mentovai
LGTM also, I can't check this in right now though because we're in the middle ...
12 years, 2 months ago (2008-10-02 20:31:22 UTC) #5
please use my chromium address
On 2008/10/02 20:31:22, Mark Mentovai wrote: > LGTM also, I can't check this in right ...
12 years, 2 months ago (2008-10-08 10:30:55 UTC) #6
Evan Martin
I'm looking at this now.
12 years, 2 months ago (2008-10-13 17:41:11 UTC) #7
Evan Martin
12 years, 2 months ago (2008-10-13 18:37:18 UTC) #8
I had to remove unistd.h for Windows.

r3298

Powered by Google App Engine
This is Rietveld 408576698