diff options
author | ralphl@chromium.org <ralphl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-03-19 21:03:13 +0000 |
---|---|---|
committer | ralphl@chromium.org <ralphl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-03-19 21:03:13 +0000 |
commit | 1fff4a0530c3ef7d2ea5a8af727100ec04a0e3e9 (patch) | |
tree | 1eddc8b151825c4e10dace29a529b71476eb207c /base/lock_impl_win.cc | |
parent | 09c136f71329b9ef5d01a86edecf185f22af8605 (diff) | |
download | chromium_src-1fff4a0530c3ef7d2ea5a8af727100ec04a0e3e9.zip chromium_src-1fff4a0530c3ef7d2ea5a8af727100ec04a0e3e9.tar.gz chromium_src-1fff4a0530c3ef7d2ea5a8af727100ec04a0e3e9.tar.bz2 |
Test was examining a variable without first acquiring a required Lock.
Also changed Lock::AssertAcquired to be const function (it already was const, in practice, just not declared that way).
Review URL: http://codereview.chromium.org/42402
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12157 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/lock_impl_win.cc')
-rw-r--r-- | base/lock_impl_win.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/base/lock_impl_win.cc b/base/lock_impl_win.cc index 503f105..0f0e424 100644 --- a/base/lock_impl_win.cc +++ b/base/lock_impl_win.cc @@ -66,7 +66,7 @@ void LockImpl::Unlock() { // In non-debug builds, this method is declared as an empty inline method. #ifndef NDEBUG -void LockImpl::AssertAcquired() { +void LockImpl::AssertAcquired() const { DCHECK(recursion_count_shadow_ > 0); DCHECK_EQ(owning_thread_id_, PlatformThread::CurrentId()); } |