Title: [227205] trunk/Source
- Revision
- 227205
- Author
- [email protected]
- Date
- 2018-01-19 09:46:06 -0800 (Fri, 19 Jan 2018)
Log Message
Remove unused WebViewPrivate _allowCookies
https://bugs.webkit.org/show_bug.cgi?id=181812
Patch by Alex Christensen <[email protected]> on 2018-01-19
Reviewed by Tim Horton.
Source/WebCore:
This SPI was in the original iOS upstreaming and has not been used in many years.
* platform/network/ResourceRequestBase.cpp:
(WebCore::ResourceRequestBase::setDefaultAllowCookies): Deleted.
(WebCore::ResourceRequestBase::defaultAllowCookies): Deleted.
* platform/network/ResourceRequestBase.h:
(WebCore::ResourceRequestBase::ResourceRequestBase):
Source/WebKitLegacy/mac:
* WebView/WebView.mm:
(+[WebView _setAllowCookies:]): Deleted.
(+[WebView _allowCookies]): Deleted.
* WebView/WebViewPrivate.h:
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (227204 => 227205)
--- trunk/Source/WebCore/ChangeLog 2018-01-19 17:45:06 UTC (rev 227204)
+++ trunk/Source/WebCore/ChangeLog 2018-01-19 17:46:06 UTC (rev 227205)
@@ -1,3 +1,18 @@
+2018-01-19 Alex Christensen <[email protected]>
+
+ Remove unused WebViewPrivate _allowCookies
+ https://bugs.webkit.org/show_bug.cgi?id=181812
+
+ Reviewed by Tim Horton.
+
+ This SPI was in the original iOS upstreaming and has not been used in many years.
+
+ * platform/network/ResourceRequestBase.cpp:
+ (WebCore::ResourceRequestBase::setDefaultAllowCookies): Deleted.
+ (WebCore::ResourceRequestBase::defaultAllowCookies): Deleted.
+ * platform/network/ResourceRequestBase.h:
+ (WebCore::ResourceRequestBase::ResourceRequestBase):
+
2018-01-18 Brady Eidson <[email protected]>
Make in-process MessagePorts be (mostly) asynchronous
Modified: trunk/Source/WebCore/platform/network/ResourceRequestBase.cpp (227204 => 227205)
--- trunk/Source/WebCore/platform/network/ResourceRequestBase.cpp 2018-01-19 17:45:06 UTC (rev 227204)
+++ trunk/Source/WebCore/platform/network/ResourceRequestBase.cpp 2018-01-19 17:46:06 UTC (rev 227205)
@@ -41,10 +41,6 @@
double ResourceRequestBase::s_defaultTimeoutInterval = 0;
#endif
-#if PLATFORM(IOS)
-bool ResourceRequestBase::s_defaultAllowCookies = true;
-#endif
-
inline const ResourceRequest& ResourceRequestBase::asResourceRequest() const
{
return *static_cast<const ResourceRequest*>(this);
@@ -644,18 +640,6 @@
}
#endif
-#if PLATFORM(IOS)
-void ResourceRequestBase::setDefaultAllowCookies(bool allowCookies)
-{
- s_defaultAllowCookies = allowCookies;
-}
-
-bool ResourceRequestBase::defaultAllowCookies()
-{
- return s_defaultAllowCookies;
-}
-#endif
-
void ResourceRequestBase::setCachePartition(const String& cachePartition)
{
#if ENABLE(CACHE_PARTITIONING)
Modified: trunk/Source/WebCore/platform/network/ResourceRequestBase.h (227204 => 227205)
--- trunk/Source/WebCore/platform/network/ResourceRequestBase.h 2018-01-19 17:45:06 UTC (rev 227204)
+++ trunk/Source/WebCore/platform/network/ResourceRequestBase.h 2018-01-19 17:46:06 UTC (rev 227205)
@@ -168,11 +168,6 @@
WEBCORE_EXPORT static double defaultTimeoutInterval(); // May return 0 when using platform default.
WEBCORE_EXPORT static void setDefaultTimeoutInterval(double);
-#if PLATFORM(IOS)
- WEBCORE_EXPORT static bool defaultAllowCookies();
- WEBCORE_EXPORT static void setDefaultAllowCookies(bool);
-#endif
-
WEBCORE_EXPORT static bool equal(const ResourceRequest&, const ResourceRequest&);
protected:
@@ -188,11 +183,7 @@
, m_timeoutInterval(s_defaultTimeoutInterval)
, m_httpMethod(ASCIILiteral("GET"))
, m_cachePolicy(policy)
-#if !PLATFORM(IOS)
, m_allowCookies(true)
-#else
- , m_allowCookies(ResourceRequestBase::defaultAllowCookies())
-#endif
, m_resourceRequestUpdated(true)
, m_resourceRequestBodyUpdated(true)
{
@@ -230,9 +221,6 @@
const ResourceRequest& asResourceRequest() const;
WEBCORE_EXPORT static double s_defaultTimeoutInterval;
-#if PLATFORM(IOS)
- static bool s_defaultAllowCookies;
-#endif
};
bool equalIgnoringHeaderFields(const ResourceRequestBase&, const ResourceRequestBase&);
Modified: trunk/Source/WebKitLegacy/mac/ChangeLog (227204 => 227205)
--- trunk/Source/WebKitLegacy/mac/ChangeLog 2018-01-19 17:45:06 UTC (rev 227204)
+++ trunk/Source/WebKitLegacy/mac/ChangeLog 2018-01-19 17:46:06 UTC (rev 227205)
@@ -1,3 +1,15 @@
+2018-01-19 Alex Christensen <[email protected]>
+
+ Remove unused WebViewPrivate _allowCookies
+ https://bugs.webkit.org/show_bug.cgi?id=181812
+
+ Reviewed by Tim Horton.
+
+ * WebView/WebView.mm:
+ (+[WebView _setAllowCookies:]): Deleted.
+ (+[WebView _allowCookies]): Deleted.
+ * WebView/WebViewPrivate.h:
+
2018-01-18 Alexey Proskuryakov <[email protected]>
Build fix, remove a call to -[NSWindow setOneShot:]
Modified: trunk/Source/WebKitLegacy/mac/WebView/WebView.mm (227204 => 227205)
--- trunk/Source/WebKitLegacy/mac/WebView/WebView.mm 2018-01-19 17:45:06 UTC (rev 227204)
+++ trunk/Source/WebKitLegacy/mac/WebView/WebView.mm 2018-01-19 17:46:06 UTC (rev 227205)
@@ -2150,16 +2150,6 @@
webResourceLoadScheduler().resumePendingRequests();
}
-+ (void)_setAllowCookies:(BOOL)allow
-{
- ResourceRequestBase::setDefaultAllowCookies(allow);
-}
-
-+ (BOOL)_allowCookies
-{
- return ResourceRequestBase::defaultAllowCookies();
-}
-
+ (BOOL)_isUnderMemoryPressure
{
return MemoryPressureHandler::singleton().isUnderMemoryPressure();
Modified: trunk/Source/WebKitLegacy/mac/WebView/WebViewPrivate.h (227204 => 227205)
--- trunk/Source/WebKitLegacy/mac/WebView/WebViewPrivate.h 2018-01-19 17:45:06 UTC (rev 227204)
+++ trunk/Source/WebKitLegacy/mac/WebView/WebViewPrivate.h 2018-01-19 17:46:06 UTC (rev 227205)
@@ -466,8 +466,6 @@
- (void)_setResourceLoadSchedulerSuspended:(BOOL)suspend;
+ (void)_setTileCacheLayerPoolCapacity:(unsigned)capacity;
-+ (void)_setAllowCookies:(BOOL)allow;
-+ (BOOL)_allowCookies;
+ (void)_releaseMemoryNow;
- (void)_replaceCurrentHistoryItem:(WebHistoryItem *)item;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes