diff options
| author | agl@chromium.org <agl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-01-23 00:25:29 +0000 |
|---|---|---|
| committer | agl@chromium.org <agl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-01-23 00:25:29 +0000 |
| commit | 0049398c72b3fd711c825d477e0db6943f8f5db6 (patch) | |
| tree | f3d923a985aaf1f2148ba0cd7a69925a8ea21e0a /base/thread_collision_warner.h | |
| parent | 1ebed56434c220f82cf220bbb72a136fea962c87 (diff) | |
| download | chromium_src-0049398c72b3fd711c825d477e0db6943f8f5db6.zip chromium_src-0049398c72b3fd711c825d477e0db6943f8f5db6.tar.gz chromium_src-0049398c72b3fd711c825d477e0db6943f8f5db6.tar.bz2 | |
Use PlatformThreadId, not int when dealing with thread ids.
Windows uses a DWORD (unsigned long) for thread ids and POSIX uses a
pid_t (int on Linux) for the same. In the code, we are currently
stuffing thread ids into an int which is dangerous on Windows (because
DWORDS can exceed an int and wrap) and will break if pid_t is ever !=
int.
This change changes all the places where we currently have an int to
use a new typedef, PlatformThreadId. This change also needs to occur
for process ids, but I'm not doing that in this CL.
Review URL: http://codereview.chromium.org/18677
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8525 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/thread_collision_warner.h')
| -rw-r--r-- | base/thread_collision_warner.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/base/thread_collision_warner.h b/base/thread_collision_warner.h index fa8b7ee..6545451 100644 --- a/base/thread_collision_warner.h +++ b/base/thread_collision_warner.h @@ -225,7 +225,7 @@ class ThreadCollisionWarner { // This stores the thread id that is inside the critical section, if the // value is 0 then no thread is inside. - volatile int valid_thread_id_; + volatile subtle::Atomic32 valid_thread_id_; // Counter to trace how many time a critical section was "pinned" // (when allowed) in order to unpin it when counter_ reaches 0. |