| Index: test/runners/chromium_remote_process.js |
| =================================================================== |
| --- a/test/runners/chromium_remote_process.js |
| +++ b/test/runners/chromium_remote_process.js |
| @@ -54,17 +54,17 @@ |
| catch (error) |
| { |
| console.error(error); |
| } |
| } |
| function startChromium(chromiumPath) |
| { |
| - fs.chmodSync(chromiumPath, fs.constants.S_IRWXU); |
| + fs.chmodSync(chromiumPath, 0o700); |
|
Sebastian Noack
2018/09/25 22:12:46
fs.constants.S_IRWXU is 0o700. This change has no
hub
2018/09/26 01:03:25
on Windows? I expect it to be undefined there.
Sebastian Noack
2018/09/26 01:10:51
Geo, can you confirm?
geo
2018/09/26 13:34:09
Well, it get's rid of the `fs.chmodSync` error. Ho
|
| let dataDir = fs.mkdtempSync(path.join(os.tmpdir(), "chromium-data")); |
| let child = null; |
| return { |
| kill: () => child && child.kill(), |
| done: new Promise((resolve, reject) => |
| { |
| child = childProcess.execFile(chromiumPath, [ |