From f053a3f2747807f8173489d809d41cf9d7317f56 Mon Sep 17 00:00:00 2001 From: Wei Huang Date: Tue, 7 Aug 2018 16:07:46 -0400 Subject: [PATCH] test(bazel): Remove --no-sandbox for protractor tests (#25362) Since we no longer need this flag to run web tests inside a docker container, this reverts https://github.com/angular/angular/pull/24906 PR Close #25362 --- packages/bazel/src/protractor/protractor.conf.js | 5 ----- 1 file changed, 5 deletions(-) diff --git a/packages/bazel/src/protractor/protractor.conf.js b/packages/bazel/src/protractor/protractor.conf.js index dae9bb3c8d..70ee78a475 100644 --- a/packages/bazel/src/protractor/protractor.conf.js +++ b/packages/bazel/src/protractor/protractor.conf.js @@ -107,11 +107,6 @@ if (process.env['WEB_TEST_METADATA']) { if (headless) { args.push('--headless'); args.push('--disable-gpu'); - const isRoot = process.getuid && process.getuid() == 0; - if (isRoot) { - // Specify --no-sandbox if it is started by root. - args.push('--no-sandbox'); - } } setConf(conf, 'directConnect', true, 'is set to true for chrome'); setConf(