Fix logic error causing unnecessary repeated SVG rasterization.

[vorporeal]

Description

Fixes an ImageCache hole that caused SVGs rendered at their intrinsic size to be rasterized again on every frame instead of reusing a cached rendered image. This also makes rendered-cache identity include fit behavior, because different fit modes can produce different pixels at the same final dimensions.

Adds regression coverage for intrinsic-size SVG reuse and for distinct same-size bitmap results produced by different fit modes.

Linked Issue

No linked issue.

Testing

• ./script/run — verified SVG conversion logging occurred during initial materialization rather than repeating across subsequent rendered frames; the temporary diagnostic log was removed before submitting.
• ./script/format
• cargo check -p warpui_core
• cargo nextest run -p warpui_core -E 'test(test_caches_svg_rendered_at_intrinsic_size) | test(test_different_fit_types_do_not_collide_in_rendered_image_cache) | test(test_evict_size_drops_arc_only_for_targeted_entry)'

Screenshots / Videos

Not applicable; this fixes internal rendering-cache behavior without changing the UI.

Agent Mode

• Warp Agent Mode - This PR was created via Warp's AI Agent Mode

CHANGELOG-NONE

Co-Authored-By: Oz oz-agent@warp.dev

[vorporeal]

• Fix logic error causing unnecessary repeated SVG rasterization. #12104 👈 (View in Graphite)
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

[oz-for-oss]

@vorporeal

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

[oz-for-oss]

Overview

This PR fixes the rendered image cache so SVGs rasterized at intrinsic size are cached and so cache entries distinguish fit behavior when the same requested bounds can produce different pixels.

Concerns

• No blocking correctness, security, or spec-alignment concerns found in the changed lines.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz