[webkit-changes] [WebKit/WebKit] 4e8b5c: [ wk2 Debug ] loader/stateobjects/pushstate-size-i...

Chris Dumez noreply at github.com
Wed Feb 28 09:29:34 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 4e8b5c1a05ae111def3a5754f16af81ff19ae3a0
      https://github.com/WebKit/WebKit/commit/4e8b5c1a05ae111def3a5754f16af81ff19ae3a0
  Author: Chris Dumez <cdumez at apple.com>
  Date:   2024-02-28 (Wed, 28 Feb 2024)

  Changed paths:
    M LayoutTests/loader/stateobjects/pushstate-size-iframe-expected.txt
    M LayoutTests/loader/stateobjects/pushstate-size-iframe.html
    M LayoutTests/loader/stateobjects/resources/pushstate-iframe.html
    M LayoutTests/platform/mac-wk2/TestExpectations
    M Source/WebCore/page/History.cpp
    M Source/WebCore/page/History.h
    M Source/WebCore/page/LocalDOMWindow.h
    M Source/WebCore/testing/Internals.cpp
    M Source/WebCore/testing/Internals.h
    M Source/WebCore/testing/Internals.idl

  Log Message:
  -----------
  [ wk2 Debug ] loader/stateobjects/pushstate-size-iframe.html is very slow, usually times out
https://bugs.webkit.org/show_bug.cgi?id=268836
rdar://122401579

Reviewed by Ryosuke Niwa.

This test was slow, especially in debug because it had to reach the 64MB size limit by doing
repeated history.pushState() calls. This would obviously take a while. To address the issue,
I am introducing a new window.internals API to override this limit. In the test, we now
lower this limit to 4MB, which makes the test run a lot faster.

* LayoutTests/loader/stateobjects/pushstate-size-iframe-expected.txt:
* LayoutTests/loader/stateobjects/pushstate-size-iframe.html:
* LayoutTests/loader/stateobjects/resources/pushstate-iframe.html:
* LayoutTests/platform/mac-wk2/TestExpectations:
* Source/WebCore/page/History.cpp:
(WebCore::History::totalStateObjectPayloadLimit const):
(WebCore::History::stateObjectAdded):
* Source/WebCore/page/History.h:
* Source/WebCore/page/LocalDOMWindow.h:
* Source/WebCore/testing/Internals.cpp:
(WebCore::Internals::resetToConsistentState):
(WebCore::Internals::setHistoryTotalStateObjectPayloadLimitOverride):
* Source/WebCore/testing/Internals.h:
* Source/WebCore/testing/Internals.idl:

Canonical link: https://commits.webkit.org/275452@main



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications


More information about the webkit-changes mailing list