Update record_controller to pass memoryDumpConfig to all controller clients.

Update record_controller to pass memoryDumpConfig to all controller clients.

BUG=chromium:726393
Review-Url: https://codereview.chromium.org/2949423002
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/06380c5ba3ee8e80dff973ac60c2b3202c2267dc

[erikchen, 2017-06-23 01:11:51]

erikchen@chromium.org changed reviewers:
+ fmeawad@chromium.org

[erikchen, 2017-06-23 01:11:51]

fmeawad: Please review.

[erikchen, 2017-06-23 16:58:20]

The CQ bit was checked by erikchen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-23 16:58:31]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-23 17:23:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-23 17:23:42]

Dry run: This issue passed the CQ dry run.

[fmeawad, 2017-06-23 17:38:09]

LGTM with a comment

https://codereview.chromium.org/2949423002/diff/1/tracing/tracing/ui/extras/a...
File tracing/tracing/ui/extras/about_tracing/record_controller.html (right):

https://codereview.chromium.org/2949423002/diff/1/tracing/tracing/ui/extras/a...
tracing/tracing/ui/extras/about_tracing/record_controller.html:99: {'mode':
'detailed', 'periodic_interval_ms': 10000});
I lost track of the original interval, but as I was testing this CL I noticed
that 10s in practice might be too long (specially for the about:tracing use
case). We should plumb it from the UI with a default value. (I will try to do it
in a follow up CL)

[erikchen, 2017-06-23 17:43:12]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2017-06-23 17:43:22]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-23 17:44:43]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1498239792931720, "parent_rev":
"8d57d4e4986bf9984793ef25c0a68db7fb1c752a", "commit_rev":
"06380c5ba3ee8e80dff973ac60c2b3202c2267dc"}

[commit-bot: I haz the power, 2017-06-23 17:44:49]

Description was changed from

==========
Update record_controller to pass memoryDumpConfig to all controller clients.

BUG=chromium:726393
==========

to

==========
Update record_controller to pass memoryDumpConfig to all controller clients.

BUG=chromium:726393

Review-Url: https://codereview.chromium.org/2949423002
Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2017-06-23 17:44:50]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...