Apply suggestions from code review

Co-authored-by: Alice Cecile <alice.i.cecile@gmail.com>

crates/bevy_render/src/camera/camera.rs

@@ -61,7 +61,7 @@ impl Default for AbsOrPercVec {
 }
 
 impl AbsOrPercVec {
-    /// Returns percentage of provided vec or absolute value from self.
+    /// Converts this size into a absolute size in pixels.
     #[inline]
     pub fn as_absolute(&self, of: UVec2) -> UVec2 {
         match self {
@@ -70,10 +70,11 @@ impl AbsOrPercVec {
         }
     }
 
-    /// Returns percentage of provided vec or absolute value.
+    ///  Converts this size into a absolute size in pixels if possible.
+
     /// Returns `None` if self is `Percentage` and argument is `None`
     #[inline]
-    pub fn as_absolute_opt(&self, of_opt: Option<UVec2>) -> Option<UVec2> {
+    pub fn try_as_absolute(&self, of_opt: Option<UVec2>) -> Option<UVec2> {
         match self {
             AbsOrPercVec::Absolute(v) => Some(*v),
             AbsOrPercVec::Percentage(_) => of_opt.map(|of| self.as_absolute(of)),
@@ -159,7 +160,7 @@ impl Camera {
     pub fn viewport_absolute_size(&self) -> Option<UVec2> {
         self.viewport
             .as_ref()
-            .and_then(|v| v.physical_size.as_absolute_opt(self.physical_target_size()))
+            .and_then(|v| v.physical_size.try_as_absolute(self.physical_target_size()))
     }
 
     /// The rendered physical bounds (minimum, maximum) of the camera. If the `viewport` field is

crates/bevy_render/src/render_phase/draw_state.rs

@@ -343,12 +343,12 @@ impl<'a> TrackedRenderPass<'a> {
     pub fn set_camera_viewport(&mut self, viewport: &Viewport, camera: &ExtractedCamera) {
         let physical_size = viewport
             .physical_size
-            .as_absolute_opt(camera.physical_target_size)
+            .try_as_absolute(camera.physical_target_size)
             .expect("Couldn't get camera.physical_target_size (needed for relative viewport size)");
 
         let physical_position = viewport
             .physical_position
-            .as_absolute_opt(camera.physical_target_size)
+            .try_as_absolute(camera.physical_target_size)
             .expect(
                 "Couldn't get camera.physical_target_size (needed for relative viewport position)",
             );

examples/3d/split_screen.rs

@@ -107,7 +107,7 @@ fn set_camera_viewports(
 
             let mut right_camera = right_camera.single_mut();
             right_camera.viewport = Some(Viewport {
-                // ... or specify a percentage of the render target.
+                // ... or specify an adaptive percentage of the render target.
                 physical_position: AbsOrPercVec::Percentage(Vec2::new(0.5, 0.0)),
                 physical_size: AbsOrPercVec::Percentage(Vec2::new(0.5, 1.0)),
                 ..default()