Title: [235063] releases/WebKitGTK/webkit-2.22/Source/WebCore
Revision
235063
Author
carlo...@webkit.org
Date
2018-08-20 05:54:47 -0700 (Mon, 20 Aug 2018)

Log Message

Merge r235000 - 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: releases/WebKitGTK/webkit-2.22/Source/WebCore/ChangeLog (235062 => 235063)


--- releases/WebKitGTK/webkit-2.22/Source/WebCore/ChangeLog	2018-08-20 12:54:42 UTC (rev 235062)
+++ releases/WebKitGTK/webkit-2.22/Source/WebCore/ChangeLog	2018-08-20 12:54:47 UTC (rev 235063)
@@ -1,3 +1,17 @@
+2018-08-17  Brent Fulgham  <bfulg...@apple.com>
+
+        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  <rn...@webkit.org>
 
         Replace canBubble and cancelable booleans in Event by enum classes

Modified: releases/WebKitGTK/webkit-2.22/Source/WebCore/loader/ResourceLoadObserver.cpp (235062 => 235063)


--- releases/WebKitGTK/webkit-2.22/Source/WebCore/loader/ResourceLoadObserver.cpp	2018-08-20 12:54:42 UTC (rev 235062)
+++ releases/WebKitGTK/webkit-2.22/Source/WebCore/loader/ResourceLoadObserver.cpp	2018-08-20 12:54:47 UTC (rev 235063)
@@ -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
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to