From 14ff288947aef1398473e20a34509fe0243de2e9 Mon Sep 17 00:00:00 2001 From: ssiddhartha Date: Mon, 1 Feb 2016 11:39:02 +0000 Subject: [PATCH] Changes from crrev.com/1584583003 Remove --no-sandbox flag from command line for telemetry Now that the browser is able to dump for child processes the --no-sandbox flag is not required for getting memory statistics. So, this is removed form the telemetry benchmarks. See crbug.com/461788 --- .../backends/chrome_inspector/tracing_backend_unittest.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/telemetry/telemetry/internal/backends/chrome_inspector/tracing_backend_unittest.py b/telemetry/telemetry/internal/backends/chrome_inspector/tracing_backend_unittest.py index dacef132809..a49ed8c1fac 100644 --- a/telemetry/telemetry/internal/backends/chrome_inspector/tracing_backend_unittest.py +++ b/telemetry/telemetry/internal/backends/chrome_inspector/tracing_backend_unittest.py @@ -23,10 +23,6 @@ class TracingBackendTest(tab_test_case.TabTestCase): @classmethod def CustomizeBrowserOptions(cls, options): options.AppendExtraBrowserArgs([ - # Memory maps currently cannot be retrieved on sandboxed processes. - # See crbug.com/461788. - '--no-sandbox', - # Workaround to disable periodic memory dumps. See crbug.com/513692. '--enable-memory-benchmarking' ])