PR

Author: chinmaygarde

shell/common/rasterizer.h

@@ -27,11 +27,11 @@ namespace flutter {
 ///
 /// The rasterizer owns the instance of the currently active on-screen render
 /// surface. On this surface, it renders the contents of layer trees submitted
-/// to it by the engine (which lives on the UI task runner).
+/// to it by the `Engine` (which lives on the UI task runner).
 ///
 /// The primary components owned by the rasterizer are the compositor context
-/// and the on-screen render surface. The compositor context all the GPU state
-/// necessary to render frames to the render surface.
+/// and the on-screen render surface. The compositor context has all the GPU
+/// state necessary to render frames to the render surface.
 ///
 class Rasterizer final : public SnapshotDelegate {
  public:
@@ -188,6 +188,8 @@ class Rasterizer final : public SnapshotDelegate {
   ///             updated in an otherwise static layer tree. To support this use
   ///             case, the rasterizer holds onto the last rendered layer tree.
   ///
+  /// @bug        https://github.com/flutter/flutter/issues/33939
+  ///
   /// @return     A pointer to the last layer or `nullptr` if this rasterizer
   ///             has never rendered a frame.
   ///
@@ -239,7 +241,11 @@ class Rasterizer final : public SnapshotDelegate {
   ///             the frame yet! On the other hand, the pipeline must own the
   ///             layer tree it renders because it keeps a reference to the last
   ///             layer tree around till a new frame is rendered. So a simple
-  ///             reference wont work either.
+  ///             reference wont work either. The `Rasterizer::DoDraw` method
+  ///             actually performs the GPU operations within the layer tree
+  ///             pipeline.
+  ///
+  /// @see        `Rasterizer::DoDraw`
   ///
   /// @param[in]  pipeline  The layer tree pipeline to take the next layer tree
   ///                       to render from.
@@ -375,8 +381,10 @@ class Rasterizer final : public SnapshotDelegate {
   ///             again with new limits after surface re-acquisition.
   ///
   /// @attention  This cache does not describe the entirety of GPU resources
-  ///             that may be cached. The raster cache also holds very large GPU
-  ///             resources.
+  ///             that may be cached. The `RasterCache` also holds very large
+  ///             GPU resources.
+  ///
+  /// @see        `RasterCache`
   ///
   /// @param[in]  max_bytes  The maximum byte size of resource that may be
   ///                        cached for GPU rendering.