Title: [224418] trunk/Source/WebCore
Revision
224418
Author
[email protected]
Date
2017-11-03 13:12:10 -0700 (Fri, 03 Nov 2017)

Log Message

Remove an invalid ASSERT that is seen intermittently.
https://bugs.webkit.org/show_bug.cgi?id=179186

Unreviewed.


* workers/service/server/SWServerRegistration.cpp:
(WebCore::SWServerRegistration::removeClientServiceWorkerRegistration): This is an invalid ASSERT,
  it's definitely racey over IPC.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (224417 => 224418)


--- trunk/Source/WebCore/ChangeLog	2017-11-03 20:02:06 UTC (rev 224417)
+++ trunk/Source/WebCore/ChangeLog	2017-11-03 20:12:10 UTC (rev 224418)
@@ -1,3 +1,14 @@
+2017-11-03  Brady Eidson  <[email protected]>
+
+        Remove an invalid ASSERT that is seen intermittently.
+        https://bugs.webkit.org/show_bug.cgi?id=179186
+
+        Unreviewed.
+
+        * workers/service/server/SWServerRegistration.cpp:
+        (WebCore::SWServerRegistration::removeClientServiceWorkerRegistration): This is an invalid ASSERT,
+          it's definitely racey over IPC.
+
 2017-11-03  Michael Catanzaro  <[email protected]>
 
         [WPE] Remove unneeded USE(LIBEPOXY) guards

Modified: trunk/Source/WebCore/workers/service/server/SWServerRegistration.cpp (224417 => 224418)


--- trunk/Source/WebCore/workers/service/server/SWServerRegistration.cpp	2017-11-03 20:02:06 UTC (rev 224417)
+++ trunk/Source/WebCore/workers/service/server/SWServerRegistration.cpp	2017-11-03 20:12:10 UTC (rev 224418)
@@ -104,7 +104,6 @@
     if (iterator == m_clientRegistrationsByConnection.end() || !iterator->value)
         return;
     
-    ASSERT(iterator->value->contains(clientRegistrationIdentifier));
     iterator->value->remove(clientRegistrationIdentifier);
     if (iterator->value->isEmpty())
         m_clientRegistrationsByConnection.remove(iterator);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to