Refactor BannerMessages to use widgets instead of data classes (#8987)

Author: kenzieschmoll

packages/devtools_app/lib/src/extensions/embedded/_view_web.dart

@@ -314,12 +314,13 @@ class _ExtensionIFrameController extends DisposableController
         '${showBannerMessageEvent.messageId}',
       ),
       screenId: '${showBannerMessageEvent.extensionName}_ext',
-      textSpans: [
-        TextSpan(
-          text: showBannerMessageEvent.message,
-          style: TextStyle(fontSize: defaultFontSize),
-        ),
-      ],
+      buildTextSpans:
+          (_) => [
+            TextSpan(
+              text: showBannerMessageEvent.message,
+              style: TextStyle(fontSize: defaultFontSize),
+            ),
+          ],
     );
     bannerMessages.addMessage(
       bannerMessage,

packages/devtools_app/lib/src/screens/debugger/debugger_screen.dart

@@ -125,7 +125,7 @@ class _DebuggerScreenBodyWrapperState extends State<_DebuggerScreenBodyWrapper>
   @override
   void didChangeDependencies() {
     super.didChangeDependencies();
-    pushDebuggerIdeRecommendationMessage(context, DebuggerScreen.id);
+    pushDebuggerIdeRecommendationMessage(DebuggerScreen.id);
   }
 
   @override

packages/devtools_app/lib/src/screens/inspector_v2/inspector_screen_body.dart

@@ -228,7 +228,7 @@ class InspectorScreenBodyState extends State<InspectorScreenBody>
     // Mark the welcome message as shown.
     await storage.setValue(_welcomeShownStorageId, 'true');
     if (context.mounted) {
-      pushWelcomeToNewInspectorMessage(context, InspectorScreen.id);
+      pushWelcomeToNewInspectorMessage(InspectorScreen.id);
     }
   }
 }

packages/devtools_app/lib/src/screens/memory/framework/screen_body.dart

@@ -37,11 +37,11 @@ class _ConnectedMemoryBodyState extends State<ConnectedMemoryBody>
   void didChangeDependencies() {
     super.didChangeDependencies();
     if (!offlineDataController.showingOfflineData.value) {
-      maybePushDebugModeMemoryMessage(context, ScreenMetaData.memory.id);
-      maybePushHttpLoggingMessage(context, ScreenMetaData.memory.id);
+      maybePushDebugModeMemoryMessage(ScreenMetaData.memory.id);
+      maybePushHttpLoggingMessage(ScreenMetaData.memory.id);
 
       addAutoDisposeListener(http_service.httpLoggingState, () {
-        maybePushHttpLoggingMessage(context, ScreenMetaData.memory.id);
+        maybePushHttpLoggingMessage(ScreenMetaData.memory.id);
       });
     }
   }

packages/devtools_app/lib/src/screens/performance/panes/flutter_frames/flutter_frames_chart.dart

@@ -123,12 +123,13 @@ class _FlutterFramesChartState extends State<_FlutterFramesChart> {
       );
       bannerMessages.addMessage(
         ShaderJankMessage(
-          offlineDataController.showingOfflineData.value
-              ? ScreenMetaData.simple.id
-              : ScreenMetaData.performance.id,
+          screenId:
+              offlineDataController.showingOfflineData.value
+                  ? ScreenMetaData.simple.id
+                  : ScreenMetaData.performance.id,
           jankyFramesCount: shaderJankFrames.length,
           jankDuration: shaderJankDuration,
-        ).build(context),
+        ),
       );
     }
   }

packages/devtools_app/lib/src/screens/performance/performance_screen.dart

@@ -73,7 +73,7 @@ class PerformanceScreenBodyState extends State<PerformanceScreenBody>
   @override
   void didChangeDependencies() {
     super.didChangeDependencies();
-    maybePushDebugModePerformanceMessage(context, PerformanceScreen.id);
+    maybePushDebugModePerformanceMessage(PerformanceScreen.id);
   }
 
   @override

packages/devtools_app/lib/src/screens/profiler/profiler_screen.dart

@@ -96,7 +96,7 @@ class _ProfilerScreenBodyState extends State<ProfilerScreenBody>
   @override
   void didChangeDependencies() {
     super.didChangeDependencies();
-    maybePushDebugModePerformanceMessage(context, ProfilerScreen.id);
+    maybePushDebugModePerformanceMessage(ProfilerScreen.id);
   }
 
   @override