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

Side by Side Diff: base/synchronization/lock_impl_win.cc

Issue 1980743002: Track thread activities in order to diagnose hangs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@readwrite-mmf
Patch Set: track locks and waitable events Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/synchronization/lock_impl.h" 5 #include "base/synchronization/lock_impl.h"
6 6
7 #include "base/debug/activity_tracker.h"
8
7 namespace base { 9 namespace base {
8 namespace internal { 10 namespace internal {
9 11
10 LockImpl::LockImpl() : native_handle_(SRWLOCK_INIT) {} 12 LockImpl::LockImpl() : native_handle_(SRWLOCK_INIT) {}
11 13
12 LockImpl::~LockImpl() = default; 14 LockImpl::~LockImpl() = default;
13 15
14 bool LockImpl::Try() { 16 bool LockImpl::Try() {
15 return !!::TryAcquireSRWLockExclusive(&native_handle_); 17 return !!::TryAcquireSRWLockExclusive(&native_handle_);
16 } 18 }
17 19
18 void LockImpl::Lock() { 20 void LockImpl::Lock() {
21 base::debug::ScopedLockActivity lock_activity(this);
19 ::AcquireSRWLockExclusive(&native_handle_); 22 ::AcquireSRWLockExclusive(&native_handle_);
20 } 23 }
21 24
22 void LockImpl::Unlock() { 25 void LockImpl::Unlock() {
23 ::ReleaseSRWLockExclusive(&native_handle_); 26 ::ReleaseSRWLockExclusive(&native_handle_);
24 } 27 }
25 28
26 } // namespace internal 29 } // namespace internal
27 } // namespace base 30 } // namespace base
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698