Fix periodic dumping for memory-infra on Android.

tracing/tracing/ui/extras/about_tracing/inspector_tracing_controller_client.html

Index: tracing/tracing/ui/extras/about_tracing/inspector_tracing_controller_client.html
diff --git a/tracing/tracing/ui/extras/about_tracing/inspector_tracing_controller_client.html b/tracing/tracing/ui/extras/about_tracing/inspector_tracing_controller_client.html
index 0971bb93c1395a7f0d40c2a79229bfe811119706..4970ea5363f5effb5ecc499bde7bb689751e89ab 100644
--- a/tracing/tracing/ui/extras/about_tracing/inspector_tracing_controller_client.html
+++ b/tracing/tracing/ui/extras/about_tracing/inspector_tracing_controller_client.html
@@ -91,14 +91,24 @@ tr.exportTo('tr.ui.e.about_tracing', function() {
         throw new Error('Already recording');
       }
       this.recording_ = 'starting';
+      const memoryConfig = { triggers: [] };
+      if (recordingOptions.includedCategories.indexOf(
+          'disabled-by-default-memory-infra') !== -1) {
+        memoryConfig.triggers.push(
+            {'mode': 'detailed', 'periodic_interval_ms': 10000});
+      }
+
+      const traceConfigStr = {
+        includedCategories: recordingOptions.includedCategories,
+        excludedCategories: recordingOptions.excludedCategories,
+        memoryDumpConfig: memoryConfig,
+        recordMode: recordingOptions.tracingRecordMode,
+        enableSystrace: recordingOptions.useSystemTracing
+      };
       let res = this.conn_.req(
           'Tracing.start',
           {
-            categories: recordingOptions.categoryFilter,
-            options:
-                [recordingOptions.tracingRecordMode,
-                  (recordingOptions.useSampling ? 'enable-sampling' : '')

[ssid, 2017/06/21 18:59:20]

+Andrey can you verify this is not required? I can't see this being used
anywhere, but I am worried some V8 tracing might break because of this?

[erikchen, 2017/06/21 22:56:50]

This is dead code, which is why I removed it. The parser for "options" does not
check for "enable-sampling" anywhere. Also, try code-search for
"enable-sampling":
https://cs.chromium.org/search/?q=%22enable-sampling%22+package:%5Echromium$&...

[fmeawad, 2017/06/22 23:16:39]

I can confirm that this is dead code, this is not related to the V8 sampler,
this is a different sampler that was removed.

-                ].join(','),
+            traceConfig: traceConfigStr,
             bufferUsageReportingInterval: 1000
           });
       res = res.then(