Bug 42859

Summary: Process link elements immediately if there are no beforeload listeners.
Product: WebKit Reporter: Andy Estes <aestes>
Component: New BugsAssignee: Andy Estes <aestes>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, eric, ossy, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
patch mjs: review+, aestes: commit-queue-

Andy Estes
Reported 2010-07-22 16:58:41 PDT
Process link elements immediately if there are no beforeload listeners.
Attachments
patch (1.74 KB, patch)
2010-07-22 17:19 PDT, Andy Estes 		mjs: review+
aestes: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andy Estes
Comment 1 2010-07-22 17:19:31 PDT
Created attachment 62364 [details] patch
Maciej Stachowiak
Comment 2 2010-07-22 17:20:51 PDT
Comment on attachment 62364 [details] patch r=me
Andy Estes
Comment 3 2010-07-22 17:26:30 PDT
Committed <http://trac.webkit.org/changeset/63924>.
WebKit Review Bot
Comment 4 2010-07-22 17:47:25 PDT
http://trac.webkit.org/changeset/63924 might have broken Qt Linux Release
Csaba Osztrogonác
Comment 5 2010-07-23 09:00:43 PDT
Qt specific expected result added: http://trac.webkit.org/changeset/63977
Andy Estes
Comment 6 2010-07-23 09:15:02 PDT
(In reply to comment #5) > Qt specific expected result added: http://trac.webkit.org/changeset/63977 Thanks, Csaba. I actually filed a bug to add this test to the Skipped list (https://bugs.webkit.org/show_bug.cgi?id=42869), because it seemed like a bug that Qt was emitting those two extra frame load events. If that's not the case, I'll just close out that bug.
Note You need to log in before you can comment on or make changes to this bug.