The class is getting updated on the span; but it doesn't look like the layout is getting dirtied.

Looks like fragments are ignored for image urls. Over th the svg & image teams for further triage.

Anyone on the team know what might be causing this? Could be a straight up invalidation bug too.

The example uses Google internal links. I'll try to sanitize it, as the problem is not inherently Google specific.

Possibly because StyleImage operator== only compares .data() (which is the ImageResource - which will be the same regardless of the fragment identifier.)

Does the following match the issue? (Loaded from HTTP, because file: have some special-cases when it comes to loading.)

Deleted: content-svgurl-fragment-update.html 329 bytes

content-svgurl-fragment-update.html 329 bytes View Download

Deleted: sprites.svg 276 bytes

sprites.svg 276 bytes Download

The effect appears the same to me.

Probably the same, but I see what you mean by needing to load from http. It doesn't repro loading from file.

Here's the original report code, with internal google URL refs switched to local.

Deleted: cr643716.html 1.4 KB

cr643716.html 1.4 KB View Download

Deleted: icons.svg 501 bytes

icons.svg 501 bytes Download

Even though the url in the OP is not public, this bug has no nonpublic information so we can remove a restrict-view label.

 Issue 683065  has been merged into this issue.

 Issue 785998  has been merged into this issue.

 Issue 873656  has been merged into this issue.

 Issue 880205  has been merged into this issue.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8468d5775293357816e12d37e26dcf02d5dd04e5

commit 8468d5775293357816e12d37e26dcf02d5dd04e5
Author: Fredrik Söderquist <fs@opera.com>
Date: Thu Sep 06 10:17:34 2018

Rework StyleImage equality determination

StyleFetchedImage and StyleFetchedImageSet needs to take the URL into
consideration because it can contain a fragment that will require the
image to change.

Bug:  643716 
Change-Id: If87d3583df0fcb37f872c1423a88f4bd33e8c38f
Reviewed-on: https://chromium-review.googlesource.com/1207390
Reviewed-by: Stephen Chenney <schenney@chromium.org>
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#589128}
[add] https://crrev.com/8468d5775293357816e12d37e26dcf02d5dd04e5/third_party/WebKit/LayoutTests/http/tests/images/image-fragment-id-invalidation-expected.html
[add] https://crrev.com/8468d5775293357816e12d37e26dcf02d5dd04e5/third_party/WebKit/LayoutTests/http/tests/images/image-fragment-id-invalidation.html
[add] https://crrev.com/8468d5775293357816e12d37e26dcf02d5dd04e5/third_party/WebKit/LayoutTests/http/tests/images/resources/sprites.svg
[modify] https://crrev.com/8468d5775293357816e12d37e26dcf02d5dd04e5/third_party/blink/renderer/core/style/style_fetched_image.cc
[modify] https://crrev.com/8468d5775293357816e12d37e26dcf02d5dd04e5/third_party/blink/renderer/core/style/style_fetched_image.h
[modify] https://crrev.com/8468d5775293357816e12d37e26dcf02d5dd04e5/third_party/blink/renderer/core/style/style_fetched_image_set.cc
[modify] https://crrev.com/8468d5775293357816e12d37e26dcf02d5dd04e5/third_party/blink/renderer/core/style/style_fetched_image_set.h
[modify] https://crrev.com/8468d5775293357816e12d37e26dcf02d5dd04e5/third_party/blink/renderer/core/style/style_generated_image.cc
[modify] https://crrev.com/8468d5775293357816e12d37e26dcf02d5dd04e5/third_party/blink/renderer/core/style/style_generated_image.h
[modify] https://crrev.com/8468d5775293357816e12d37e26dcf02d5dd04e5/third_party/blink/renderer/core/style/style_image.h
[modify] https://crrev.com/8468d5775293357816e12d37e26dcf02d5dd04e5/third_party/blink/renderer/core/style/style_pending_image.h