Bug 231150 – Safari complains about invalid `imagesrcset` value in <link rel=preload> where Chrome doesn't

• Component: Page Loading
• Status: RESOLVED FIXED

[Chris J. Shull]

See Console for https://recipes.twospoons.us/recipes/Smoked_Chicken

    <link rel="preload" as="image" imagesrcset="../photos/smoked_chicken.256.jpg?526d5 256w, ../photos/smoked_chicken.512.jpg?526d5 512w, ../photos/smoked_chicken.768.jpg?526d5 768w, ../photos/smoked_chicken.1024.jpg?526d5 1024w, ../photos/smoked_chicken.1280.jpg?526d5 1280w, ../photos/smoked_chicken.1536.jpg?526d5 1536w, ../photos/smoked_chicken.1792.jpg?526d5 1792w, ../photos/smoked_chicken.2048.jpg?526d5 2048w, ../photos/smoked_chicken.2304.jpg?526d5 2304w, ../photos/smoked_chicken.2560.jpg?526d5 2560w, ../photos/smoked_chicken.2816.jpg?526d5 2816w, ../photos/smoked_chicken.3072.jpg?526d5 3072w, ../photos/smoked_chicken.3264.jpg?526d5 3264w" imagesizes="(max-width: 700px) 100vw, 700px">

FWIW, this works fine in Chrome.

[Kevin Neal]

Thanks for filing. The appropriate engineers have been notified.

[Radar WebKit Bug Importer]

<rdar://problem/83891023>

[Tim]

Created attachment 464102 [details]
Link preload responsive images experimental feature

It seems from Safari 15.3 onwards, you can enable the correct behaviour via an Experimental Features flag.

Hopefully this will be enabled by default in an upcoming version. Until then it will expect a `href` attribute on a rel="preload" as="image" link tag and throw an error if that does not exist.

[styfle]

I believe this bug causes Safari to preload the wrong image since link.href refers to the img.src but it should use link.imagesrcset when available to match the img.srcset and fetch the correct image.

When can we expect the fix to ship unflagged?

[Ahmad Saleem]

(In reply to styfle from comment #4)
> I believe this bug causes Safari to preload the wrong image since link.href
> refers to the img.src but it should use link.imagesrcset when available to
> match the img.srcset and fetch the correct image.
> 
> When can we expect the fix to ship unflagged?

It is currently as 'testable':

https://github.com/WebKit/WebKit/blob/597aa395dcdcdb2c1c48069f450f7419a30df72d/Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml#L3735

You can get idea on how flag works:

https://github.com/WebKit/WebKit/blob/597aa395dcdcdb2c1c48069f450f7419a30df72d/Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml#L27

[antwon]

Any updates on when this will ship unflagged?

[Anthony Ricaud]

Seems like this was implemented via bug 192950, bug 198533 and bug 198960.

Rob: Any plans to ship this?

[Brent Fulgham]

It looks like `wpt /preload/dynamic-adding-preload-imagesrcset.html` shows this works properly when activated.

[Chris Dumez]

Pull request: https://github.com/WebKit/WebKit/pull/16499

[EWS]

Committed 266787@main (a32d1a3606d7): <https://commits.webkit.org/266787@main>

Reviewed commits have been landed. Closing PR #16499 and removing active labels.