Reuse hasPendingActivity() in ImageLoader

https://bugs.webkit.org/show_bug.cgi?id=290914

Reviewed by Tim Nguyen.

hasPendingActivity() encompasses the exact logic that is needed in
updatedHasPendingEvent() so might as well use it.

* Source/WebCore/loader/ImageLoader.cpp:
(WebCore::ImageLoader::updatedHasPendingEvent):

Canonical link: https://commits.webkit.org/293118@main

Author: annevk

Source/WebCore/loader/ImageLoader.cpp

@@ -547,12 +547,7 @@ void ImageLoader::updatedHasPendingEvent()
     // destroyed by DOM manipulation or garbage collection.
     // If such an Element wishes for the load to stop when removed from the DOM it needs to stop the ImageLoader explicitly.
     bool wasProtected = m_elementIsProtected;
-
-    // Because of lazy image loading, an image's load may be deferred indefinitely. To avoid leaking the element, we only
-    // protect it once the load has actually started.
-    bool imageWillBeLoadedLater = m_image && !m_image->isLoading() && m_image->stillNeedsLoad();
-
-    m_elementIsProtected = (m_hasPendingLoadEvent && !imageWillBeLoadedLater) || m_hasPendingErrorEvent;
+    m_elementIsProtected = hasPendingActivity();
     if (wasProtected == m_elementIsProtected)
         return;