From 6195010d76a3f3d0da67c54b31fe266a104866db Mon Sep 17 00:00:00 2001 From: Sam Clegg Date: Thu, 7 Aug 2025 16:57:36 -0700 Subject: [PATCH] [file_packager] Remove redundant `isNode` check This was added back in #22802 but seems redundant since the check just below for the indexedDB global should be enough to handle the node case. Split out from #24883 --- tools/file_packager.py | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/tools/file_packager.py b/tools/file_packager.py index 3318f077e0f6d..6095268dcda6d 100755 --- a/tools/file_packager.py +++ b/tools/file_packager.py @@ -805,15 +805,9 @@ def generate_js(data_target, data_files, metadata): var DB_VERSION = 1; var METADATA_STORE_NAME = 'METADATA'; var PACKAGE_STORE_NAME = 'PACKAGES'; - function openDatabase(callback, errback) {''' - if options.support_node: - code += ''' - if (isNode) { - return errback(); - }''' - code += ''' + function openDatabase(callback, errback) { if (typeof indexedDB == 'undefined') { - throw 'using IndexedDB to cache data can only be done on a web page or in a web worker'; + return errback('using IndexedDB to cache data can only be done on a web page or in a web worker'); } try { var openRequest = indexedDB.open(DB_NAME, DB_VERSION);