Fix step selection in Recorder

it regressed at some time in the past. Missing stopPropagation
was resetting the selection immediately.

Bug: none
Change-Id: I1497b3ee47e1f0bccc2f8e36f49fb2d86bdd2998
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/7264765
Commit-Queue: Alex Rudenko <[email protected]>
Reviewed-by: Ergün Erdoğmuş <[email protected]>
Auto-Submit: Alex Rudenko <[email protected]>

Author: OrKoN

front_end/panels/recorder/components/RecordingView.ts

@@ -910,7 +910,7 @@ export class RecordingView extends UI.Widget.Widget {
   }
 
   #isTitleInvalid = false;
-  #selectedStep?: Models.Schema.Step|null;
+  #selectedStep: Models.Schema.Step|null = null;
   #replaySettingsExpanded = false;
   #showCodeView = false;
   #code = '';
@@ -1090,10 +1090,10 @@ export class RecordingView extends UI.Widget.Widget {
   }
 
   #onWrapperClick(): void {
-    if (this.#selectedStep === undefined) {
+    if (this.#selectedStep) {
       return;
     }
-    this.#selectedStep = undefined;
+    this.#selectedStep = null;
     this.performUpdate();
   }
 

front_end/panels/recorder/components/StepView.ts

@@ -412,7 +412,8 @@ export const DEFAULT_VIEW = (input: ViewInput, _output: ViewOutput, target: HTML
         }
         data-step-index=${input.stepIndex}
         data-section-index=${input.sectionIndex}
-        @click=${() => {
+        @click=${(event: Event) => {
+          event.stopPropagation();
           const stepOrSection = input.step || input.section;
           if (stepOrSection) {
             input.onStepClick(stepOrSection);