Title: [87633] trunk/Source/WebCore
- Revision
- 87633
- Author
- [email protected]
- Date
- 2011-05-28 21:42:18 -0700 (Sat, 28 May 2011)
Log Message
Suggested by Simon Fraser.
REGRESSION (r85375): Load event is sometimes lost when multiple image elements use the same URL
https://bugs.webkit.org/show_bug.cgi?id=61692
<rdar://problem/9488628>
* loader/ImageLoader.cpp: (WebCore::ImageLoader::notifyFinished): There was no need to use
ASSERT_UNUSED here.
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (87632 => 87633)
--- trunk/Source/WebCore/ChangeLog 2011-05-29 02:46:00 UTC (rev 87632)
+++ trunk/Source/WebCore/ChangeLog 2011-05-29 04:42:18 UTC (rev 87633)
@@ -1,5 +1,16 @@
2011-05-28 Alexey Proskuryakov <[email protected]>
+ Suggested by Simon Fraser.
+
+ REGRESSION (r85375): Load event is sometimes lost when multiple image elements use the same URL
+ https://bugs.webkit.org/show_bug.cgi?id=61692
+ <rdar://problem/9488628>
+
+ * loader/ImageLoader.cpp: (WebCore::ImageLoader::notifyFinished): There was no need to use
+ ASSERT_UNUSED here.
+
+2011-05-28 Alexey Proskuryakov <[email protected]>
+
Reviewed by Geoff Garen.
REGRESSION (r85375): Load event is sometimes lost when multiple image elements use the same URL
Modified: trunk/Source/WebCore/loader/ImageLoader.cpp (87632 => 87633)
--- trunk/Source/WebCore/loader/ImageLoader.cpp 2011-05-29 02:46:00 UTC (rev 87632)
+++ trunk/Source/WebCore/loader/ImageLoader.cpp 2011-05-29 04:42:18 UTC (rev 87633)
@@ -225,7 +225,7 @@
void ImageLoader::notifyFinished(CachedResource* resource)
{
ASSERT(m_failedLoadURL.isEmpty());
- ASSERT_UNUSED(m_image, resource == m_image.get());
+ ASSERT(resource == m_image.get());
m_imageComplete = true;
if (haveFiredBeforeLoadEvent())
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes