Title: [213048] releases/WebKitGTK/webkit-2.16/Source/WebKit2
- Revision
- 213048
- Author
- [email protected]
- Date
- 2017-02-27 05:25:35 -0800 (Mon, 27 Feb 2017)
Log Message
Merge r212816 - NetworkProcess: Stop disabling buffering when NETWORK_CACHE is disabled in build
https://bugs.webkit.org/show_bug.cgi?id=168637
Reviewed by Alex Christensen.
It was added in r193752 as part of bug #137692 to fix an infinite loop in network process that happened in EFL
because they didn't enable the network cache at that time. I think that was actually a workaround, and it was
added without any comment so it has stayed there even when EFL enabled disk cache, and now that is gone. Looking
at current code I see no reason why buffering can't work with the disk cache disabled, so I think it's time to
remove that workaround.
* WebProcess/Network/WebLoaderStrategy.cpp:
(WebKit::maximumBufferingTime):
Modified Paths
Diff
Modified: releases/WebKitGTK/webkit-2.16/Source/WebKit2/ChangeLog (213047 => 213048)
--- releases/WebKitGTK/webkit-2.16/Source/WebKit2/ChangeLog 2017-02-27 13:24:53 UTC (rev 213047)
+++ releases/WebKitGTK/webkit-2.16/Source/WebKit2/ChangeLog 2017-02-27 13:25:35 UTC (rev 213048)
@@ -1,3 +1,19 @@
+2017-02-22 Carlos Garcia Campos <[email protected]>
+
+ NetworkProcess: Stop disabling buffering when NETWORK_CACHE is disabled in build
+ https://bugs.webkit.org/show_bug.cgi?id=168637
+
+ Reviewed by Alex Christensen.
+
+ It was added in r193752 as part of bug #137692 to fix an infinite loop in network process that happened in EFL
+ because they didn't enable the network cache at that time. I think that was actually a workaround, and it was
+ added without any comment so it has stayed there even when EFL enabled disk cache, and now that is gone. Looking
+ at current code I see no reason why buffering can't work with the disk cache disabled, so I think it's time to
+ remove that workaround.
+
+ * WebProcess/Network/WebLoaderStrategy.cpp:
+ (WebKit::maximumBufferingTime):
+
2017-02-21 Maureen Daum <[email protected]>
Check what LocalStorage data exists in deleteDatabasesModifiedSince() before attempting deletion.
Modified: releases/WebKitGTK/webkit-2.16/Source/WebKit2/WebProcess/Network/WebLoaderStrategy.cpp (213047 => 213048)
--- releases/WebKitGTK/webkit-2.16/Source/WebKit2/WebProcess/Network/WebLoaderStrategy.cpp 2017-02-27 13:24:53 UTC (rev 213047)
+++ releases/WebKitGTK/webkit-2.16/Source/WebKit2/WebProcess/Network/WebLoaderStrategy.cpp 2017-02-27 13:25:35 UTC (rev 213048)
@@ -98,10 +98,6 @@
static std::chrono::milliseconds maximumBufferingTime(CachedResource* resource)
{
-#if !ENABLE(NETWORK_CACHE)
- return 0ms;
-#endif
-
if (!resource)
return 0ms;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
