Bug 1362284 - Assign the background thread before flushing pending events at GamepadEventChannelParent; r?lenzak draft
authorDaosheng Mu <daoshengmu@gmail.com>
Fri, 05 May 2017 01:21:39 +0800
changeset 572995 1059562dfbd743d17e309204799fa436a22c3e10
parent 572730 0b255199db9d6a6f189b89b7906f99155bde3726
child 627187 040eb728ac05ac2afbee7487429c49162eb0d84d
push id57256
push userbmo:dmu@mozilla.com
push dateFri, 05 May 2017 03:53:32 +0000
reviewerslenzak
bugs1362284
milestone55.0a1
Bug 1362284 - Assign the background thread before flushing pending events at GamepadEventChannelParent; r?lenzak MozReview-Commit-ID: FcnAM1JSON2
dom/gamepad/ipc/GamepadEventChannelParent.cpp
--- a/dom/gamepad/ipc/GamepadEventChannelParent.cpp
+++ b/dom/gamepad/ipc/GamepadEventChannelParent.cpp
@@ -40,18 +40,18 @@ class SendGamepadUpdateRunnable final : 
 } // namespace
 
 GamepadEventChannelParent::GamepadEventChannelParent()
   : mHasGamepadListener(false)
 {
   RefPtr<GamepadPlatformService> service =
     GamepadPlatformService::GetParentService();
   MOZ_ASSERT(service);
+  mBackgroundThread = NS_GetCurrentThread();
   service->AddChannelParent(this);
-  mBackgroundThread = NS_GetCurrentThread();
 }
 
 mozilla::ipc::IPCResult
 GamepadEventChannelParent::RecvGamepadListenerAdded()
 {
   AssertIsOnBackgroundThread();
   MOZ_ASSERT(!mHasGamepadListener);
   mHasGamepadListener = true;