Title: [135328] trunk/Source/WebCore
- Revision
- 135328
- Author
- [email protected]
- Date
- 2012-11-20 16:12:55 -0800 (Tue, 20 Nov 2012)
Log Message
Unreviewed. Chromium build fix.
Removed unnecessary mutex unlock that triggered crash on Linux debug build.
Chromium Linux Debug layout tests should be green:
platform/chromium/virtual/deferred/fast/images/image-in-map.html
* platform/graphics/chromium/LazyDecodingPixelRef.cpp:
(WebCore::LazyDecodingPixelRef::onLockPixels):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (135327 => 135328)
--- trunk/Source/WebCore/ChangeLog 2012-11-21 00:07:02 UTC (rev 135327)
+++ trunk/Source/WebCore/ChangeLog 2012-11-21 00:12:55 UTC (rev 135328)
@@ -1,3 +1,15 @@
+2012-11-20 Alpha Lam <[email protected]>
+
+ Unreviewed. Chromium build fix.
+
+ Removed unnecessary mutex unlock that triggered crash on Linux debug build.
+
+ Chromium Linux Debug layout tests should be green:
+ platform/chromium/virtual/deferred/fast/images/image-in-map.html
+
+ * platform/graphics/chromium/LazyDecodingPixelRef.cpp:
+ (WebCore::LazyDecodingPixelRef::onLockPixels):
+
2012-11-20 Ryosuke Niwa <[email protected]>
Fix another typo. I need to checking that type() != NodeListCollectionType,
Modified: trunk/Source/WebCore/platform/graphics/chromium/LazyDecodingPixelRef.cpp (135327 => 135328)
--- trunk/Source/WebCore/platform/graphics/chromium/LazyDecodingPixelRef.cpp 2012-11-21 00:07:02 UTC (rev 135327)
+++ trunk/Source/WebCore/platform/graphics/chromium/LazyDecodingPixelRef.cpp 2012-11-21 00:12:55 UTC (rev 135328)
@@ -60,10 +60,8 @@
m_mutex.lock();
ASSERT(m_lockedBitmap.isNull());
m_lockedBitmap = m_frameGenerator->decodeAndScale(m_scaledSize, m_scaledSubset);
- if (m_lockedBitmap.isNull()) {
- m_mutex.unlock();
+ if (m_lockedBitmap.isNull())
return 0;
- }
m_lockedBitmap.lockPixels();
return m_lockedBitmap.getAddr(0, 0);
}
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes
