tor-browser

commit d6393e58a45a3a1fc674b2818224d15a861bfce0
parent c00ab92de8d3865179fbf95d8ac06deeb7f22038
Author: Nicolas Silva <nical@fastmail.com>
Date:   Mon, 15 Dec 2025 10:53:23 +0000

Bug 1998913 - Part 13 - Inline PicturePrimitive::pre_update. r=gfx-reviewers,jnicol

It only call one function and that function has a more useful name.

Differential Revision: https://phabricator.services.mozilla.com/D276249

Diffstat:
M
gfx/wr/webrender/src/picture.rs
 | 
12
+-----------
M
gfx/wr/webrender/src/picture_graph.rs
 | 
2
+-

2 files changed, 2 insertions(+), 12 deletions(-)
diff --git a/gfx/wr/webrender/src/picture.rs b/gfx/wr/webrender/src/picture.rs
@@ -616,7 +616,7 @@ impl PicturePrimitive {
         pt.end_level();
     }
 
-    fn resolve_scene_properties(&mut self, properties: &SceneProperties) {
+    pub fn resolve_scene_properties(&mut self, properties: &SceneProperties) {
         match self.composite_mode {
             Some(PictureCompositeMode::Filter(ref mut filter)) => {
                 match *filter {
@@ -1071,16 +1071,6 @@ impl PicturePrimitive {
         }
     }
 
-    /// Do initial checks to determine whether this picture should be drawn as part of the
-    /// frame build.
-    pub fn pre_update(
-        &mut self,
-        frame_context: &FrameBuildingContext,
-    ) {
-        // Resolve animation properties
-        self.resolve_scene_properties(frame_context.scene_properties);
-    }
-
     /// Called during initial picture traversal, before we know the
     /// bounding rect of children. It is possible to determine the
     /// surface / raster config now though.
diff --git a/gfx/wr/webrender/src/picture_graph.rs b/gfx/wr/webrender/src/picture_graph.rs
@@ -174,7 +174,7 @@ fn assign_update_pass(
     info.parent = parent_pic_index;
 
     // Run pre-update to resolve animation properties etc
-    pic.pre_update(frame_context);
+    pic.resolve_scene_properties(frame_context.scene_properties);
 
     let can_be_drawn = match info.update_pass {
         Some(update_pass) => {