Skip to content

Commit ee0193d

Browse files
committed
Avoid throwing an NPE in the Grid UI
1 parent 9049a6f commit ee0193d

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

javascript/grid-ui/src/util/browser-logo.tsx

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,10 @@ import safariTechnologyPreviewLogo
99
import unknownBrowserLogo from '../assets/browsers/unknown.svg'
1010

1111
const browserLogo = (browser: string): string => {
12+
if (!browser) {
13+
return unknownBrowserLogo
14+
}
15+
1216
switch (browser.toLowerCase()) {
1317
case 'chrome':
1418
return chromeLogo

javascript/grid-ui/src/util/os-logo.tsx

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,10 @@ import linuxLogo from '../assets/operating-systems/linux.svg'
44
import unknownOsLogo from '../assets/operating-systems/unknown.svg'
55

66
const osLogo = (os: string): string => {
7+
if (!os) {
8+
return unknownOsLogo
9+
}
10+
711
const osLowerCase: string = os.toLowerCase()
812
if (osLowerCase.includes('win')) {
913
return windowsLogo

0 commit comments

Comments
 (0)