Bug 1593183 - Restore the AbandonBFCacheEntry call that bug 1536471 removed. r=annyG

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

--HG--
extra : rebase_source : bfa37a8296131acb2d55537820b95545cd545b05
extra : source : 199ada1714a27bf5635bf479a0f1b8f73b4cb78a
extra : histedit_source : bdb3cb1326ed5cf864d2b37b77d9098bbe289fac

docshell/shistory/nsSHEntry.cpp

@@ -1201,6 +1201,12 @@ nsLegacySHEntry::AbandonBFCacheEntry() {
   return NS_OK;
 }
 
+NS_IMETHODIMP_(void)
+nsLegacySHEntry::ClearEntry() {
+  nsSHEntry::ClearEntry();
+  AbandonBFCacheEntry();
+}
+
 NS_IMETHODIMP
 nsLegacySHEntry::GetBfcacheID(uint64_t* aBFCacheID) {
   *aBFCacheID = mShared->GetID();

docshell/shistory/nsSHEntry.h

@@ -117,6 +117,7 @@ class nsLegacySHEntry final : public nsSHEntry {
   NS_IMETHOD_(bool) HasDetachedEditor() override;
   NS_IMETHOD_(bool) HasBFCacheEntry(nsIBFCacheEntry* aEntry) override;
   NS_IMETHOD AbandonBFCacheEntry() override;
+  NS_IMETHOD_(void) ClearEntry() override;
   NS_IMETHOD CreateLoadInfo(nsDocShellLoadState** aLoadState) override;
   NS_IMETHOD GetBfcacheID(uint64_t* aBFCacheID) override;