Changeset 277917 in webkit for trunk/Source/JavaScriptCore/interpreter

Timestamp:

May 22, 2021, 4:17:01 AM (4 years ago)

Author:

[email protected]

Message:

Unreviewed, reverting r277913.
​https://bugs.webkit.org/show_bug.cgi?id=226140

Caused multiple layout-test crash on mac debug queues

Reverted changeset:

"Replace LockHolder with Locker in local variables"
​https://bugs.webkit.org/show_bug.cgi?id=226133
​https://trac.webkit.org/changeset/277913

File:

• trunk/Source/JavaScriptCore/interpreter/CLoopStack.cpp (modified) (2 diffs)

trunk/Source/JavaScriptCore/interpreter/CLoopStack.cpp

@@ -140 +140 @@
 void CLoopStack::addToCommittedByteCount(long byteCount)
 {
-    Locker locker { stackStatisticsMutex };
+    LockHolder locker(stackStatisticsMutex);
     ASSERT(static_cast<long>(committedBytesCount) + byteCount > -1);
     committedBytesCount += byteCount;
@@ -160 +160 @@
 size_t CLoopStack::committedByteCount()
 {
-    Locker locker { stackStatisticsMutex };
+    LockHolder locker(stackStatisticsMutex);
     return committedBytesCount;
 }