Title: [137689] trunk/Source/WebKit2
- Revision
- 137689
- Author
- [email protected]
- Date
- 2012-12-13 17:29:03 -0800 (Thu, 13 Dec 2012)
Log Message
Resource loads sometimes stall
https://bugs.webkit.org/show_bug.cgi?id=104976
Reviewed by Anders Carlsson.
Quick and dirty partial fix.
* Shared/BlockingResponseMap.h: (BlockingResponseMap::didReceiveResponse):
Wake up all threads, so that non-sequential responses don't break us. This is still
horribly inefficient,but should improve behavior quite a bit.
Modified Paths
Diff
Modified: trunk/Source/WebKit2/ChangeLog (137688 => 137689)
--- trunk/Source/WebKit2/ChangeLog 2012-12-14 01:15:08 UTC (rev 137688)
+++ trunk/Source/WebKit2/ChangeLog 2012-12-14 01:29:03 UTC (rev 137689)
@@ -1,3 +1,16 @@
+2012-12-13 Alexey Proskuryakov <[email protected]>
+
+ Resource loads sometimes stall
+ https://bugs.webkit.org/show_bug.cgi?id=104976
+
+ Reviewed by Anders Carlsson.
+
+ Quick and dirty partial fix.
+
+ * Shared/BlockingResponseMap.h: (BlockingResponseMap::didReceiveResponse):
+ Wake up all threads, so that non-sequential responses don't break us. This is still
+ horribly inefficient,but should improve behavior quite a bit.
+
2012-12-13 Seokju Kwon <[email protected]>
Fix unused parameter compile warnings
Modified: trunk/Source/WebKit2/Shared/BlockingResponseMap.h (137688 => 137689)
--- trunk/Source/WebKit2/Shared/BlockingResponseMap.h 2012-12-14 01:15:08 UTC (rev 137688)
+++ trunk/Source/WebKit2/Shared/BlockingResponseMap.h 2012-12-14 01:29:03 UTC (rev 137689)
@@ -54,7 +54,8 @@
ASSERT(!m_responses.contains(requestID));
m_responses.set(requestID, response);
- m_condition.signal();
+ // FIXME (NetworkProcess): Waking up all threads is quite inefficient.
+ m_condition.broadcast();
}
private:
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes
