Merge PR #1631: Memory: floating pictures' textures are bounded, sized to what is drawn, and dropped with their workbook

Conflicts:
- ROADMAP.md: header change line names both memory changes; both progress-log and revision rows kept

ROADMAP.md

@@ -2,7 +2,7 @@
 
 **Stage: alpha** · next: beta, ~25 points of "ready for real work" and ~160–240 h away
 
-> **Last reviewed:** 2026-10-11 · **Last updated:** 2026-10-16 · **Change:** minor (undo steps share picture and macro bytes) · **Target:** Microsoft Excel (Microsoft 365)
+> **Last reviewed:** 2026-10-11 · **Last updated:** 2026-10-16 · **Change:** minor (undo steps share picture and macro bytes; floating pictures' textures bounded and kept per workbook) · **Target:** Microsoft Excel (Microsoft 365)
 
 GridCraft aims at full Microsoft Excel parity, and at being better in places: faster, open (XLSX
 is the native format), fully scriptable by agents (MCP, control channel, CLI), and on the web.
@@ -134,6 +134,7 @@ Ranked; estimates in Opus 5.5 agent-hours. Detail and the full beta-gate list in
 |---|---|
 | 2026-10-13 | XLSX export writes a picture used on several sheets once, as one `xl/media` part every drawing points to (it wrote a copy per drawing: a 16-sheet workbook with one 112 KB logo grew from 0.4 to 1.9 MB) (#985) |
 | 2026-10-16 | Memory: undo steps share a sheet's picture bytes and the workbook's VBA project instead of copying them: 50 edits on a sheet with a 20 MB picture keep 22 MB instead of 992 MB |
+| 2026-10-16 | Memory: floating pictures are decoded at about the size they are drawn, with limits, into a cache bounded by bytes and dropped with their workbook, instead of at full size into one that never let go; two open workbooks no longer show each other's picture. 12 large pictures scrolled through keep 40 MB of textures instead of 258 MB, and none once the workbook closes |
 | 2026-10-12 | Rows auto-fit oversized fonts: opening a file and whole-row/column formatting grow non-custom rows to their tallest font (#833) |
 | 2026-10-12 | Number formats: parsing a format code is linear in its length (long runs of `[`, `,` or `/` no longer take quadratic time), and an unclosed `[` is a literal when splitting sections as well as when tokenizing (#810) |
 | 2026-10-12 | Number formats: a fraction denominator with 65,536 zeros no longer crashes; format codes are capped at 255 characters like Excel (longer codes display as General, `TEXT` gives `#VALUE!`) (#807) |
@@ -197,6 +198,7 @@ Ranked; estimates in Opus 5.5 agent-hours. Detail and the full beta-gate list in
 |---|---|---|
 | 2026-10-13 | trivial | Progress log: shared pictures written once on XLSX export (#985) |
 | 2026-10-16 | minor | Progress log: undo steps share picture and macro bytes |
+| 2026-10-16 | minor | Progress log: floating pictures' textures bounded and kept per workbook |
 | 2026-10-12 | minor | Rows auto-fit oversized fonts on open and whole-row/column formatting (#833); progress log row |
 | 2026-10-12 | minor | Progress log: multi-area row/column delete (#758) |
 | 2026-10-12 | trivial | Progress log: XLSX chart sheets kept on save (#727) |

crates/ui-egui/src/chartview.rs

@@ -1,9 +1,6 @@
 //! Floating objects on the sheet: charts (rendered by `gridcraft-chart` into primitives that
 //! we paint with egui), pictures and shapes.
 
-use std::collections::HashMap;
-use std::sync::Mutex;
-
 use egui::epaint::{PathShape, PathStroke};
 use egui::{Align2, Color32, FontId, Painter, Pos2, Rect, Stroke, StrokeKind, pos2, vec2};
 use gridcraft_chart::{HAlign, Prim, VAlign};
@@ -249,8 +246,6 @@ pub(crate) fn anchor_rect(geo: &Geo, sh: &Sheet, a: &Anchor) -> Rect {
     Rect::from_min_size(pos2(x, y), vec2(a.width * geo.z, a.height.max(1.0) * geo.z))
 }
 
-static TEXTURES: std::sync::LazyLock<Mutex<HashMap<(u32, usize), egui::TextureHandle>>> = std::sync::LazyLock::new(|| Mutex::new(HashMap::new()));
-
 /// The shapes one floating object painted (indices into the painter's layer) and its logical
 /// rectangle, so a right-to-left sheet can move it as a whole instead of mirroring it.
 pub(crate) struct PaintedObject {
@@ -270,6 +265,8 @@ pub(crate) fn paint_objects(app: &SheetApp, p: &Painter, geo: &Geo, wb: &Workboo
         return painted;
     }
     let mut done = |start: egui::layers::ShapeIdx, rect: Rect| painted.push(PaintedObject { shapes: start.0..p.add(egui::Shape::Noop).0, rect });
+    // The grid paints the active workbook: its pictures are cached under it.
+    let doc = app.session.active().map_or(0, |d| d.uid);
     for sp in &sh.shapes {
         let r = anchor_rect(geo, sh, &sp.anchor);
         if !r.intersects(p.clip_rect()) {
@@ -341,18 +338,7 @@ pub(crate) fn paint_objects(app: &SheetApp, p: &Painter, geo: &Geo, wb: &Workboo
             continue;
         }
         let start = p.add(egui::Shape::Noop);
-        let key = (im.id, im.data.len());
-        let mut cache = TEXTURES.lock().unwrap_or_else(std::sync::PoisonError::into_inner);
-        if !cache.contains_key(&key)
-            && let Ok(img) = image::load_from_memory(&im.data)
-        {
-            let rgba = img.to_rgba8();
-            let size = [rgba.width() as usize, rgba.height() as usize];
-            let ci = egui::ColorImage::from_rgba_unmultiplied(size, rgba.as_raw());
-            let tex = p.ctx().load_texture(format!("img{}", im.id), ci, egui::TextureOptions::LINEAR);
-            cache.insert(key, tex);
-        }
-        if let Some(tex) = cache.get(&key) {
+        if let Some(tex) = crate::floating_pictures::texture(p.ctx(), doc, im, r) {
             p.image(tex.id(), r, Rect::from_min_max(pos2(0.0, 0.0), pos2(1.0, 1.0)), Color32::WHITE);
         } else {
             p.rect_filled(r, 0.0, Color32::from_gray(230));

crates/ui-egui/src/floating_pictures.rs

@@ -0,0 +1,274 @@
+//! Floating pictures (Insert › Pictures) as textures: decoded once at about the size they are
+//! drawn, and kept in a cache per egui context that is bounded by bytes. The cache used to be
+//! process-wide, with no limit, decoding every picture at full size and keeping it until the app
+//! closed, after its workbook was closed too.
+
+use std::collections::HashMap;
+use std::io::Cursor;
+use std::sync::{Arc, Mutex};
+
+use egui::TextureHandle;
+use gridcraft_engine::model::Image;
+
+/// Larger pictures are not decoded: the header is read first, so nothing is allocated for them.
+const MAX_SIDE: u32 = 16_384;
+const MAX_PIXELS: u64 = 64_000_000;
+const MAX_DECODE_BYTES: u64 = 512 * 1024 * 1024;
+/// A texture is never wider or taller than this (nor than the GPU allows).
+const MAX_TEXTURE_SIDE: u32 = 4096;
+/// The smallest texture side, so a picture drawn small and zoomed in a little isn't decoded again.
+const MIN_TEXTURE_SIDE: u32 = 64;
+/// Pictures drawn in the last frame stay whatever this says, so none turns into a placeholder;
+/// the others are dropped, least recently drawn first, to keep the cache under it.
+const MAX_CACHE_BYTES: usize = 128 * 1024 * 1024;
+const MAX_CACHE_ITEMS: usize = 4096;
+
+/// A picture: its workbook, its object id there, and its size in bytes (object ids are per
+/// workbook, so two open workbooks can both have a picture 1).
+#[derive(Clone, Copy, Debug, Hash, PartialEq, Eq)]
+struct Key {
+    doc: u64,
+    id: u32,
+    len: usize,
+}
+
+#[derive(Default)]
+struct Cache {
+    entries: HashMap<Key, Entry>,
+    bytes: usize,
+    /// The open workbooks when the cache was last pruned.
+    docs: Vec<u64>,
+}
+
+struct Entry {
+    /// `None` when the picture can't be decoded: kept, so it isn't tried again every frame.
+    texture: Option<TextureHandle>,
+    bytes: usize,
+    /// The longest side it was decoded for; `u32::MAX` when it holds all there is (the source
+    /// size or the largest texture allowed), or when it can't be decoded.
+    side: u32,
+    used: u64,
+}
+
+impl Cache {
+    /// Drops the pictures of workbooks that are no longer open.
+    fn forget_closed(&mut self, open: &[u64]) {
+        if self.docs == open {
+            return;
+        }
+        self.entries.retain(|key, entry| {
+            let keep = open.contains(&key.doc);
+            if !keep {
+                self.bytes = self.bytes.saturating_sub(entry.bytes);
+            }
+            keep
+        });
+        self.docs = open.to_vec();
+    }
+
+    /// The texture for picture `key` drawn with its longer side `want` pixels long.
+    fn texture(&mut self, ctx: &egui::Context, key: Key, data: &[u8], want: u32) -> Option<TextureHandle> {
+        let frame = ctx.cumulative_frame_nr();
+        let side = texture_side(want, ctx.input(|i| i.max_texture_side));
+        if let Some(entry) = self.entries.get_mut(&key) {
+            entry.used = frame;
+            if entry.side >= side {
+                return entry.texture.clone();
+            }
+        }
+        // Not cached, or drawn larger than it was decoded for: decode it again at the new size.
+        if let Some(old) = self.entries.remove(&key) {
+            self.bytes = self.bytes.saturating_sub(old.bytes);
+        }
+        let decoded = decode(data, side);
+        let bytes = decoded.as_ref().map_or(0, |(image, _)| image.pixels.len().saturating_mul(4));
+        while self.entries.len() >= MAX_CACHE_ITEMS || self.bytes.saturating_add(bytes) > MAX_CACHE_BYTES {
+            let oldest = self.entries.iter().filter(|(_, e)| e.used.saturating_add(1) < frame).min_by_key(|(_, e)| e.used).map(|(key, _)| *key);
+            let Some(oldest) = oldest else { break };
+            if let Some(entry) = self.entries.remove(&oldest) {
+                self.bytes = self.bytes.saturating_sub(entry.bytes);
+            }
+        }
+        let (texture, side) = match decoded {
+            Some((image, complete)) => {
+                (Some(ctx.load_texture(format!("img{}", key.id), image, egui::TextureOptions::LINEAR)), if complete { u32::MAX } else { side })
+            }
+            None => (None, u32::MAX),
+        };
+        self.bytes = self.bytes.saturating_add(bytes);
+        self.entries.insert(key, Entry { texture: texture.clone(), bytes, side, used: frame });
+        texture
+    }
+}
+
+/// The texture side for a picture drawn `want` pixels long: the next power of two or three
+/// quarters of one (…, 768, 1024, 1536, 2048), so zooming in decodes it again only at every
+/// half or third larger, within the smallest and largest sides allowed.
+fn texture_side(want: u32, gpu_max: usize) -> u32 {
+    let max = MAX_TEXTURE_SIDE.min(u32::try_from(gpu_max).unwrap_or(u32::MAX)).max(1);
+    let want = want.max(MIN_TEXTURE_SIDE);
+    let Some(p) = want.checked_next_power_of_two() else { return max };
+    let side = if want <= p / 4 * 3 { p / 4 * 3 } else { p };
+    side.min(max)
+}
+
+/// Decodes a picture to fit a `side` × `side` square, and says whether that is the whole picture
+/// (it was no larger, or `side` is as large as a texture may be).
+fn decode(data: &[u8], side: u32) -> Option<(egui::ColorImage, bool)> {
+    let format = image::guess_format(data).ok()?;
+    let (w, h) = image::ImageReader::with_format(Cursor::new(data), format).into_dimensions().ok()?;
+    if w == 0 || h == 0 || w > MAX_SIDE || h > MAX_SIDE || u64::from(w) * u64::from(h) > MAX_PIXELS {
+        return None;
+    }
+    let mut reader = image::ImageReader::with_format(Cursor::new(data), format);
+    let mut limits = image::Limits::default();
+    limits.max_image_width = Some(MAX_SIDE);
+    limits.max_image_height = Some(MAX_SIDE);
+    limits.max_alloc = Some(MAX_DECODE_BYTES);
+    reader.limits(limits);
+    let mut picture = reader.decode().ok()?;
+    let complete = w.max(h) <= side || side >= MAX_TEXTURE_SIDE;
+    if w.max(h) > side {
+        picture = picture.thumbnail(side, side);
+    }
+    let rgba = picture.into_rgba8();
+    Some((egui::ColorImage::from_rgba_unmultiplied([rgba.width() as usize, rgba.height() as usize], rgba.as_raw()), complete))
+}
+
+fn cache(ctx: &egui::Context) -> Arc<Mutex<Cache>> {
+    // Context-local, like cell pictures: texture handles never leak between offscreen, native
+    // and browser contexts.
+    ctx.data_mut(|data| data.get_temp_mut_or_default::<Arc<Mutex<Cache>>>(egui::Id::new("gridcraft.floating_pictures")).clone())
+}
+
+/// Drops the textures of workbooks that are no longer open. Called every frame the grid paints.
+pub(crate) fn forget_closed(ctx: &egui::Context, open: &[u64]) {
+    cache(ctx).lock().unwrap_or_else(std::sync::PoisonError::into_inner).forget_closed(open);
+}
+
+/// The texture of picture `image` of workbook `doc`, drawn in `rect` (points); `None` when it
+/// can't be decoded or is too large to be.
+pub(crate) fn texture(ctx: &egui::Context, doc: u64, image: &Image, rect: egui::Rect) -> Option<TextureHandle> {
+    let px = (rect.width().max(rect.height()) * ctx.pixels_per_point()).ceil();
+    let want = if px.is_finite() && px > 0.0 { px.min(u32::MAX as f32) as u32 } else { 1 };
+    let key = Key { doc, id: image.id, len: image.data.len() };
+    cache(ctx).lock().unwrap_or_else(std::sync::PoisonError::into_inner).texture(ctx, key, &image.data, want)
+}
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    fn png(w: u32, h: u32, color: [u8; 4]) -> Vec<u8> {
+        let image = image::RgbaImage::from_pixel(w, h, image::Rgba(color));
+        let mut bytes = Cursor::new(Vec::new());
+        image.write_to(&mut bytes, image::ImageFormat::Png).unwrap();
+        bytes.into_inner()
+    }
+
+    fn key(doc: u64, id: u32, data: &[u8]) -> Key {
+        Key { doc, id, len: data.len() }
+    }
+
+    /// Runs one frame, so the frame number moves on as it does when the app paints.
+    fn frame(ctx: &egui::Context) {
+        let mut out = ctx.run_ui(Default::default(), |_| {});
+        out.textures_delta.clear(); // nothing consumes the uploads in a headless test
+    }
+
+    #[test]
+    fn pictures_are_decoded_at_about_the_size_drawn() {
+        let ctx = egui::Context::default();
+        let mut cache = Cache::default();
+        let data = png(3000, 1000, [10, 120, 200, 255]);
+        let k = key(1, 1, &data);
+        let small = cache.texture(&ctx, k, &data, 300).unwrap();
+        assert_eq!(small.size(), [384, 128], "drawn 300 px long: decoded for 384");
+        assert_eq!(cache.texture(&ctx, k, &data, 380).unwrap().id(), small.id(), "still within its size: not decoded again");
+        let larger = cache.texture(&ctx, k, &data, 900).unwrap();
+        assert_eq!(larger.size(), [1024, 341], "drawn larger: decoded again for 1024");
+        assert_eq!(cache.entries.len(), 1);
+        assert_eq!(cache.bytes, 1024 * 341 * 4, "the smaller texture's bytes are given back");
+        assert_eq!(texture_side(1280, 8192), 1536);
+        assert_eq!(texture_side(1600, 8192), 2048);
+        assert_eq!(texture_side(1, 8192), MIN_TEXTURE_SIDE);
+        assert_eq!(texture_side(u32::MAX, 8192), MAX_TEXTURE_SIDE);
+        // Never past the GPU's limit (2048 in a default context) nor the source.
+        assert_eq!(cache.texture(&ctx, k, &data, 50_000).unwrap().size(), [2048, 683]);
+        let tiny = png(40, 20, [0, 0, 0, 255]);
+        assert_eq!(cache.texture(&ctx, key(1, 2, &tiny), &tiny, 4000).unwrap().size(), [40, 20]);
+        assert_eq!(cache.entries.get(&key(1, 2, &tiny)).map(|e| e.side), Some(u32::MAX), "the whole picture: never decoded again");
+    }
+
+    #[test]
+    fn the_cache_stays_within_its_budget_but_keeps_what_is_on_screen() {
+        let ctx = egui::Context::default();
+        let mut cache = Cache::default();
+        // 40 pictures of 2048 × 2048 drawn in turn, one frame each: 640 MB decoded in all.
+        let data: Vec<Vec<u8>> = (0..40u8).map(|i| png(2048, 2048, [i, 0, 0, 255])).collect();
+        for (i, d) in data.iter().enumerate() {
+            assert!(cache.texture(&ctx, key(1, i as u32, d), d, 2048).is_some());
+            frame(&ctx);
+        }
+        assert!(cache.bytes <= MAX_CACHE_BYTES, "{} bytes kept", cache.bytes);
+        assert!(cache.entries.len() < data.len());
+        // Many on screen at once, over the budget: every one keeps its texture, frame after frame.
+        let shown: Vec<_> = data.iter().enumerate().take(12).map(|(i, d)| cache.texture(&ctx, key(1, i as u32, d), d, 2048).unwrap().id()).collect();
+        for _ in 0..3 {
+            frame(&ctx);
+            for (i, d) in data.iter().enumerate().take(12) {
+                assert_eq!(cache.texture(&ctx, key(1, i as u32, d), d, 2048).map(|t| t.id()), Some(shown[i]));
+            }
+        }
+    }
+
+    #[test]
+    fn a_closed_workbook_takes_its_pictures_with_it() {
+        let ctx = egui::Context::default();
+        let mut cache = Cache::default();
+        let data = png(64, 64, [0, 200, 0, 255]);
+        let first = cache.texture(&ctx, key(1, 1, &data), &data, 64).unwrap();
+        // Another workbook's picture 1, of the same size: a texture of its own.
+        let other = png(64, 64, [200, 0, 0, 255]);
+        assert_eq!(other.len(), data.len());
+        assert_ne!(cache.texture(&ctx, key(2, 1, &other), &other, 64).unwrap().id(), first.id());
+        cache.forget_closed(&[1, 2]);
+        assert_eq!(cache.entries.len(), 2);
+        cache.forget_closed(&[2]);
+        assert_eq!(cache.entries.len(), 1);
+        assert_eq!(cache.bytes, 64 * 64 * 4);
+        cache.forget_closed(&[]);
+        assert!(cache.entries.is_empty());
+        assert_eq!(cache.bytes, 0);
+    }
+
+    #[test]
+    fn broken_and_oversized_pictures_are_refused_once() {
+        let ctx = egui::Context::default();
+        let mut cache = Cache::default();
+        let garbage = vec![0xAB; 100];
+        assert!(cache.texture(&ctx, key(1, 1, &garbage), &garbage, 100).is_none());
+        assert_eq!(cache.entries.get(&key(1, 1, &garbage)).map(|e| (e.side, e.bytes)), Some((u32::MAX, 0)), "remembered, not retried");
+        // A PNG whose header says 20,000 × 20,000 is refused from its header: nothing is decoded.
+        let mut huge = png(1, 1, [0, 0, 0, 255]);
+        let ihdr = huge.windows(4).position(|w| w == b"IHDR").unwrap();
+        huge[ihdr + 4..ihdr + 8].copy_from_slice(&20_000u32.to_be_bytes());
+        huge[ihdr + 8..ihdr + 12].copy_from_slice(&20_000u32.to_be_bytes());
+        let crc = crc32(&huge[ihdr..ihdr + 17]);
+        huge[ihdr + 17..ihdr + 21].copy_from_slice(&crc.to_be_bytes());
+        assert_eq!(image::ImageReader::with_format(Cursor::new(&huge), image::ImageFormat::Png).into_dimensions().unwrap(), (20_000, 20_000));
+        assert!(decode(&huge, 64).is_none());
+        assert!(decode(&png(4, 2, [255, 0, 0, 255]), 64).is_some_and(|(i, complete)| i.size == [4, 2] && complete));
+    }
+
+    fn crc32(bytes: &[u8]) -> u32 {
+        let mut crc = !0u32;
+        for &b in bytes {
+            crc ^= u32::from(b);
+            for _ in 0..8 {
+                crc = if crc & 1 == 1 { (crc >> 1) ^ 0xEDB8_8320 } else { crc >> 1 };
+            }
+        }
+        !crc
+    }
+}

crates/ui-egui/src/lib.rs

@@ -14,6 +14,7 @@ pub mod control;
 pub mod credits;
 pub mod dialogs;
 pub mod editor;
+mod floating_pictures;
 pub mod fonts;
 pub mod formula_bar;
 mod formula_locale;
@@ -1035,6 +1036,9 @@ impl SheetApp {
         // A keytip control request lives one frame: the ribbon consumes it during `ui`, and
         // anything that never reached a control (collapsed ribbon, tab switch) is dropped.
         self.ui.keytip_invoke = None;
+        // Floating pictures' textures go with their workbook, even when no sheet is painted.
+        let open: Vec<u64> = self.session.documents().iter().map(|d| d.uid).collect();
+        floating_pictures::forget_closed(ctx, &open);
         if self.fonts_han.is_some_and(|han| han != theme::HanOrder::of(self.ui.language)) {
             self.fonts_ready = false;
             self.fonts_set = false;

crates/ui-egui/tests/floating_pictures.rs

@@ -0,0 +1,68 @@
+//! Floating pictures on the sheet, with original synthetic pictures.
+
+use std::io::Cursor;
+
+use egui::{Shape, TextureId, vec2};
+use gridcraft_engine::Session;
+use gridcraft_ui_egui::SheetApp;
+use serde_json::json;
+
+fn png(color: [u8; 4]) -> Vec<u8> {
+    let image = image::RgbaImage::from_pixel(120, 60, image::Rgba(color));
+    let mut bytes = Cursor::new(Vec::new());
+    image.write_to(&mut bytes, image::ImageFormat::Png).unwrap();
+    bytes.into_inner()
+}
+
+fn harness() -> egui_kittest::Harness<'static, SheetApp> {
+    let mut session = Session::new();
+    session.new_workbook();
+    let mut h = egui_kittest::Harness::builder().with_size(vec2(1200.0, 800.0)).build_ui_state(
+        |ui, app: &mut SheetApp| {
+            app.logic(&ui.ctx().clone());
+            app.ui(ui);
+        },
+        SheetApp::new(session, Default::default()),
+    );
+    h.run_steps(4);
+    h
+}
+
+fn picture_textures(h: &egui_kittest::Harness<'static, SheetApp>) -> Vec<TextureId> {
+    h.output()
+        .shapes
+        .iter()
+        .filter_map(|shape| match &shape.shape {
+            Shape::Mesh(mesh) if mesh.texture_id != TextureId::default() => Some(mesh.texture_id),
+            _ => None,
+        })
+        .collect()
+}
+
+/// Picture ids are per workbook, so two open workbooks both have a picture 1. Each must show its
+/// own picture, even when the two are the same number of bytes: the texture cache used to tell
+/// pictures apart by id and size alone, for the whole app.
+#[test]
+fn each_workbook_shows_its_own_picture() {
+    let (green, red) = (png([24, 150, 110, 255]), png([200, 40, 40, 255]));
+    assert_eq!(green.len(), red.len());
+    let mut h = harness();
+    h.state_mut().run("insert.picture", json!({"base64": gridcraft_engine::io::base64_encode(&green), "at": "B2"})).unwrap();
+    h.run_steps(4);
+    let first = picture_textures(&h);
+    assert_eq!(first.len(), 1, "the picture is painted");
+
+    h.state_mut().run("file.new", json!({})).unwrap();
+    h.state_mut().run("insert.picture", json!({"base64": gridcraft_engine::io::base64_encode(&red), "at": "B2"})).unwrap();
+    h.run_steps(4);
+    let doc = h.state().session.active().unwrap();
+    assert_eq!(doc.wb.active().unwrap().images[0].id, h.state().session.documents()[0].wb.active().unwrap().images[0].id);
+    let second = picture_textures(&h);
+    assert_eq!(second.len(), 1);
+    assert_ne!(second, first, "another workbook's picture 1 has a texture of its own");
+
+    // Closing the workbook and painting again leaves the first one's picture as it was.
+    h.state_mut().run("file.close", json!({"force": true})).unwrap();
+    h.run_steps(4);
+    assert_eq!(picture_textures(&h), first);
+}