layout: Let OutsideMarker store an IndependentFormattingContext

[Loirooriol]

OutsideMarker was storing the BlockFormattingContext and the LayoutBoxBase in separate fields. Now it will store an entire IndependentFormattingContext in a single field.

In particular, this fixes the issue that OutsideMarker::repair_style() wasn't calling BlockFormattingContext::repair_style(). Now we can just rely on IndependentFormattingContext::repair_style(), which correctly repairs the style of both the BlockFormattingContext and the LayoutBoxBase.

Also, the BlockLevelBox::repair_style() was repairing the style of the LayoutBoxBase twice, for all kinds of block-level boxes. This removes the duplication.

Testing: Adding a new test
Fixes: #42779

[github-actions]

🔨 Triggering try run (#22423493339) for Linux (WPT)

[servo-wpt-sync]

🤖 Opened new upstream WPT pull request (web-platform-tests/wpt#58060) with upstreamable changes.

[github-actions]

Test results for linux-wpt from try job (#22423493339):

Flaky unexpected result (25)

• OK [expected TIMEOUT] /IndexedDB/idbfactory_open.any.worker.html
  • PASS [expected FAIL] subtest: Calling open() with version argument 1.5 should not throw.
  • PASS [expected TIMEOUT] subtest: Calling open() with version argument 9007199254740991 should not throw.
  • PASS [expected TIMEOUT] subtest: Calling open() with version argument undefined should not throw.
• OK /IndexedDB/idbtransaction-oncomplete.any.html (#42845)
  • FAIL [expected PASS] subtest: IDBTransaction - complete event

    assert_array_equals: lengths differ, expected array ["upgradeneeded", "complete", "success", "opencursor"] length 4, got ["upgradeneeded", "complete", "success"] length 3
    

• OK /_mozilla/css/offset_properties_inline.html (#40543)
  • FAIL [expected PASS] subtest: offsetTop

    assert_equals: offsetTop of #inline-1 should be 0. expected 0 but got -1
    

  • FAIL [expected PASS] subtest: offsetLeft

    assert_equals: offsetLeft of #inline-2 should be 40. expected 40 but got 25
    

• TIMEOUT [expected OK] /_mozilla/mozilla/img_find_non_sibling_map.html
• CRASH [expected PASS] /_mozilla/shadow-dom/move-element-with-ua-shadow-tree-crash.html (#39473)
• TIMEOUT [expected OK] /fetch/api/redirect/redirect-keepalive.https.any.html (#32153)
  • TIMEOUT [expected PASS] subtest: [keepalive][iframe][load] mixed content redirect; setting up

    Test timed out
    

• OK /html/browsers/browsing-the-web/navigating-across-documents/005.html (#27062)
  • PASS [expected FAIL] subtest: Link with onclick navigation and href navigation
• CRASH [expected OK] /html/browsers/browsing-the-web/navigating-across-documents/about-srcdoc-navigation-blocked.window.html
• OK /html/browsers/browsing-the-web/navigating-across-documents/navigation-unload-same-origin-fragment.html (#20768)
  • PASS [expected FAIL] subtest: Tests that a fragment navigation in the unload handler will not block the initial navigation
• CRASH [expected TIMEOUT] /html/canvas/offscreen/text/2d.text.measure.fillTextCluster-baseline.tentative.w.html
• TIMEOUT [expected OK] /html/interaction/focus/the-autofocus-attribute/document-with-fragment-empty.html (#28259)
  • TIMEOUT [expected FAIL] subtest: Autofocus elements in top-level browsing context's documents with empty fragments should work.

    Test timed out
    

• TIMEOUT [expected OK] /html/interaction/focus/the-autofocus-attribute/update-the-rendering.html (#24145)
  • TIMEOUT [expected FAIL] subtest: "Flush autofocus candidates" should be happen before a scroll event and animation frame callbacks

    Test timed out
    

• OK /html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/allow-scripts-flag-changing-2.html (#39703)
  • FAIL [expected PASS] subtest: Meta refresh of the original iframe is not blocked if moved into a sandboxed iframe

    uncaught exception: Error: assert_unreached: The iframe into which the meta was moved must not refresh Reached unreachable code
    

• TIMEOUT [expected OK] /html/semantics/embedded-content/the-iframe-element/iframe_sandbox_navigate_other_frame_popup.sub.html (#39702)
  • TIMEOUT [expected FAIL] subtest: Sandboxed iframe can not navigate other frame's popup

    Test timed out
    

• CRASH [expected OK] /html/webappapis/scripting/reporterror.any.worker.html
• OK /mixed-content/tentative/autoupgrades/audio-upgrade.https.sub.html (#41697)
  • FAIL [expected PASS] subtest: Audio of other host autoupgraded

    assert_equals: Length of other host audio is correct expected 1 but got Infinity
    

• OK [expected CRASH] /resource-timing/render-blocking-status-link.html (#41664)
• OK /touch-events/single-tap-when-touchend-listener-use-sync-xhr.html (#41175)
  • PASS [expected FAIL] subtest: Click event should be fired when touchend opens synchronous XHR
• OK [expected TIMEOUT] /trusted-types/trusted-types-navigation.html?06-10 (#37920)
  • PASS [expected TIMEOUT] subtest: Navigate a frame via anchor with javascript:-urls w/ default policy in report-only mode.
  • FAIL [expected NOTRUN] subtest: Navigate a window via anchor with javascript:-urls w/ a default policy throwing an exception in enforcing mode.

    promise_test: Unhandled rejection with value: "Unexpected message received: \"No securitypolicyviolation reported!\""
    

  • FAIL [expected NOTRUN] subtest: Navigate a window via anchor with javascript:-urls w/ a default policy throwing an exception in report-only mode.

    promise_test: Unhandled rejection with value: "Unexpected message received: \"No securitypolicyviolation reported!\""
    

• OK /visual-viewport/resize-event-order.html (#41981)
  • PASS [expected FAIL] subtest: Popup: DOMWindow resize fired before VisualViewport.
• OK /wasm/webapi/abort.any.html (#39966)
  • FAIL [expected PASS] subtest: instantiateStreaming() asynchronously racing with abort should succeed or reject with AbortError

    assert_equals: expected "AbortError" but got "CompileError"
    

• OK /webdriver/tests/classic/accept_alert/accept.py
  • FAIL [expected PASS] subtest: test_accept_in_popup_window

    AssertionError: no such alert (404): No user prompt is currently active.
    

• OK /webdriver/tests/classic/take_screenshot/iframe.py
  • ERROR [expected PASS] subtest: test_always_captures_top_browsing_context

    setup error: ConnectionRefusedError: [Errno 111] Connection refused
    

  • ERROR [expected PASS] subtest: test_source_origin[same_origin]

    setup error: ConnectionRefusedError: [Errno 111] Connection refused
    

  • ERROR [expected PASS] subtest: test_source_origin[cross_origin]

    setup error: ConnectionRefusedError: [Errno 111] Connection refused
    

• OK [expected ERROR] /webxr/render_state_update.https.html (#27535)
• CRASH [expected OK] /workers/WorkerGlobalScope_setInterval.htm

Stable unexpected results that are known to be intermittent (21)

• OK /IndexedDB/transaction-scheduling-mixed-scopes.any.html (#42753)
  • PASS [expected FAIL] subtest: Check that scope restrictions on mixed transactions are enforced.
• FAIL [expected PASS] /_mozilla/mozilla/sslfail.html (#10760)
• TIMEOUT [expected OK] /_mozilla/mozilla/window_resize_event.html (#36741)
  • TIMEOUT [expected PASS] subtest: Popup onresize event fires after resizeTo

    Test timed out
    

• OK /_webgl/conformance/textures/misc/texture-upload-size.html (#21770)
  • PASS [expected FAIL] subtest: WebGL test #45
  • PASS [expected FAIL] subtest: WebGL test #47
  • PASS [expected FAIL] subtest: WebGL test #49
  • PASS [expected FAIL] subtest: WebGL test #51
  • FAIL [expected PASS] subtest: WebGL test #53

    assert_true: Texture was smaller than the expected size 2x2 expected true got false
    

  • FAIL [expected PASS] subtest: WebGL test #55

    assert_true: getError expected: INVALID_VALUE. Was NO_ERROR : when calling texSubImage2D with the same texture upload with offset 1, 1 expected true got false
    

  • FAIL [expected PASS] subtest: WebGL test #57

    assert_true: Texture was smaller than the expected size 2x2 expected true got false
    

  • FAIL [expected PASS] subtest: WebGL test #59

    assert_true: getError expected: INVALID_VALUE. Was NO_ERROR : when calling texSubImage2D with the same texture upload with offset 1, 1 expected true got false
    

  • PASS [expected FAIL] subtest: WebGL test #61
  • PASS [expected FAIL] subtest: WebGL test #63
  • And 14 more unexpected results...
• ERROR [expected CRASH] /_webgl/conformance2/misc/uninitialized-test-2.html (#41656)
• OK /beacon/beacon-basic.https.window.html (#41723)
  • FAIL [expected PASS] subtest: Payload size restriction should be accumulated: type = arraybuffer

    assert_false: expected false got true
    

• OK /css/css-fonts/generic-family-keywords-001.html (#37467)
  • FAIL [expected PASS] subtest: @font-face matching for quoted and unquoted generic(kai)

    assert_equals: quoted generic(kai) matches  @font-face rule expected 50 but got 30
    

  • FAIL [expected PASS] subtest: @font-face matching for quoted and unquoted generic(khmer-mul)

    assert_equals: quoted generic(khmer-mul) matches  @font-face rule expected 50 but got 30
    

• OK /css/css-fonts/generic-family-keywords-003.html (#38994)
  • PASS [expected FAIL] subtest: @font-face matching for quoted and unquoted cursive (drawing text in a canvas)
• OK /fetch/metadata/generated/css-font-face.https.sub.tentative.html (#32732)
  • PASS [expected FAIL] subtest: sec-fetch-dest
• OK /fetch/metadata/generated/css-font-face.sub.tentative.html (#34624)
  • PASS [expected FAIL] subtest: sec-fetch-storage-access - Not sent to non-trustworthy same-origin destination
  • PASS [expected FAIL] subtest: sec-fetch-storage-access - Not sent to non-trustworthy same-site destination
• TIMEOUT /fetch/metadata/generated/css-images.sub.tentative.html (#29047)
  • FAIL [expected PASS] subtest: content sec-fetch-site - HTTPS downgrade-upgrade

    assert_unreached: Reached unreachable code
    

• OK [expected ERROR] /fetch/metadata/window-open.https.sub.html (#40339)
• OK /html/browsers/history/the-history-interface/traverse_the_history_3.html (#21383)
  • FAIL [expected PASS] subtest: Multiple history traversals, last would be aborted

    assert_array_equals: Pages opened during history navigation expected property 1 to be 3 but got 2 (expected array [6, 3] got [6, 2])
    

• OK [expected TIMEOUT] /html/interaction/focus/the-autofocus-attribute/autofocus-dialog.html (#29087)
  • FAIL [expected TIMEOUT] subtest: <dialog>-contained autofocus element gets focused when the dialog is shown

    assert_equals: expected "DIV" but got "BODY"
    

• TIMEOUT /html/interaction/focus/the-autofocus-attribute/supported-elements.html (#24145)
  • TIMEOUT [expected PASS] subtest: Non-HTMLElement should not support autofocus

    Test timed out
    

• TIMEOUT [expected OK] /html/semantics/embedded-content/media-elements/src_object_blob.html (#40340)
  • TIMEOUT [expected PASS] subtest: HTMLMediaElement.srcObject blob

    Test timed out
    

• OK /html/semantics/scripting-1/the-script-element/execution-timing/077.html (#22139)
  • PASS [expected FAIL] subtest: adding several types of scripts through the DOM and removing some of them confuses scheduler
• OK /preload/prefetch-document.html (#37210)
  • FAIL [expected PASS] subtest: different-site document prefetch with 'as=document' should not be consumed

    assert_equals: expected 2 but got 1
    

• OK /resource-timing/buffer-full-add-then-clear.html (#40819)
  • FAIL [expected PASS] subtest: Test that if the buffer is cleared after entries were added to the secondary buffer, those entries make it into the primary one

    assert_equals: Number of entries does not match the expected value. expected 3 but got 0
    

• OK /resource-timing/test_resource_timing.html (#25720)
  • PASS [expected FAIL] subtest: PerformanceEntry has correct name, initiatorType, startTime, and duration (iframe)
  • PASS [expected FAIL] subtest: PerformanceEntry has correct name, initiatorType, startTime, and duration (img)
  • PASS [expected FAIL] subtest: PerformanceEntry has correct name, initiatorType, startTime, and duration (xmlhttprequest)
• TIMEOUT /trusted-types/trusted-types-navigation.html?31-35 (#38034)
  • TIMEOUT [expected NOTRUN] subtest: Navigate a frame via form-submission with javascript:-urls w/ default policy in report-only mode.

    Test timed out
    

[github-actions]

✨ Try run (#22423493339) succeeded.