Changeset 246807 in webkit
- Timestamp:
- Jun 25, 2019 2:08:23 PM (5 years ago)
- Location:
- trunk/Source
- Files:
-
- 18 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Source/WebCore/PAL/ChangeLog
r246616 r246807 1 2019-06-25 Alex Christensen <achristensen@webkit.org> 2 3 Make HTTPCookieAcceptPolicy an enum class 4 https://bugs.webkit.org/show_bug.cgi?id=199081 5 6 Reviewed by Michael Catanzaro 7 8 * pal/spi/cf/CFNetworkSPI.h: 9 1 10 2019-06-19 Devin Rousso <drousso@apple.com> 2 11 -
trunk/Source/WebCore/PAL/pal/spi/cf/CFNetworkSPI.h
r246616 r246807 101 101 typedef CFIndex CFURLRequestPriority; 102 102 typedef int CFHTTPCookieStorageAcceptPolicy; 103 104 CF_ENUM(CFHTTPCookieStorageAcceptPolicy) 105 { 106 CFHTTPCookieStorageAcceptPolicyAlways = 0, 107 CFHTTPCookieStorageAcceptPolicyNever = 1, 108 CFHTTPCookieStorageAcceptPolicyOnlyFromMainDocumentDomain = 2, 109 CFHTTPCookieStorageAcceptPolicyExclusivelyFromMainDocumentDomain = 3, 110 }; 103 111 104 112 #ifdef __BLOCKS__ -
trunk/Source/WebKit/ChangeLog
r246805 r246807 1 2019-06-25 Alex Christensen <achristensen@webkit.org> 2 3 Make HTTPCookieAcceptPolicy an enum class 4 https://bugs.webkit.org/show_bug.cgi?id=199081 5 6 Reviewed by Michael Catanzaro. 7 8 Doing so reveals some interesting bugs in the C API that are also fixed: 9 WKPreferencesGetIncrementalRenderingSuppressionTimeout was using its toAPI function to convert a double to a double because HTTPCookieAcceptPolicy used to be an unsigned integer. 10 toAPI(WebCore::MouseButton) was also using the toAPI(HTTPCookieAcceptPolicy) because HTTPCookieAcceptPolicy used to be an unsigned integer. 11 Yikes! Type safety is our friend. 12 13 * NetworkProcess/Cookies/WebCookieManager.messages.in: 14 * NetworkProcess/Cookies/mac/WebCookieManagerMac.mm: 15 (WebKit::toCFHTTPCookieStorageAcceptPolicy): 16 (WebKit::WebCookieManager::platformSetHTTPCookieAcceptPolicy): 17 (WebKit::WebCookieManager::platformGetHTTPCookieAcceptPolicy): 18 * Shared/API/c/WKSharedAPICast.h: 19 (WebKit::toAPI): 20 * Shared/HTTPCookieAcceptPolicy.h: 21 (): Deleted. 22 * UIProcess/API/C/WKAPICast.h: 23 (WebKit::toHTTPCookieAcceptPolicy): 24 (WebKit::toAPI): 25 * UIProcess/API/C/WKPreferences.cpp: 26 (WKPreferencesGetIncrementalRenderingSuppressionTimeout): 27 * UIProcess/API/Cocoa/WKProcessPool.mm: 28 (toHTTPCookieAcceptPolicy): 29 * UIProcess/WebCookieManagerProxy.cpp: 30 (WebKit::WebCookieManagerProxy::didGetHTTPCookieAcceptPolicy): 31 * UIProcess/WebCookieManagerProxy.h: 32 * UIProcess/WebCookieManagerProxy.messages.in: 33 1 34 2019-06-25 Tim Horton <timothy_horton@apple.com> 2 35 -
trunk/Source/WebKit/NetworkProcess/Cookies/WebCookieManager.messages.in
r244924 r246807 36 36 void DeleteAllCookiesModifiedSince(PAL::SessionID sessionID, WallTime time, WebKit::CallbackID callbackID) 37 37 38 void SetHTTPCookieAcceptPolicy( uint32_tpolicy, WebKit::OptionalCallbackID callbackID)38 void SetHTTPCookieAcceptPolicy(enum:uint8_t WebKit::HTTPCookieAcceptPolicy policy, WebKit::OptionalCallbackID callbackID) 39 39 void GetHTTPCookieAcceptPolicy(WebKit::CallbackID callbackID) 40 40 -
trunk/Source/WebKit/NetworkProcess/Cookies/curl/WebCookieManagerCurl.cpp
r242517 r246807 38 38 CookieAcceptPolicy curlPolicy = CookieAcceptPolicy::OnlyFromMainDocumentDomain; 39 39 switch (policy) { 40 case HTTPCookieAcceptPolicy Always:40 case HTTPCookieAcceptPolicy::AlwaysAccept: 41 41 curlPolicy = CookieAcceptPolicy::Always; 42 42 break; 43 case HTTPCookieAcceptPolicy Never:43 case HTTPCookieAcceptPolicy::Never: 44 44 curlPolicy = CookieAcceptPolicy::Never; 45 45 break; 46 case HTTPCookieAcceptPolicy OnlyFromMainDocumentDomain:46 case HTTPCookieAcceptPolicy::OnlyFromMainDocumentDomain: 47 47 curlPolicy = CookieAcceptPolicy::OnlyFromMainDocumentDomain; 48 48 break; 49 case HTTPCookieAcceptPolicy ExclusivelyFromMainDocumentDomain:49 case HTTPCookieAcceptPolicy::ExclusivelyFromMainDocumentDomain: 50 50 curlPolicy = CookieAcceptPolicy::ExclusivelyFromMainDocumentDomain; 51 51 break; … … 62 62 switch (networkStorageSession.cookieStorage().cookieAcceptPolicy(networkStorageSession)) { 63 63 case CookieAcceptPolicy::Always: 64 return HTTPCookieAcceptPolicy Always;64 return HTTPCookieAcceptPolicy::AlwaysAccept; 65 65 case CookieAcceptPolicy::Never: 66 return HTTPCookieAcceptPolicy Never;66 return HTTPCookieAcceptPolicy::Never; 67 67 case CookieAcceptPolicy::OnlyFromMainDocumentDomain: 68 return HTTPCookieAcceptPolicy OnlyFromMainDocumentDomain;68 return HTTPCookieAcceptPolicy::OnlyFromMainDocumentDomain; 69 69 case CookieAcceptPolicy::ExclusivelyFromMainDocumentDomain: 70 return HTTPCookieAcceptPolicy ExclusivelyFromMainDocumentDomain;70 return HTTPCookieAcceptPolicy::ExclusivelyFromMainDocumentDomain; 71 71 } 72 72 73 73 ASSERT_NOT_REACHED(); 74 return HTTPCookieAcceptPolicy OnlyFromMainDocumentDomain;74 return HTTPCookieAcceptPolicy::OnlyFromMainDocumentDomain; 75 75 } 76 76 -
trunk/Source/WebKit/NetworkProcess/Cookies/mac/WebCookieManagerMac.mm
r240292 r246807 36 36 using namespace WebCore; 37 37 38 static CFHTTPCookieStorageAcceptPolicy toCFHTTPCookieStorageAcceptPolicy(HTTPCookieAcceptPolicy policy) 39 { 40 switch (policy) { 41 case HTTPCookieAcceptPolicy::AlwaysAccept: 42 return CFHTTPCookieStorageAcceptPolicyAlways; 43 case HTTPCookieAcceptPolicy::Never: 44 return CFHTTPCookieStorageAcceptPolicyNever; 45 case HTTPCookieAcceptPolicy::OnlyFromMainDocumentDomain: 46 return CFHTTPCookieStorageAcceptPolicyOnlyFromMainDocumentDomain; 47 case HTTPCookieAcceptPolicy::ExclusivelyFromMainDocumentDomain: 48 return CFHTTPCookieStorageAcceptPolicyExclusivelyFromMainDocumentDomain; 49 } 50 ASSERT_NOT_REACHED(); 51 52 return CFHTTPCookieStorageAcceptPolicyAlways; 53 } 54 38 55 void WebCookieManager::platformSetHTTPCookieAcceptPolicy(HTTPCookieAcceptPolicy policy) 39 56 { … … 44 61 m_process.forEachNetworkStorageSession([&] (const auto& networkStorageSession) { 45 62 if (auto cookieStorage = networkStorageSession.cookieStorage()) 46 CFHTTPCookieStorageSetCookieAcceptPolicy(cookieStorage.get(), policy);63 CFHTTPCookieStorageSetCookieAcceptPolicy(cookieStorage.get(), toCFHTTPCookieStorageAcceptPolicy(policy)); 47 64 }); 48 65 } … … 52 69 ASSERT(hasProcessPrivilege(ProcessPrivilege::CanAccessRawCookies)); 53 70 54 return [[NSHTTPCookieStorage sharedHTTPCookieStorage] cookieAcceptPolicy]; 71 switch ([[NSHTTPCookieStorage sharedHTTPCookieStorage] cookieAcceptPolicy]) { 72 case NSHTTPCookieAcceptPolicyAlways: 73 return HTTPCookieAcceptPolicy::AlwaysAccept; 74 case NSHTTPCookieAcceptPolicyOnlyFromMainDocumentDomain: 75 return HTTPCookieAcceptPolicy::OnlyFromMainDocumentDomain; 76 case NSHTTPCookieAcceptPolicyExclusivelyFromMainDocumentDomain: 77 return HTTPCookieAcceptPolicy::ExclusivelyFromMainDocumentDomain; 78 case NSHTTPCookieAcceptPolicyNever: 79 return HTTPCookieAcceptPolicy::Never; 80 } 81 82 ASSERT_NOT_REACHED(); 83 return HTTPCookieAcceptPolicy::AlwaysAccept; 55 84 } 56 85 -
trunk/Source/WebKit/NetworkProcess/Cookies/soup/WebCookieManagerSoup.cpp
r240858 r246807 42 42 SoupCookieJarAcceptPolicy soupPolicy = SOUP_COOKIE_JAR_ACCEPT_NO_THIRD_PARTY; 43 43 switch (policy) { 44 case HTTPCookieAcceptPolicy Always:44 case HTTPCookieAcceptPolicy::AlwaysAccept: 45 45 soupPolicy = SOUP_COOKIE_JAR_ACCEPT_ALWAYS; 46 46 break; 47 case HTTPCookieAcceptPolicy Never:47 case HTTPCookieAcceptPolicy::Never: 48 48 soupPolicy = SOUP_COOKIE_JAR_ACCEPT_NEVER; 49 49 break; 50 case HTTPCookieAcceptPolicy OnlyFromMainDocumentDomain:50 case HTTPCookieAcceptPolicy::OnlyFromMainDocumentDomain: 51 51 soupPolicy = SOUP_COOKIE_JAR_ACCEPT_NO_THIRD_PARTY; 52 52 break; … … 62 62 switch (soup_cookie_jar_get_accept_policy(m_process.defaultStorageSession().cookieStorage())) { 63 63 case SOUP_COOKIE_JAR_ACCEPT_ALWAYS: 64 return HTTPCookieAcceptPolicy Always;64 return HTTPCookieAcceptPolicy::AlwaysAccept; 65 65 case SOUP_COOKIE_JAR_ACCEPT_NEVER: 66 return HTTPCookieAcceptPolicy Never;66 return HTTPCookieAcceptPolicy::Never; 67 67 case SOUP_COOKIE_JAR_ACCEPT_NO_THIRD_PARTY: 68 return HTTPCookieAcceptPolicy OnlyFromMainDocumentDomain;68 return HTTPCookieAcceptPolicy::OnlyFromMainDocumentDomain; 69 69 } 70 70 71 71 ASSERT_NOT_REACHED(); 72 return HTTPCookieAcceptPolicy OnlyFromMainDocumentDomain;72 return HTTPCookieAcceptPolicy::OnlyFromMainDocumentDomain; 73 73 } 74 74 -
trunk/Source/WebKit/NetworkProcess/NetworkProcessCreationParameters.h
r246763 r246807 87 87 88 88 #if USE(SOUP) 89 HTTPCookieAcceptPolicy cookieAcceptPolicy { HTTPCookieAcceptPolicy Always};89 HTTPCookieAcceptPolicy cookieAcceptPolicy { HTTPCookieAcceptPolicy::AlwaysAccept }; 90 90 bool ignoreTLSErrors { false }; 91 91 Vector<String> languages; -
trunk/Source/WebKit/Shared/API/c/WKSharedAPICast.h
r241000 r246807 56 56 #include <WebCore/IntRect.h> 57 57 #include <WebCore/LayoutMilestone.h> 58 #include <WebCore/PlatformMouseEvent.h> 58 59 #include <WebCore/SecurityOrigin.h> 59 60 #include <WebCore/UserContentTypes.h> … … 313 314 break; 314 315 case WebMouseEvent::RightButton: 316 wkMouseButton = kWKEventMouseButtonRightButton; 317 break; 318 } 319 320 return wkMouseButton; 321 } 322 323 inline WKEventMouseButton toAPI(WebCore::MouseButton mouseButton) 324 { 325 WKEventMouseButton wkMouseButton = kWKEventMouseButtonNoButton; 326 327 switch (mouseButton) { 328 case WebCore::MouseButton::NoButton: 329 wkMouseButton = kWKEventMouseButtonNoButton; 330 break; 331 case WebCore::MouseButton::LeftButton: 332 wkMouseButton = kWKEventMouseButtonLeftButton; 333 break; 334 case WebCore::MouseButton::MiddleButton: 335 wkMouseButton = kWKEventMouseButtonMiddleButton; 336 break; 337 case WebCore::MouseButton::RightButton: 315 338 wkMouseButton = kWKEventMouseButtonRightButton; 316 339 break; -
trunk/Source/WebKit/Shared/HTTPCookieAcceptPolicy.h
r162166 r246807 24 24 */ 25 25 26 #ifndef HTTPCookieAcceptPolicy_h 27 #define HTTPCookieAcceptPolicy_h 26 #pragma once 28 27 29 28 namespace WebKit { 30 29 31 enum {32 HTTPCookieAcceptPolicyAlways= 0,33 HTTPCookieAcceptPolicyNever = 1,34 HTTPCookieAcceptPolicyOnlyFromMainDocumentDomain = 2,35 HTTPCookieAcceptPolicyExclusivelyFromMainDocumentDomain = 3,30 enum class HTTPCookieAcceptPolicy : uint8_t { 31 AlwaysAccept = 0, 32 Never = 1, 33 OnlyFromMainDocumentDomain = 2, 34 ExclusivelyFromMainDocumentDomain = 3, 36 35 }; 37 typedef unsigned HTTPCookieAcceptPolicy;38 36 39 37 } // namespace WebKit 40 38 41 #endif // HTTPCookieAcceptPolicy_h 39 namespace WTF { 40 41 template<> struct EnumTraits<WebKit::HTTPCookieAcceptPolicy> { 42 using values = EnumValues< 43 WebKit::HTTPCookieAcceptPolicy, 44 WebKit::HTTPCookieAcceptPolicy::AlwaysAccept, 45 WebKit::HTTPCookieAcceptPolicy::Never, 46 WebKit::HTTPCookieAcceptPolicy::OnlyFromMainDocumentDomain, 47 WebKit::HTTPCookieAcceptPolicy::ExclusivelyFromMainDocumentDomain 48 >; 49 }; 50 51 } -
trunk/Source/WebKit/UIProcess/API/C/WKAPICast.h
r239671 r246807 368 368 switch (policy) { 369 369 case kWKHTTPCookieAcceptPolicyAlways: 370 return HTTPCookieAcceptPolicy Always;370 return HTTPCookieAcceptPolicy::AlwaysAccept; 371 371 case kWKHTTPCookieAcceptPolicyNever: 372 return HTTPCookieAcceptPolicy Never;372 return HTTPCookieAcceptPolicy::Never; 373 373 case kWKHTTPCookieAcceptPolicyOnlyFromMainDocumentDomain: 374 return HTTPCookieAcceptPolicy OnlyFromMainDocumentDomain;374 return HTTPCookieAcceptPolicy::OnlyFromMainDocumentDomain; 375 375 case kWKHTTPCookieAcceptPolicyExclusivelyFromMainDocumentDomain: 376 return HTTPCookieAcceptPolicy ExclusivelyFromMainDocumentDomain;377 } 378 379 ASSERT_NOT_REACHED(); 380 return HTTPCookieAcceptPolicy Always;376 return HTTPCookieAcceptPolicy::ExclusivelyFromMainDocumentDomain; 377 } 378 379 ASSERT_NOT_REACHED(); 380 return HTTPCookieAcceptPolicy::AlwaysAccept; 381 381 } 382 382 … … 384 384 { 385 385 switch (policy) { 386 case HTTPCookieAcceptPolicy Always:386 case HTTPCookieAcceptPolicy::AlwaysAccept: 387 387 return kWKHTTPCookieAcceptPolicyAlways; 388 case HTTPCookieAcceptPolicy Never:388 case HTTPCookieAcceptPolicy::Never: 389 389 return kWKHTTPCookieAcceptPolicyNever; 390 case HTTPCookieAcceptPolicy OnlyFromMainDocumentDomain:390 case HTTPCookieAcceptPolicy::OnlyFromMainDocumentDomain: 391 391 return kWKHTTPCookieAcceptPolicyOnlyFromMainDocumentDomain; 392 case HTTPCookieAcceptPolicy ExclusivelyFromMainDocumentDomain:392 case HTTPCookieAcceptPolicy::ExclusivelyFromMainDocumentDomain: 393 393 return kWKHTTPCookieAcceptPolicyExclusivelyFromMainDocumentDomain; 394 394 } -
trunk/Source/WebKit/UIProcess/API/C/WKPreferences.cpp
r246444 r246807 1308 1308 double WKPreferencesGetIncrementalRenderingSuppressionTimeout(WKPreferencesRef preferencesRef) 1309 1309 { 1310 return to API(toImpl(preferencesRef)->incrementalRenderingSuppressionTimeout());1310 return toImpl(preferencesRef)->incrementalRenderingSuppressionTimeout(); 1311 1311 } 1312 1312 -
trunk/Source/WebKit/UIProcess/API/Cocoa/WKProcessPool.mm
r245979 r246807 208 208 switch (static_cast<NSUInteger>(policy)) { 209 209 case NSHTTPCookieAcceptPolicyAlways: 210 return WebKit::HTTPCookieAcceptPolicy Always;210 return WebKit::HTTPCookieAcceptPolicy::AlwaysAccept; 211 211 case NSHTTPCookieAcceptPolicyNever: 212 return WebKit::HTTPCookieAcceptPolicy Never;212 return WebKit::HTTPCookieAcceptPolicy::Never; 213 213 case NSHTTPCookieAcceptPolicyOnlyFromMainDocumentDomain: 214 return WebKit::HTTPCookieAcceptPolicy OnlyFromMainDocumentDomain;214 return WebKit::HTTPCookieAcceptPolicy::OnlyFromMainDocumentDomain; 215 215 case NSHTTPCookieAcceptPolicyExclusivelyFromMainDocumentDomain: 216 return WebKit::HTTPCookieAcceptPolicy ExclusivelyFromMainDocumentDomain;216 return WebKit::HTTPCookieAcceptPolicy::ExclusivelyFromMainDocumentDomain; 217 217 } 218 218 219 219 ASSERT_NOT_REACHED(); 220 return WebKit::HTTPCookieAcceptPolicy Always;220 return WebKit::HTTPCookieAcceptPolicy::AlwaysAccept; 221 221 } 222 222 -
trunk/Source/WebKit/UIProcess/API/glib/WebKitCookieManager.cpp
r241903 r246807 91 91 { 92 92 switch (httpPolicy) { 93 case HTTPCookieAcceptPolicy Always:93 case HTTPCookieAcceptPolicy::AlwaysAccept: 94 94 return WEBKIT_COOKIE_POLICY_ACCEPT_ALWAYS; 95 case HTTPCookieAcceptPolicy Never:95 case HTTPCookieAcceptPolicy::Never: 96 96 return WEBKIT_COOKIE_POLICY_ACCEPT_NEVER; 97 case HTTPCookieAcceptPolicy OnlyFromMainDocumentDomain:97 case HTTPCookieAcceptPolicy::OnlyFromMainDocumentDomain: 98 98 return WEBKIT_COOKIE_POLICY_ACCEPT_NO_THIRD_PARTY; 99 99 default: … … 107 107 switch (kitPolicy) { 108 108 case WEBKIT_COOKIE_POLICY_ACCEPT_ALWAYS: 109 return HTTPCookieAcceptPolicy Always;109 return HTTPCookieAcceptPolicy::AlwaysAccept; 110 110 case WEBKIT_COOKIE_POLICY_ACCEPT_NEVER: 111 return HTTPCookieAcceptPolicy Never;111 return HTTPCookieAcceptPolicy::Never; 112 112 case WEBKIT_COOKIE_POLICY_ACCEPT_NO_THIRD_PARTY: 113 return HTTPCookieAcceptPolicy OnlyFromMainDocumentDomain;113 return HTTPCookieAcceptPolicy::OnlyFromMainDocumentDomain; 114 114 default: 115 115 ASSERT_NOT_REACHED(); 116 return HTTPCookieAcceptPolicy Always;116 return HTTPCookieAcceptPolicy::AlwaysAccept; 117 117 } 118 118 } -
trunk/Source/WebKit/UIProcess/WebCookieManagerProxy.cpp
r244924 r246807 259 259 } 260 260 261 void WebCookieManagerProxy::didGetHTTPCookieAcceptPolicy( uint32_tpolicy, WebKit::CallbackID callbackID)261 void WebCookieManagerProxy::didGetHTTPCookieAcceptPolicy(HTTPCookieAcceptPolicy policy, WebKit::CallbackID callbackID) 262 262 { 263 263 m_callbacks.take<HTTPCookieAcceptPolicyCallback>(callbackID)->performCallbackWithReturnValue(policy); -
trunk/Source/WebKit/UIProcess/WebCookieManagerProxy.h
r244924 r246807 110 110 111 111 void didGetHostnamesWithCookies(const Vector<String>&, WebKit::CallbackID); 112 void didGetHTTPCookieAcceptPolicy( uint32_t policy, WebKit::CallbackID);112 void didGetHTTPCookieAcceptPolicy(HTTPCookieAcceptPolicy, WebKit::CallbackID); 113 113 114 114 void didSetHTTPCookieAcceptPolicy(WebKit::CallbackID); -
trunk/Source/WebKit/UIProcess/WebCookieManagerProxy.messages.in
r220857 r246807 23 23 messages -> WebCookieManagerProxy { 24 24 DidGetHostnamesWithCookies(Vector<String> hostnames, WebKit::CallbackID callbackID); 25 DidGetHTTPCookieAcceptPolicy( uint32_tpolicy, WebKit::CallbackID callbackID);25 DidGetHTTPCookieAcceptPolicy(enum:uint8_t WebKit::HTTPCookieAcceptPolicy policy, WebKit::CallbackID callbackID); 26 26 27 27 DidSetHTTPCookieAcceptPolicy(WebKit::CallbackID callbackID); -
trunk/Source/WebKit/UIProcess/WebProcessPool.h
r246500 r246807 681 681 682 682 #if USE(SOUP) 683 HTTPCookieAcceptPolicy m_initialHTTPCookieAcceptPolicy { HTTPCookieAcceptPolicy OnlyFromMainDocumentDomain };683 HTTPCookieAcceptPolicy m_initialHTTPCookieAcceptPolicy { HTTPCookieAcceptPolicy::OnlyFromMainDocumentDomain }; 684 684 WebCore::SoupNetworkProxySettings m_networkProxySettings; 685 685 #endif
Note: See TracChangeset
for help on using the changeset viewer.