Backed out changeset 0a5a34ef66d1 (bug 1388998) for mass test failures during shutdown. r=backout on a CLOSED TREE

storage/mozStorageService.cpp

@@ -309,15 +309,8 @@ Service::unregisterConnection(Connection *aConnection)
         // Ensure the connection is released on its opening thread.  Note, we
         // must use .forget().take() so that we can manually cast to an
         // unambiguous nsISupports type.
-        //
-        // We specify aAlwaysProxy=true because if we don't, the destructor for
-        // the connection will be invoked if the connection is owned by the
-        // main thread.  And since the Connection destructor calls Close() and
-        // that may in turn call SpinningSynchronousClose() if the connection
-        // was not properly shutdown, that can lead to re-entrancy problems.
         NS_ProxyRelease(
-          "storage::Service::mConnections", thread, mConnections[i].forget(),
-          true);
+          "storage::Service::mConnections", thread, mConnections[i].forget());
 
         mConnections.RemoveElementAt(i);
         return;