[webkit-changes] [252651] trunk/Source/WebCore

Tue, 19 Nov 2019 14:32:27 -0800

Title: [252651] trunk/Source/WebCore
Revision
252651
Author
[email protected]
Date
2019-11-19 14:31:11 -0800 (Tue, 19 Nov 2019)

Log Message

Unreviewed, follow-up after r252642
https://bugs.webkit.org/show_bug.cgi?id=204346

* Modules/indexeddb/IDBActiveDOMObject.h:
* Modules/indexeddb/client/TransactionOperation.h:
(WebCore::IDBClient::TransactionOperation::~TransactionOperation):
(WebCore::IDBClient::TransactionOperation::perform):
(WebCore::IDBClient::TransactionOperation::transitionToCompleteOnThisThread):
(WebCore::IDBClient::TransactionOperation::transitionToComplete):
(WebCore::IDBClient::TransactionOperation::doComplete):
* platform/Supplementable.h:
(WebCore::Supplementable::provideSupplement):
(WebCore::Supplementable::removeSupplement):
(WebCore::Supplementable::requireSupplement):
* platform/Timer.cpp:
(WebCore::TimerBase::~TimerBase):
(WebCore::TimerBase::start):
(WebCore::TimerBase::stop):
(WebCore::TimerBase::setNextFireTime):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (252650 => 252651)


--- trunk/Source/WebCore/ChangeLog	2019-11-19 22:17:58 UTC (rev 252650)
+++ trunk/Source/WebCore/ChangeLog	2019-11-19 22:31:11 UTC (rev 252651)
@@ -1,3 +1,25 @@
+2019-11-19  Yusuke Suzuki  <[email protected]>
+
+        Unreviewed, follow-up after r252642
+        https://bugs.webkit.org/show_bug.cgi?id=204346
+
+        * Modules/indexeddb/IDBActiveDOMObject.h:
+        * Modules/indexeddb/client/TransactionOperation.h:
+        (WebCore::IDBClient::TransactionOperation::~TransactionOperation):
+        (WebCore::IDBClient::TransactionOperation::perform):
+        (WebCore::IDBClient::TransactionOperation::transitionToCompleteOnThisThread):
+        (WebCore::IDBClient::TransactionOperation::transitionToComplete):
+        (WebCore::IDBClient::TransactionOperation::doComplete):
+        * platform/Supplementable.h:
+        (WebCore::Supplementable::provideSupplement):
+        (WebCore::Supplementable::removeSupplement):
+        (WebCore::Supplementable::requireSupplement):
+        * platform/Timer.cpp:
+        (WebCore::TimerBase::~TimerBase):
+        (WebCore::TimerBase::start):
+        (WebCore::TimerBase::stop):
+        (WebCore::TimerBase::setNextFireTime):
+
 2019-11-19  Daniel Bates  <[email protected]>
 
         ASSERTION FAILURE: useDownstream ? (result > vp) : (result < vp) in WebCore::nextSentenceBoundaryInDirection()

Modified: trunk/Source/WebCore/Modules/indexeddb/IDBActiveDOMObject.h (252650 => 252651)


--- trunk/Source/WebCore/Modules/indexeddb/IDBActiveDOMObject.h	2019-11-19 22:17:58 UTC (rev 252650)
+++ trunk/Source/WebCore/Modules/indexeddb/IDBActiveDOMObject.h	2019-11-19 22:31:11 UTC (rev 252651)
@@ -38,7 +38,7 @@
     Thread& originThread() const { return m_originThread.get(); }
 
     void contextDestroyed() final {
-        ASSERT(canCurrentThreadAccessThreadLocalData(m_originThread.get()));
+        ASSERT(canCurrentThreadAccessThreadLocalData(originThread()));
 
         Locker<Lock> lock(m_scriptExecutionContextLock);
         ActiveDOMObject::contextDestroyed();

Modified: trunk/Source/WebCore/Modules/indexeddb/client/TransactionOperation.h (252650 => 252651)


--- trunk/Source/WebCore/Modules/indexeddb/client/TransactionOperation.h	2019-11-19 22:17:58 UTC (rev 252650)
+++ trunk/Source/WebCore/Modules/indexeddb/client/TransactionOperation.h	2019-11-19 22:31:11 UTC (rev 252651)
@@ -51,12 +51,12 @@
 public:
     virtual ~TransactionOperation()
     {
-        ASSERT(canCurrentThreadAccessThreadLocalData(m_originThread.get()));
+        ASSERT(canCurrentThreadAccessThreadLocalData(originThread()));
     }
 
     void perform()
     {
-        ASSERT(canCurrentThreadAccessThreadLocalData(m_originThread.get()));
+        ASSERT(canCurrentThreadAccessThreadLocalData(originThread()));
         ASSERT(m_performFunction);
         m_performFunction();
         m_performFunction = { };
@@ -64,7 +64,7 @@
 
     void transitionToCompleteOnThisThread(const IDBResultData& data)
     {
-        ASSERT(canCurrentThreadAccessThreadLocalData(m_originThread.get()));
+        ASSERT(canCurrentThreadAccessThreadLocalData(originThread()));
         m_transaction->operationCompletedOnServer(data, *this);
     }
 
@@ -72,7 +72,7 @@
     {
         ASSERT(isMainThread());
 
-        if (canCurrentThreadAccessThreadLocalData(m_originThread.get()))
+        if (canCurrentThreadAccessThreadLocalData(originThread()))
             transitionToCompleteOnThisThread(data);
         else {
             m_transaction->performCallbackOnOriginThread(*this, &TransactionOperation::transitionToCompleteOnThisThread, data);
@@ -83,7 +83,7 @@
 
     void doComplete(const IDBResultData& data)
     {
-        ASSERT(canCurrentThreadAccessThreadLocalData(m_originThread.get()));
+        ASSERT(canCurrentThreadAccessThreadLocalData(originThread()));
 
         if (m_performFunction)
             m_performFunction = { };

Modified: trunk/Source/WebCore/platform/Supplementable.h (252650 => 252651)


--- trunk/Source/WebCore/platform/Supplementable.h	2019-11-19 22:17:58 UTC (rev 252650)
+++ trunk/Source/WebCore/platform/Supplementable.h	2019-11-19 22:31:11 UTC (rev 252651)
@@ -98,7 +98,7 @@
 public:
     void provideSupplement(const char* key, std::unique_ptr<Supplement<T>> supplement)
     {
-        ASSERT(canCurrentThreadAccessThreadLocalData(m_thread.get()));
+        ASSERT(canCurrentThreadAccessThreadLocalData(m_thread));
         ASSERT(!m_supplements.get(key));
         m_supplements.set(key, WTFMove(supplement));
     }
@@ -105,13 +105,13 @@
 
     void removeSupplement(const char* key)
     {
-        ASSERT(canCurrentThreadAccessThreadLocalData(m_thread.get()));
+        ASSERT(canCurrentThreadAccessThreadLocalData(m_thread));
         m_supplements.remove(key);
     }
 
     Supplement<T>* requireSupplement(const char* key)
     {
-        ASSERT(canCurrentThreadAccessThreadLocalData(m_thread.get()));
+        ASSERT(canCurrentThreadAccessThreadLocalData(m_thread));
         return m_supplements.get(key);
     }

Modified: trunk/Source/WebCore/platform/Timer.cpp (252650 => 252651)


--- trunk/Source/WebCore/platform/Timer.cpp	2019-11-19 22:17:58 UTC (rev 252650)
+++ trunk/Source/WebCore/platform/Timer.cpp	2019-11-19 22:31:11 UTC (rev 252651)
@@ -258,8 +258,8 @@
 
 TimerBase::~TimerBase()
 {
-    ASSERT(canCurrentThreadAccessThreadLocalData(m_thread.get()));
-    RELEASE_ASSERT(canCurrentThreadAccessThreadLocalData(m_thread.get()) || shouldSuppressThreadSafetyCheck());
+    ASSERT(canCurrentThreadAccessThreadLocalData(m_thread));
+    RELEASE_ASSERT(canCurrentThreadAccessThreadLocalData(m_thread) || shouldSuppressThreadSafetyCheck());
     stop();
     ASSERT(!inHeap());
     if (m_heapItem)
@@ -269,7 +269,7 @@
 
 void TimerBase::start(Seconds nextFireInterval, Seconds repeatInterval)
 {
-    ASSERT(canCurrentThreadAccessThreadLocalData(m_thread.get()));
+    ASSERT(canCurrentThreadAccessThreadLocalData(m_thread));
 
     m_repeatInterval = repeatInterval;
     setNextFireTime(MonotonicTime::now() + nextFireInterval);
@@ -277,7 +277,7 @@
 
 void TimerBase::stop()
 {
-    ASSERT(canCurrentThreadAccessThreadLocalData(m_thread.get()));
+    ASSERT(canCurrentThreadAccessThreadLocalData(m_thread));
 
     m_repeatInterval = 0_s;
     setNextFireTime(MonotonicTime { });
@@ -461,8 +461,8 @@
 
 void TimerBase::setNextFireTime(MonotonicTime newTime)
 {
-    ASSERT(canCurrentThreadAccessThreadLocalData(m_thread.get()));
-    RELEASE_ASSERT(canCurrentThreadAccessThreadLocalData(m_thread.get()) || shouldSuppressThreadSafetyCheck());
+    ASSERT(canCurrentThreadAccessThreadLocalData(m_thread));
+    RELEASE_ASSERT(canCurrentThreadAccessThreadLocalData(m_thread) || shouldSuppressThreadSafetyCheck());
     bool timerHasBeenDeleted = std::isnan(m_unalignedNextFireTime);
     RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(!timerHasBeenDeleted);
 

_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to