Bug 1594166 - Don't add a manual ref when constructing FileChannelChild, since it's a refcounted IPDL class. r=nika

Differential Revision: https://phabricator.services.mozilla.com/D51904

--HG--
extra : moz-landing-system : lando

netwerk/protocol/file/FileChannelChild.cpp

@@ -15,8 +15,7 @@ namespace net {
 
 NS_IMPL_ISUPPORTS_INHERITED(FileChannelChild, nsFileChannel, nsIChildChannel)
 
-FileChannelChild::FileChannelChild(nsIURI* uri)
+FileChannelChild::FileChannelChild(nsIURI* uri) : nsFileChannel(uri) {}
-    : nsFileChannel(uri), mIPCOpen(false) {}
 
 NS_IMETHODIMP
 FileChannelChild::ConnectParent(uint32_t id) {
@@ -29,8 +28,6 @@ FileChannelChild::ConnectParent(uint32_t id) {
   if (!gNeckoChild->SendPFileChannelConstructor(this, id)) {
     return NS_ERROR_FAILURE;
   }
-
-  AddIPDLReference();
   return NS_OK;
 }
 
@@ -45,22 +42,12 @@ FileChannelChild::CompleteRedirectSetup(nsIStreamListener* listener,
     return rv;
   }
 
-  if (mIPCOpen) {
+  if (CanSend()) {
     Unused << Send__delete__(this);
   }
 
   return NS_OK;
 }
 
-void FileChannelChild::AddIPDLReference() {
-  AddRef();  // Released in NeckoChild::DeallocPFileChannelChild.
-  mIPCOpen = true;
-}
-
-void FileChannelChild::ActorDestroy(ActorDestroyReason why) {
-  MOZ_ASSERT(mIPCOpen);
-  mIPCOpen = false;
-}
-
 }  // namespace net
 }  // namespace mozilla

netwerk/protocol/file/FileChannelChild.h

@@ -25,16 +25,8 @@ class FileChannelChild : public nsFileChannel,
   NS_DECL_ISUPPORTS_INHERITED
   NS_DECL_NSICHILDCHANNEL
 
- protected:
-  virtual void ActorDestroy(ActorDestroyReason why) override;
-
  private:
   ~FileChannelChild() = default;
-  ;
-
-  void AddIPDLReference();
-
-  bool mIPCOpen;
 };
 
 }  // namespace net