#54026: test: re-enable the loadURL data URL base test
Merged
Description of Change
Chromium CL 5802682 changed non-special URL parsing and broke the Windows fixture. Electron #43968 (40cae71) fixed it but left the test skipped. Re-enable it and let navigation errors fail the test.
The test is also skipped on 45-x-y, 44-x-y, 43-x-y, and 42-x-y.
Validation: 30 no-retry runs and three controls per build passed on Linux/Xvfb with Electron 44 and 46. The controls detect missing base URLs, missing scripts, and swallowed navigation errors. Scoped lint and formatting checks passed. Native Windows/macOS and the full suite remain untested.
Checklist
- I have tested this change (test-only; no rebuild required)
- I have filled out the PR description
- I have reviewed and verified the changes
-
npm testpasses - tests are changed or added
- PR release notes describe the change in a way relevant to app developers.
Release Notes
Notes: none
Backports
Semver Impact
Major
Breaking changes
Minor
New features
Patch
Bug fixes
None
Docs, tests, etc.
Semantic Versioning helps users understand the impact of updates:
- Major (X.y.z): Breaking changes that may require code modifications
- Minor (x.Y.z): New features that maintain backward compatibility
- Patch (x.y.Z): Bug fixes that don't change the API
- None: Changes that don't affect using facing parts of Electron