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

Issue 8972: Switch from the benighted PlatformThread to the shiny SimpleThread (Closed)

Created:
12 years, 1 month ago by dank
Modified:
9 years, 7 months ago
Reviewers:
Dean McNamee
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Switch from the benighted PlatformThread to the shiny SimpleThread Also add a comment to explain the strange tls gyrations in SlotReturnFunction. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=4481

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -11 lines) Patch
M base/stats_table.cc View 2 1 chunk +3 lines, -0 lines 0 comments Download
M base/stats_table_unittest.cc View 1 2 3 chunks +13 lines, -11 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
dank
Here y'go.
12 years, 1 month ago (2008-10-31 20:19:04 UTC) #1
Dean McNamee
12 years, 1 month ago (2008-10-31 20:30:20 UTC) #2
Thanks.  LGTM with the changes.

http://codereview.chromium.org/8972/diff/202/204
File base/stats_table.cc (right):

http://codereview.chromium.org/8972/diff/202/204#newcode349
Line 349: // on some platforms as already cleared the TLS
typo.  Plus, your comment line are really short, you can
make the 80 char, it will look at a lot nicer.

http://codereview.chromium.org/8972/diff/202/204#newcode350
Line 350: // info.  To be portable, must use the tls_data
XXX must use, this sentence doesn't read right.

http://codereview.chromium.org/8972/diff/202/203
File base/stats_table_unittest.cc (right):

http://codereview.chromium.org/8972/diff/202/203#newcode74
Line 74: base::SimpleThread(name), id_(id) { };
This goes on the same line as the :

No ; at the end, this is a definition.

http://codereview.chromium.org/8972/diff/202/203#newcode117
Line 117: StatsTableThread *threads[kMaxThreads];
* is on the wrong place, goes on the type.

Powered by Google App Engine
This is Rietveld 408576698