Make SVGImage::CurrentFrameHasSingleSecurityOrigin() less strict
The call to CheckLoaded() applies the restriction that the image must
have completed loading before the overall check can be performed. This
in turn means that callers ImageResourceContent::IsAccessAllowed() can
only be safely do after the resource has finished loading. This clashes
with all forms of progressive rendering of images.
Drop the CheckLoaded() call since we've locked down the loading of
subresources for SVGImage in a more thorough way since the check was
added. This allows answering the security-origin query in a
progressive way and avoids the crash in the referenced bug.
Fixed: 366547215
Change-Id: I2adfe7b3406572f0e1b1a956ac6aa4bf50b19c5d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5892795
Reviewed-by: Philip Rogers pdr@chromium.org
Commit-Queue: Fredrik Söderquist fs@opera.com
Cr-Commit-Position: refs/heads/main@{#1363100}