From bc2c84ac2b5075c1f16ccf0d148f043808ca5e5d Mon Sep 17 00:00:00 2001 From: Gerald Squelart Date: Mon, 11 May 2020 13:09:50 +0000 Subject: [PATCH] Bug 1636398 - Tweak default profiler buffer size in tests - r=canaltinova Also removed explicit size in test_feature_nativeallocations, as we want a reasonably-big buffer in this test. Other tests with small buffer sizes are fine, since the profiler imposes a minimum size that should be big enough to capture at least 8 large stacks (see bug 1571171). Differential Revision: https://phabricator.services.mozilla.com/D74588 --- tools/profiler/tests/shared-head.js | 2 +- tools/profiler/tests/xpcshell/test_feature_nativeallocations.js | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/tools/profiler/tests/shared-head.js b/tools/profiler/tests/shared-head.js index a52138189d13..e9da2ec3f3a7 100644 --- a/tools/profiler/tests/shared-head.js +++ b/tools/profiler/tests/shared-head.js @@ -11,7 +11,7 @@ var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm"); const defaultSettings = { - entries: 1000000, // 9MB + entries: 8 * 1024 * 1024, // 8M entries = 64MB interval: 1, // ms features: ["threads"], threads: ["GeckoMain"], diff --git a/tools/profiler/tests/xpcshell/test_feature_nativeallocations.js b/tools/profiler/tests/xpcshell/test_feature_nativeallocations.js index 25f2df64f950..989331541785 100644 --- a/tools/profiler/tests/xpcshell/test_feature_nativeallocations.js +++ b/tools/profiler/tests/xpcshell/test_feature_nativeallocations.js @@ -28,7 +28,6 @@ add_task(async () => { { info("Start the profiler."); startProfiler({ - entries: 1e6, // Only instrument the main thread. threads: ["GeckoMain"], features: ["threads", "leaf", "nativeallocations"],