[Webkit-unassigned] [Bug 86658] New: Some image loading logic happens in an unintuitive and incorrect location

[bugzilla-daemon at webkit.org]

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

           Summary: Some image loading logic happens in an unintuitive and
                    incorrect location
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Keywords: InRadar
          Severity: Normal
          Priority: P2
         Component: Page Loading
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: jeffrey at endrift.com
                CC: mjs at apple.com, beidson at apple.com


When loading images, some of the logic regarding making the images appear on the page seems to occur somewhere within the beforeload logic (somewhere within code called by ImageLoader::dispatchPendingBeforeLoadEvent()). This means that a beforeload event needs to be dispatched on all loaded images, even within some contexts like ImageDocuments.

While calling dispatchPendingBeforeLoadEvent while the image is being first updated (ImageLoader::updateFromElement), instead of being deferred (as happens when the page has a beforeload handler) ensures that listeners never actually sees the event, it still shouldn't fire the event in the first place. This means that the logic for connecting the images so they appear needs to be moved out of whatever dispatchPendingBeforeLoadEvent calls and into a separate function.

I've tried to find where this logic occurs, but I have as of yet been unsuccessful.

<rdar://problem/11465863>

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.