Title: [235000] trunk/Source/WebCore
- Revision
- 235000
- Author
- [email protected]
- Date
- 2018-08-17 17:09:32 -0700 (Fri, 17 Aug 2018)
Log Message
Remove Adobe SSO exception now that StorageAccess API is available
https://bugs.webkit.org/show_bug.cgi?id=188710
<rdar://problem/35056707>
Reviewed by Alex Christensen.
Remove custom quirk now that proper API exists to allow the same functions.
* loader/ResourceLoadObserver.cpp:
(WebCore::ResourceLoadObserver::logSubresourceLoading): Remove unneeded quirk.
(WebCore::resourceNeedsSSOQuirk): Deleted.
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (234999 => 235000)
--- trunk/Source/WebCore/ChangeLog 2018-08-18 00:00:25 UTC (rev 234999)
+++ trunk/Source/WebCore/ChangeLog 2018-08-18 00:09:32 UTC (rev 235000)
@@ -1,3 +1,17 @@
+2018-08-17 Brent Fulgham <[email protected]>
+
+ Remove Adobe SSO exception now that StorageAccess API is available
+ https://bugs.webkit.org/show_bug.cgi?id=188710
+ <rdar://problem/35056707>
+
+ Reviewed by Alex Christensen.
+
+ Remove custom quirk now that proper API exists to allow the same functions.
+
+ * loader/ResourceLoadObserver.cpp:
+ (WebCore::ResourceLoadObserver::logSubresourceLoading): Remove unneeded quirk.
+ (WebCore::resourceNeedsSSOQuirk): Deleted.
+
2018-08-17 Ryosuke Niwa <[email protected]>
Replace canBubble and cancelable booleans in Event by enum classes
Modified: trunk/Source/WebCore/loader/ResourceLoadObserver.cpp (234999 => 235000)
--- trunk/Source/WebCore/loader/ResourceLoadObserver.cpp 2018-08-18 00:00:25 UTC (rev 234999)
+++ trunk/Source/WebCore/loader/ResourceLoadObserver.cpp 2018-08-18 00:09:32 UTC (rev 235000)
@@ -104,15 +104,6 @@
return DeprecatedGlobalSettings::resourceLoadStatisticsEnabled() && !page->usesEphemeralSession() && m_notificationCallback;
}
-// FIXME: This quirk was added to address <rdar://problem/33325881> and should be removed once content is fixed.
-static bool resourceNeedsSSOQuirk(Page* page, const URL& url)
-{
- if (!shouldEnableSiteSpecificQuirks(page))
- return false;
-
- return equalIgnoringASCIICase(url.host(), "sp.auth.adobe.com");
-}
-
void ResourceLoadObserver::logSubresourceLoading(const Frame* frame, const ResourceRequest& newRequest, const ResourceResponse& redirectResponse)
{
ASSERT(frame->page());
@@ -139,9 +130,6 @@
if (areDomainsAssociated(page, targetPrimaryDomain, mainFramePrimaryDomain) || (isRedirect && areDomainsAssociated(page, targetPrimaryDomain, sourcePrimaryDomain)))
return;
- if (resourceNeedsSSOQuirk(page, targetURL))
- return;
-
bool shouldCallNotificationCallback = false;
{
auto& targetStatistics = ensureResourceStatisticsForPrimaryDomain(targetPrimaryDomain);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes