Substitute CrossOriginPreflightResultCache::clear() for CrossOriginPreflightResultCac...
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 7 May 2018 17:38:25 +0000 (17:38 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 7 May 2018 17:38:25 +0000 (17:38 +0000)
https://bugs.webkit.org/show_bug.cgi?id=185170

Reviewed by Per Arne Vollan.

Source/WebCore:

Rename CrossOriginPreflightResultCache::empty() to CrossOriginPreflightResultCache::clear() make
it consistent with the terminology we use in WebKit to signify a function that clears a collection.
A member function named "empty" is expected to return an instance of a class in its "empty state".
For example, StringImpl::empty() returns a StringImpl instance that represents the empty string.
However CrossOriginPreflightResultCache::empty() clears out the cache in-place. We should rename
this function to better describe its purpose.

* loader/CrossOriginPreflightResultCache.cpp:
(WebCore::CrossOriginPreflightResultCache::clear):
(WebCore::CrossOriginPreflightResultCache::empty): Deleted.
* loader/CrossOriginPreflightResultCache.h:

Source/WebKit:

* WebProcess/WebProcess.cpp:
(WebKit::WebProcess::clearResourceCaches):
(WebKit::WebProcess::deleteWebsiteData):

Source/WebKitLegacy/mac:

* Misc/WebCache.mm:
(+[WebCache empty]):

Source/WebKitLegacy/win:

* WebCache.cpp:
(WebCache::empty):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@231442 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/loader/CrossOriginPreflightResultCache.cpp
Source/WebCore/loader/CrossOriginPreflightResultCache.h
Source/WebKit/ChangeLog
Source/WebKit/WebProcess/WebProcess.cpp
Source/WebKitLegacy/mac/ChangeLog
Source/WebKitLegacy/mac/Misc/WebCache.mm
Source/WebKitLegacy/win/ChangeLog
Source/WebKitLegacy/win/WebCache.cpp

index e5d587c..690ecb9 100644 (file)
@@ -1,3 +1,22 @@
+2018-05-07  Daniel Bates  <dabates@apple.com>
+
+        Substitute CrossOriginPreflightResultCache::clear() for CrossOriginPreflightResultCache::empty()
+        https://bugs.webkit.org/show_bug.cgi?id=185170
+
+        Reviewed by Per Arne Vollan.
+
+        Rename CrossOriginPreflightResultCache::empty() to CrossOriginPreflightResultCache::clear() make
+        it consistent with the terminology we use in WebKit to signify a function that clears a collection.
+        A member function named "empty" is expected to return an instance of a class in its "empty state".
+        For example, StringImpl::empty() returns a StringImpl instance that represents the empty string.
+        However CrossOriginPreflightResultCache::empty() clears out the cache in-place. We should rename
+        this function to better describe its purpose.
+
+        * loader/CrossOriginPreflightResultCache.cpp:
+        (WebCore::CrossOriginPreflightResultCache::clear):
+        (WebCore::CrossOriginPreflightResultCache::empty): Deleted.
+        * loader/CrossOriginPreflightResultCache.h:
+
 2018-05-06  Dean Jackson  <dino@apple.com>
 
         WebGL: Reset simulated values after validation fails
index 7087ab0..632b68d 100644 (file)
@@ -145,7 +145,7 @@ bool CrossOriginPreflightResultCache::canSkipPreflight(const String& origin, con
     return false;
 }
 
-void CrossOriginPreflightResultCache::empty()
+void CrossOriginPreflightResultCache::clear()
 {
     ASSERT(isMainThread());
     m_preflightHashMap.clear();
index af2a5f1..dc07130 100644 (file)
@@ -62,14 +62,11 @@ private:
 
 class CrossOriginPreflightResultCache {
     WTF_MAKE_NONCOPYABLE(CrossOriginPreflightResultCache); WTF_MAKE_FAST_ALLOCATED;
-
 public:
     WEBCORE_EXPORT static CrossOriginPreflightResultCache& singleton();
-
     WEBCORE_EXPORT void appendEntry(const String& origin, const URL&, std::unique_ptr<CrossOriginPreflightResultCacheItem>);
     WEBCORE_EXPORT bool canSkipPreflight(const String& origin, const URL&, StoredCredentialsPolicy, const String& method, const HTTPHeaderMap& requestHeaders);
-
-    WEBCORE_EXPORT void empty();
+    WEBCORE_EXPORT void clear();
 
 private:
     friend NeverDestroyed<CrossOriginPreflightResultCache>;
index 5127ae1..b920026 100644 (file)
@@ -1,3 +1,14 @@
+2018-05-07  Daniel Bates  <dabates@apple.com>
+
+        Substitute CrossOriginPreflightResultCache::clear() for CrossOriginPreflightResultCache::empty()
+        https://bugs.webkit.org/show_bug.cgi?id=185170
+
+        Reviewed by Per Arne Vollan.
+
+        * WebProcess/WebProcess.cpp:
+        (WebKit::WebProcess::clearResourceCaches):
+        (WebKit::WebProcess::deleteWebsiteData):
+
 2018-05-07  Brian Burg  <bburg@apple.com>
 
         Web Inspector: opt out of process swap on navigation if a Web Inspector frontend is connected
index 982ddfd..c20db66 100644 (file)
@@ -757,7 +757,7 @@ void WebProcess::clearResourceCaches(ResourceCachesToClear resourceCachesToClear
     MemoryCache::singleton().evictResources();
 
     // Empty the cross-origin preflight cache.
-    CrossOriginPreflightResultCache::singleton().empty();
+    CrossOriginPreflightResultCache::singleton().clear();
 }
 
 static inline void addCaseFoldedCharacters(StringHasher& hasher, const String& string)
@@ -1288,7 +1288,7 @@ void WebProcess::deleteWebsiteData(PAL::SessionID sessionID, OptionSet<WebsiteDa
         PageCache::singleton().pruneToSizeNow(0, PruningReason::None);
         MemoryCache::singleton().evictResources(sessionID);
 
-        CrossOriginPreflightResultCache::singleton().empty();
+        CrossOriginPreflightResultCache::singleton().clear();
     }
 
     if (websiteDataTypes.contains(WebsiteDataType::Credentials)) {
index 3be1b62..4483a51 100644 (file)
@@ -1,3 +1,13 @@
+2018-05-07  Daniel Bates  <dabates@apple.com>
+
+        Substitute CrossOriginPreflightResultCache::clear() for CrossOriginPreflightResultCache::empty()
+        https://bugs.webkit.org/show_bug.cgi?id=185170
+
+        Reviewed by Per Arne Vollan.
+
+        * Misc/WebCache.mm:
+        (+[WebCache empty]):
+
 2018-05-04  Timothy Hatcher  <timothy@apple.com>
 
         Deprecate legacy WebView and friends
index 2419097..1bd56cb 100644 (file)
     webApplicationCacheStorage().empty();
 
     // Empty the Cross-Origin Preflight cache
-    WebCore::CrossOriginPreflightResultCache::singleton().empty();
+    WebCore::CrossOriginPreflightResultCache::singleton().clear();
 }
 
 #if PLATFORM(IOS)
index 7073627..f2cce84 100644 (file)
@@ -1,3 +1,13 @@
+2018-05-07  Daniel Bates  <dabates@apple.com>
+
+        Substitute CrossOriginPreflightResultCache::clear() for CrossOriginPreflightResultCache::empty()
+        https://bugs.webkit.org/show_bug.cgi?id=185170
+
+        Reviewed by Per Arne Vollan.
+
+        * WebCache.cpp:
+        (WebCache::empty):
+
 2018-04-27  Chris Dumez  <cdumez@apple.com>
 
         Use WindowProxy instead of DOMWindow in our IDL
index 75632df..4bf8ef0 100644 (file)
@@ -223,7 +223,7 @@ HRESULT WebCache::empty()
     WebApplicationCache::storage().empty();
 
     // Empty the Cross-Origin Preflight cache
-    WebCore::CrossOriginPreflightResultCache::singleton().empty();
+    WebCore::CrossOriginPreflightResultCache::singleton().clear();
 
     return S_OK;
 }