RESOLVED FIXED 82283
ImageLoader::m_firedLoadEvent is a confusing name 
https://bugs.webkit.org/show_bug.cgi?id=82283
Summary ImageLoader::m_firedLoadEvent is a confusing name
WebKit Review Bot
Reported 2012-03-26 18:39:01 PDT
Clean up ImageLoader::m_firedLoadEvent Requested by abarth on #webkit.
Attachments
Patch (12.02 KB, patch)
2012-03-26 22:56 PDT, Adam Barth 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2012-03-26 22:56:15 PDT
Created attachment 133980 [details] Patch
Kentaro Hara
Comment 2 2012-03-26 23:04:03 PDT
Comment on attachment 133980 [details] Patch OK
WebKit Review Bot
Comment 3 2012-03-27 00:21:55 PDT
Comment on attachment 133980 [details] Patch Clearing flags on attachment: 133980 Committed r112215: <http://trac.webkit.org/changeset/112215>
WebKit Review Bot
Comment 4 2012-03-27 00:21:59 PDT
All reviewed patches have been landed. Closing bug.
Alexey Proskuryakov
Comment 5 2012-03-27 14:21:44 PDT
I had an abandoned patch for the same some time ago, <https://bugs.webkit.org/attachment.cgi?id=76669&action=review>. It got stuck because it was trying to do too much at once. Now that you're at it, perhaps you'll have the interest to finish the job, and make script elements sensible, too?
Adam Barth
Comment 6 2012-03-27 14:30:30 PDT
> Now that you're at it, perhaps you'll have the interest to finish the job, and make script elements sensible, too? Yeah, I saw the script element one in my grep as well. I'm happy to change that one too.
Note You need to log in before you can comment on or make changes to this bug.