Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(277)

Unified Diff: tracing/tracing/ui/extras/about_tracing/record_controller.html

Issue 2949423002: Update record_controller to pass memoryDumpConfig to all controller clients. (Closed)
Patch Set: Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tracing/tracing/ui/extras/about_tracing/inspector_tracing_controller_client_test.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tracing/tracing/ui/extras/about_tracing/record_controller.html
diff --git a/tracing/tracing/ui/extras/about_tracing/record_controller.html b/tracing/tracing/ui/extras/about_tracing/record_controller.html
index 8883da7a89b2b63fe4aa0f326ac9bce5cfcb2845..a9b42b589d884ba63dc9f05ca63b53c4a9f553f7 100644
--- a/tracing/tracing/ui/extras/about_tracing/record_controller.html
+++ b/tracing/tracing/ui/extras/about_tracing/record_controller.html
@@ -92,6 +92,13 @@ tr.exportTo('tr.ui.e.about_tracing', function() {
enable_systrace: selectionDlg.useSystemTracing,
record_mode: selectionDlg.tracingRecordMode,
};
+ if (categories.included.indexOf(
+ 'disabled-by-default-memory-infra') !== -1) {
+ const memoryConfig = { triggers: [] };
+ memoryConfig.triggers.push(
+ {'mode': 'detailed', 'periodic_interval_ms': 10000});
fmeawad 2017/06/23 17:38:09 I lost track of the original interval, but as I wa
+ recordingOptions.memory_dump_config = memoryConfig;
+ }
const requestPromise = tracingControllerClient.beginRecording(
recordingOptions);
« no previous file with comments | « tracing/tracing/ui/extras/about_tracing/inspector_tracing_controller_client_test.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698