Revert of Fix flaky crash in WebPagePopupImpl::closePopup.

Source/web/WebViewImpl.cpp

Index: Source/web/WebViewImpl.cpp
diff --git a/Source/web/WebViewImpl.cpp b/Source/web/WebViewImpl.cpp
index cf319d8d9143b1e5cb6bd836b4cbecaea5ac8cdd..6eb50580c2bf29e160f74b222c7187075b105a8d 100644
--- a/Source/web/WebViewImpl.cpp
+++ b/Source/web/WebViewImpl.cpp
@@ -1646,15 +1646,11 @@
 {
     ASSERT(popup);
     WebPagePopupImpl* popupImpl = toWebPagePopupImpl(popup);
-    // It's possible that closePagePopup is called from pagePopup->closePopup()
-    // below because the main frame of the page popup can have the last
-    // reference to the pagePopupOwner Element. So, we clear m_pagePopup before
-    // pagePopup->closePopup(), and do nothing if m_pagepopup is nullptr.
-    ASSERT(!m_pagePopup || m_pagePopup.get() == popupImpl);
-    if (!m_pagePopup || m_pagePopup.get() != popupImpl)
-        return;
-    RefPtr<WebPagePopupImpl> pagePopup = m_pagePopup.release();
-    pagePopup->closePopup();
+    ASSERT(m_pagePopup.get() == popupImpl);
+    if (m_pagePopup.get() != popupImpl)
+        return;
+    m_pagePopup->closePopup();
+    m_pagePopup = nullptr;
     disablePopupMouseWheelEventListener();
 }