Changeset 254301 in webkit


Ignore:
Timestamp:
Jan 9, 2020 2:41:28 PM (4 years ago)
Author:
Brent Fulgham
Message:

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.
Location:
trunk
Files:
2 added
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r254296 r254301  
     12020-01-09  Brent Fulgham  <bfulgham@apple.com>
     2
     3        REGRESSION (r253662): Large Data URLs are not being handled properly
     4        https://bugs.webkit.org/show_bug.cgi?id=205979
     5        <rdar://problem/58346124>
     6
     7        Reviewed by Youenn Fablet.
     8
     9        * fast/url/data-url-large-expected.txt: Added.
     10        * fast/url/data-url-large.html: Added.
     11
    1122020-01-09  John Wilander  <wilander@apple.com>
    213

  • trunk/Source/WebCore/ChangeLog

    r254298 r254301  
     12020-01-09  Brent Fulgham  <bfulgham@apple.com>
     2
     3        REGRESSION (r253662): Large Data URLs are not being handled properly
     4        https://bugs.webkit.org/show_bug.cgi?id=205979
     5        <rdar://problem/58346124>
     6
     7        Reviewed by Youenn Fablet.
     8
     9        The URL size limitation added in r253662 was too low. We should bump it to handle
     10        reasonable data URI sizes.
     11
     12        Test: fast/url/data-url-large.html.
     13
     14        * page/SecurityOrigin.cpp:
     15
    1162020-01-09  Eamon Sisk  <esisk@apple.com>
    217

  • trunk/Source/WebCore/page/SecurityOrigin.cpp

    r253946 r254301  
    4747namespace WebCore {
    4848
    49 constexpr unsigned maximumURLSize = 0x8000;
     49constexpr unsigned maximumURLSize = 0x04000000;
    5050
    5151static bool schemeRequiresHost(const URL& url)
Note: See TracChangeset for help on using the changeset viewer.