Bug 1110928, part 1 - Hoist the LOAD_END PokeGC out of nsJSContext::LoadEnd. r=smaug

nsDocumentViewer knows which document is involved which will help us later.

Also, fix a typo in the comment.

MozReview-Commit-ID: L6AyRUV7Y3x

dom/base/nsJSEnvironment.cpp

@@ -1860,9 +1860,7 @@ nsJSContext::LoadEnd()
     return;
   }
 
-  // Its probably a good idea to GC soon since we have finished loading.
   sLoadingInProgress = false;
-  PokeGC(JS::gcreason::LOAD_END);
 }
 
 // Only trigger expensive timers when they have been checked a number of times.

layout/base/nsDocumentViewer.cpp

@@ -1037,6 +1037,9 @@ nsDocumentViewer::LoadComplete(nsresult aStatus)
 
   nsJSContext::LoadEnd();
 
+  // It's probably a good idea to GC soon since we have finished loading.
+  PokeGC(JS::gcreason::LOAD_END);
+
 #ifdef NS_PRINTING
   // Check to see if someone tried to print during the load
   if (mPrintIsPending) {