[RPP] Fix faster zoom with shift

We have a logic for a 0.8 zoom instead of 0.3 when the zoom keys 's' and 'w' are clicked together with shift, but the zoom was never applied due to the zoom handling code prematurely exiting if any modifier keys were detected. The issue has been resolved by adding an exception for the 'shift' key, allowing the intended zoom behavior to function correctly.

Bug: None
Change-Id: I6f7ae5a1c30328b919a64c147d10b5bbff4e68ed
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/5938662
Reviewed-by: Jack Franklin <[email protected]>
Auto-Submit: Alina Varkki <[email protected]>
Commit-Queue: Alina Varkki <[email protected]>

Author: AlinaVarkki

front_end/ui/legacy/components/perf_ui/ChartViewport.ts

@@ -393,10 +393,11 @@ export class ChartViewport extends UI.Widget.VBox {
   }
 
   private handleZoomPanKeys(e: Event): void {
-    if (!UI.KeyboardShortcut.KeyboardShortcut.hasNoModifiers(e)) {
+    const keyboardEvent = (e as KeyboardEvent);
+    // Do not zoom if the key combination has any modifiers other than shift key
+    if (UI.KeyboardShortcut.KeyboardShortcut.hasAtLeastOneModifier(e) && !keyboardEvent.shiftKey) {
       return;
     }
-    const keyboardEvent = (e as KeyboardEvent);
     const zoomFactor = keyboardEvent.shiftKey ? 0.8 : 0.3;
     const panOffset = keyboardEvent.shiftKey ? 320 : 160;
     switch (keyboardEvent.code) {