Bug 1041148 - Move HoldJSObjects call to XMLHttpRequest constructor t…

…o simplify the code. r=bz
dothq · Aug 6, 2014 · 82972d1 · 82972d1
1 parent d1a4aad
commit 82972d1
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/dom/workers/XMLHttpRequest.cpp b/dom/workers/XMLHttpRequest.cpp
@@ -1577,6 +1577,8 @@ XMLHttpRequest::XMLHttpRequest(WorkerPrivate* aWorkerPrivate)
   mWorkerPrivate->AssertIsOnWorkerThread();
 
   SetIsDOMBinding();
+
+  mozilla::HoldJSObjects(this);
 }
 
 XMLHttpRequest::~XMLHttpRequest()
@@ -2365,7 +2367,4 @@ XMLHttpRequest::UpdateState(const StateData& aStateData,
   else {
     mStateData = aStateData;
   }
-  if (mStateData.mResponse.isGCThing()) {
-    mozilla::HoldJSObjects(this);
-  }
 }