Bug 13185
| Summary: | Bug in DOM masks incorrect LayoutTest for HTMLImageObject.src() | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Nicholas Shanks <nickshanks> |
| Component: | DOM | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED WONTFIX | ||
| Severity: | Normal | CC: | ahmad.saleem792, ap, bfulgham, rniwa |
| Priority: | P2 | ||
| Version: | 523.x (Safari 3) | ||
| Hardware: | Mac | ||
| OS: | OS X 10.4 | ||
Nicholas Shanks
The file at LayoutTests/fast/dom/HTMLImageElement/image-src-absolute-url.html changes the base URL and is supposed to check that the src URL changed to reflect that. Instead it checks against the old base URL seemingly in error, and furthermore, the url is not updated as would be expected. The two problems cancel each other out, which is why the test passes.
Specifically, the first of the two "after" lines currently reads:
SUCCESS (document.images[0].src = file:///does/not/exist/test.jpg)
I believe it should read:
FAILURE (document.images[0].src = file:///also/does/not/exist/test.jpg)
The reason that this is failing will need to be identified and fixed and the layout test updated.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Alexey Proskuryakov
This test verifies that the behavior matches WinIE - is there a reason why it should be different?
Ahmad Saleem
https://github.com/WebKit/WebKit/blob/main/LayoutTests/fast/dom/HTMLImageElement/image-src-absolute-url.html#L31
This test exist and Comment 01 refers that it is intentional behaviour. Is anything needed here or this can be marked as "RESOLVED INVALID" or "RESOLVED WONTFIX"? Thanks!
Alexey Proskuryakov
The test as is passes in Firefox and Chrome too, so it's not harmful. Nicholas may be right in that the test is incorrect, but at this point, investigating it seems unnecessary.