Avoid show (empty) for node targets in the isolate selector

The main target created for a node process in
`front_end/entrypoints/js_app/js_app.ts` is not assigned with
a URL. Format the title with its name `main`.

Bug: 400292281
Change-Id: If57717bcacc5891c36f0c2df8e7b0902becb1e4b
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6318319
Reviewed-by: Simon Zünd <[email protected]>
Commit-Queue: Simon Zünd <[email protected]>
Reviewed-by: Philip Pfaffe <[email protected]>

Author: legendecas

front_end/panels/profiler/IsolateSelector.ts

@@ -272,13 +272,17 @@ export class ListItem {
 
   updateTitle(): void {
     const modelCountByName = new Map<string, number>();
+    const targetManager = SDK.TargetManager.TargetManager.instance();
     for (const model of this.isolate.models()) {
       const target = model.target();
-      const name = SDK.TargetManager.TargetManager.instance().primaryPageTarget() !== target ? target.name() : '';
+      const isPrimaryPageTarget = targetManager.primaryPageTarget() === target;
+      const name = target.name();
       const parsedURL = new Common.ParsedURL.ParsedURL(target.inspectedURL());
       const domain = parsedURL.isValid ? parsedURL.domain() : '';
-      const title =
-          target.decorateLabel(domain && name ? `${domain}: ${name}` : name || domain || i18nString(UIStrings.empty));
+      // If it is primary page target, omit `domain` in the title.
+      // Otherwise show its `domain` and `name` as title if available.
+      const title = target.decorateLabel(
+          domain && !isPrimaryPageTarget ? `${domain}: ${name}` : name || domain || i18nString(UIStrings.empty));
       modelCountByName.set(title, (modelCountByName.get(title) || 0) + 1);
     }
     this.nameDiv.removeChildren();