| commit | 28c7c024888608f8e11c6bf4c09970a0fedead95 | [log] [tgz] |
|---|---|---|
| author | Nikolay Vitkov <[email protected]> | Thu Apr 17 15:04:27 2025 |
| committer | Devtools-frontend LUCI CQ <devtools-frontend-scoped@luci-project-accounts.iam.gserviceaccount.com> | Thu Apr 17 16:19:02 2025 |
| tree | 79519a29d745d30013af3c558bde5c35a12532dc | |
| parent | abd0d4829764eaf52a30cf5a3ae2f72f3a86e3cf [diff] [blame] |
Fix edge case with empty string Else we add an empty feature and the flag start with `-enable-ff=,<default-flags>`; Bug: none Change-Id: Id2da1598d9826deee2f63d38bdb5dfb761f4b53a Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6468517 Auto-Submit: Nikolay Vitkov <[email protected]> Reviewed-by: Benedikt Meurer <[email protected]> Commit-Queue: Benedikt Meurer <[email protected]>
diff --git a/scripts/devtools_build.mjs b/scripts/devtools_build.mjs index dd7473f..36bdbcc 100644 --- a/scripts/devtools_build.mjs +++ b/scripts/devtools_build.mjs
@@ -97,6 +97,9 @@ } static parse(text) { + if (!text) { + return []; + } const features = []; for (const str of text.split(',')) { const parts = str.split(':');