Skip to content

Remove unused resources#38245

Merged
jdm merged 1 commit intoservo:mainfrom
lumiscosity:remove-unused-resources
Jul 24, 2025
Merged

Remove unused resources#38245
jdm merged 1 commit intoservo:mainfrom
lumiscosity:remove-unused-resources

Conversation

@lumiscosity
Copy link
Copy Markdown
Contributor

As previously discussed in #38229. To recap: badcert.jpg was made unused in 8422dac and tumbeast.png and its associated file of not-found.html were superseded at some point by a message in neterror.html (if it were still used, it'd surely be embedded.)

This PR also removes two unnecessary spaces inside of the new tab's CSS file, ensuring a consistent 2 space indentation in the file and saving 2 bytes.

Testing: Unnecessary, as this commit only removes unused assets (and fixes one style nit).

@jdm jdm enabled auto-merge July 23, 2025 20:05
@jdm jdm added this pull request to the merge queue Jul 23, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 23, 2025
@jdm jdm added this pull request to the merge queue Jul 24, 2025
Merged via the queue into servo:main with commit bb5cb57 Jul 24, 2025
21 checks passed
PotatoCP added a commit to PotatoCP/servo that referenced this pull request Jul 24, 2025
{"fail_fast": false, "matrix": [{"name": "Linux (WPT)", "workflow": "linux", "wpt": true, "profile": "release", "unit_tests": false, "build_libservo": false, "bencher": false, "build_args": "", "wpt_args": "./tests/wpt/tests/webdriver/tests/classic/ --product servodriver --headless --processes 1", "number_of_wpt_chunks": 2}]}
@yezhizhen
Copy link
Copy Markdown
Member

yezhizhen commented Jul 24, 2025

Somehow /html/semantics/embedded-content/media-elements/seeking/seek-to-currentTime.html fails stably when I
./mach test-wpt -r /html/semantics/embedded-content/media-elements/seeking/seek-to-currentTime.html --headless

FAIL seek to currentTime - assert_array_equals: fired events lengths differ, expected array ["seeking", "timeupdate", "seeked"] length 3, got ["seeking", "timeupdate", "timeupdate", "seeked"] length 4

Is the behaviour platform dependent?

Portal to file: https://github.com/servo/servo/blob/main/tests/wpt/tests/html/semantics/embedded-content/media-elements/seeking/seek-to-currentTime.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants