Bug 1381311 - change main thread check to STS thread check. r=mayhemer
authorShih-Chiang Chien <schien@mozilla.com>
Mon, 17 Jul 2017 09:51:54 +0800
changeset 369292 7f71a2cc11518ba16e695d07698ffbfa5687fdcd
parent 369291 216a5bf264b2dae2aefef068533cc0e1bb4d21d7
child 369293 18aeda1e84fb36f120a3938b0c86f7cf5a0b5e30
push id46632
push userschien@mozilla.com
push dateTue, 18 Jul 2017 00:44:05 +0000
treeherderautoland@7f71a2cc1151 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmayhemer
bugs1381311, 1338493
milestone56.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1381311 - change main thread check to STS thread check. r=mayhemer In Bug 1338493 I moved the operation thread of HttpBackgroundChannelChild from main thread to STS thread. All the thread assertion shoudl be updated. MozReview-Commit-ID: 307jBR48aYc
netwerk/protocol/http/HttpBackgroundChannelChild.cpp
--- a/netwerk/protocol/http/HttpBackgroundChannelChild.cpp
+++ b/netwerk/protocol/http/HttpBackgroundChannelChild.cpp
@@ -447,17 +447,17 @@ HttpBackgroundChannelChild::ActorDestroy
   // If background channel is not closed by normal IPDL actor deletion,
   // remove the HttpChannelChild reference and notify background channel
   // destroyed immediately.
   if (aWhy == Deletion && !mQueuedRunnables.IsEmpty()) {
     LOG(("  > pending until queued messages are flushed\n"));
     RefPtr<HttpBackgroundChannelChild> self = this;
     mQueuedRunnables.AppendElement(NS_NewRunnableFunction(
       "HttpBackgroundChannelChild::ActorDestroy", [self]() {
-        MOZ_ASSERT(NS_IsMainThread());
+        MOZ_ASSERT(OnSocketThread());
         RefPtr<HttpChannelChild> channelChild = self->mChannelChild.forget();
 
         if (channelChild) {
           channelChild->OnBackgroundChildDestroyed(self);
         }
       }));
     return;
   }