[webkit-changes] [254309] branches/safari-609-branch

Thu, 09 Jan 2020 15:48:16 -0800

Title: [254309] branches/safari-609-branch

Diff

Modified: branches/safari-609-branch/LayoutTests/ChangeLog (254308 => 254309)


--- branches/safari-609-branch/LayoutTests/ChangeLog	2020-01-09 23:33:51 UTC (rev 254308)
+++ branches/safari-609-branch/LayoutTests/ChangeLog	2020-01-09 23:47:02 UTC (rev 254309)
@@ -1,41 +1,3 @@
-2020-01-09  Alan Coon  <alanc...@apple.com>
-
-        Cherry-pick r254301. rdar://problem/58346124
-
-    REGRESSION (r253662): Large Data URLs are not being handled properly
-    https://bugs.webkit.org/show_bug.cgi?id=205979
-    <rdar://problem/58346124>
-    
-    Reviewed by Youenn Fablet.
-    
-    Source/WebCore:
-    
-    The URL size limitation added in r253662 was too low. We should bump it to handle
-    reasonable data URI sizes.
-    
-    Test: fast/url/data-url-large.html.
-    
-    * page/SecurityOrigin.cpp:
-    
-    LayoutTests:
-    
-    * fast/url/data-url-large-expected.txt: Added.
-    * fast/url/data-url-large.html: Added.
-    
-    
-    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@254301 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
-    2020-01-09  Brent Fulgham  <bfulg...@apple.com>
-
-            REGRESSION (r253662): Large Data URLs are not being handled properly
-            https://bugs.webkit.org/show_bug.cgi?id=205979
-            <rdar://problem/58346124>
-
-            Reviewed by Youenn Fablet.
-
-            * fast/url/data-url-large-expected.txt: Added.
-            * fast/url/data-url-large.html: Added.
-
 2020-01-07  Alan Coon  <alanc...@apple.com>
 
         Cherry-pick r254111. rdar://problem/58093690

Deleted: branches/safari-609-branch/LayoutTests/fast/url/data-url-large-expected.txt (254308 => 254309)


--- branches/safari-609-branch/LayoutTests/fast/url/data-url-large-expected.txt	2020-01-09 23:33:51 UTC (rev 254308)
+++ branches/safari-609-branch/LayoutTests/fast/url/data-url-large-expected.txt	2020-01-09 23:47:02 UTC (rev 254309)
@@ -1,5 +0,0 @@
-PASS Large URL handled properly.
-PASS successfullyParsed is true
-
-TEST COMPLETE
-

Deleted: branches/safari-609-branch/LayoutTests/fast/url/data-url-large.html (254308 => 254309)


--- branches/safari-609-branch/LayoutTests/fast/url/data-url-large.html	2020-01-09 23:33:51 UTC (rev 254308)
+++ branches/safari-609-branch/LayoutTests/fast/url/data-url-large.html	2020-01-09 23:47:02 UTC (rev 254309)
@@ -1,22 +0,0 @@
-<!DOCTYPE html>
-<html>
-<body>
-<script src=""
-<script>
-jsTestIsAsync = true;
-const url = "" + "q".repeat(32535);
-
-const xhr = new XMLHttpRequest();
-xhr._onload_ = function() {
-    testPassed("Large URL handled properly.");
-    finishJSTest();
-}
-xhr._onerror_ = function() {
-    testFailed("Large URL was improperly rejected.");
-    finishJSTest();
-}
-xhr.open("GET", url);
-xhr.send();
-</script>
-</body>
-</html>

Modified: branches/safari-609-branch/Source/WebCore/ChangeLog (254308 => 254309)


--- branches/safari-609-branch/Source/WebCore/ChangeLog	2020-01-09 23:33:51 UTC (rev 254308)
+++ branches/safari-609-branch/Source/WebCore/ChangeLog	2020-01-09 23:47:02 UTC (rev 254309)
@@ -1,45 +1,3 @@
-2020-01-09  Alan Coon  <alanc...@apple.com>
-
-        Cherry-pick r254301. rdar://problem/58346124
-
-    REGRESSION (r253662): Large Data URLs are not being handled properly
-    https://bugs.webkit.org/show_bug.cgi?id=205979
-    <rdar://problem/58346124>
-    
-    Reviewed by Youenn Fablet.
-    
-    Source/WebCore:
-    
-    The URL size limitation added in r253662 was too low. We should bump it to handle
-    reasonable data URI sizes.
-    
-    Test: fast/url/data-url-large.html.
-    
-    * page/SecurityOrigin.cpp:
-    
-    LayoutTests:
-    
-    * fast/url/data-url-large-expected.txt: Added.
-    * fast/url/data-url-large.html: Added.
-    
-    
-    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@254301 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
-    2020-01-09  Brent Fulgham  <bfulg...@apple.com>
-
-            REGRESSION (r253662): Large Data URLs are not being handled properly
-            https://bugs.webkit.org/show_bug.cgi?id=205979
-            <rdar://problem/58346124>
-
-            Reviewed by Youenn Fablet.
-
-            The URL size limitation added in r253662 was too low. We should bump it to handle
-            reasonable data URI sizes.
-
-            Test: fast/url/data-url-large.html.
-
-            * page/SecurityOrigin.cpp:
-
 2020-01-03  Sihui Liu  <sihui_...@apple.com>
 
         Crash in com.apple.WebKit.Networking at UniqueIDBDatabase::performCurrentOpenOperation

Modified: branches/safari-609-branch/Source/WebCore/page/SecurityOrigin.cpp (254308 => 254309)


--- branches/safari-609-branch/Source/WebCore/page/SecurityOrigin.cpp	2020-01-09 23:33:51 UTC (rev 254308)
+++ branches/safari-609-branch/Source/WebCore/page/SecurityOrigin.cpp	2020-01-09 23:47:02 UTC (rev 254309)
@@ -46,7 +46,7 @@
 
 namespace WebCore {
 
-constexpr unsigned maximumURLSize = 0x04000000;
+constexpr unsigned maximumURLSize = 0x8000;
 
 static bool schemeRequiresHost(const URL& url)
 {

_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to