mojo-ify child histogram messages

Desktop / Chromium - Nicholas Verne [chromium.org] - 12 October 2017 17:14 EDT

We replace ChildHistogramMessageFilter with ChildHistogramImpl for the child process side of the connections.

HistogramController owns sets of interface pointers to the various child process host types. Its singleton is allowed to leak because it should only die when its process is being killed.

Bug: 577685 Change-Id: I230ce331c2491047f06654fef1e7c0586b4c47c1 Reviewed-on: https://chromium-review.googlesource.com/664158

91bf4a6 mojo-ify child histogram messages.
content/browser/browser_child_process_host_impl.cc | 14 ++-
content/browser/histogram_controller.cc | 126 ++++++++++++++++++---
content/browser/histogram_controller.h | 37 +++++-
content/browser/histogram_message_filter.cc | 9 --
content/browser/histogram_message_filter.h | 2 -
.../renderer_host/render_process_host_impl.cc | 19 +++-
content/child/BUILD.gn | 4 +-
content/child/child_histogram_fetcher_impl.cc | 93 +++++++++++++++
content/child/child_histogram_fetcher_impl.h | 64 +++++++++++
content/child/child_histogram_message_filter.cc | 95 ----------------
content/child/child_histogram_message_filter.h | 59 ----------
content/child/child_thread_impl.cc | 12 +-
content/child/child_thread_impl.h | 7 --
content/common/BUILD.gn | 1 +
content/common/child_process_messages.h | 16 ---
content/common/histogram_fetcher.mojom | 17 +++
content/public/app/mojo/content_gpu_manifest.json | 2 +
.../public/app/mojo/content_plugin_manifest.json | 2 +
.../public/app/mojo/content_renderer_manifest.json | 2 +
.../public/app/mojo/content_utility_manifest.json | 2 +
.../browser/browser_child_process_host_iterator.cc | 5 +
.../browser/browser_child_process_host_iterator.h | 2 +
content/renderer/render_thread_impl.cc | 1 -
23 files changed, 373 insertions(+), 218 deletions(-)

Upstream: git.chromium.org


  • Share