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.h | |
| 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.h')
| -rw-r--r-- | base/lock.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/base/lock.h b/base/lock.h index f3653cf..31ad9a0 100644 --- a/base/lock.h +++ b/base/lock.h @@ -23,7 +23,7 @@ class Lock { // calling thread. If the lock has not been acquired, then the method // will DCHECK(). In non-debug builds, the LockImpl's implementation of // AssertAcquired() is an empty inline method. - void AssertAcquired() { return lock_.AssertAcquired(); } + void AssertAcquired() const { return lock_.AssertAcquired(); } // Return the underlying lock implementation. // TODO(awalker): refactor lock and condition variables so that this is |