New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 787422 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Task



Sign in to add a comment

Port MediaStreamTrackMetrics IPC messages to mojo

Project Member Reported by guidou@chromium.org, Nov 21 2017

Issue description

This is part of the ongoing work to port all IPC to Mojo.

 
Components: Blink>Internals>Modularization
Labels: -Type-Bug Type-Task
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 2 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/869faf0b5aee0f32a130bb9afb6fb80424dbbddc

commit 869faf0b5aee0f32a130bb9afb6fb80424dbbddc
Author: mck.giri <mck.giri@samsung.com>
Date: Fri Mar 02 16:15:04 2018

Port MediaStreamTrackMetrics IPC messages to mojo.

This CL migrates renderer to browser IPC messages.

Also, This CL registers MediaStreamTrackMetricsHost as a Mojo interface to be
exposed to the renderer from RenderProcessHostImpl. The host is created
whenever a request is made from the renderer.

Bug:  787422 
Change-Id: I113b774858633d787f300099177277835af69da2
Reviewed-on: https://chromium-review.googlesource.com/795524
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Tommi <tommi@chromium.org>
Commit-Queue: Guido Urdaneta <guidou@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540528}
[modify] https://crrev.com/869faf0b5aee0f32a130bb9afb6fb80424dbbddc/content/browser/renderer_host/media/media_stream_track_metrics_host.cc
[modify] https://crrev.com/869faf0b5aee0f32a130bb9afb6fb80424dbbddc/content/browser/renderer_host/media/media_stream_track_metrics_host.h
[modify] https://crrev.com/869faf0b5aee0f32a130bb9afb6fb80424dbbddc/content/browser/renderer_host/render_process_host_impl.cc
[modify] https://crrev.com/869faf0b5aee0f32a130bb9afb6fb80424dbbddc/content/browser/renderer_host/render_process_host_impl.h
[modify] https://crrev.com/869faf0b5aee0f32a130bb9afb6fb80424dbbddc/content/common/BUILD.gn
[modify] https://crrev.com/869faf0b5aee0f32a130bb9afb6fb80424dbbddc/content/common/content_message_generator.h
[modify] https://crrev.com/869faf0b5aee0f32a130bb9afb6fb80424dbbddc/content/common/media/media_stream.mojom
[delete] https://crrev.com/7a0533a54ecaa935b7f923b13d4dba8e80c158ff/content/common/media/media_stream_track_metrics_host_messages.h
[modify] https://crrev.com/869faf0b5aee0f32a130bb9afb6fb80424dbbddc/content/public/app/mojo/content_browser_manifest.json
[modify] https://crrev.com/869faf0b5aee0f32a130bb9afb6fb80424dbbddc/content/renderer/media/webrtc/media_stream_track_metrics.cc
[modify] https://crrev.com/869faf0b5aee0f32a130bb9afb6fb80424dbbddc/content/renderer/media/webrtc/media_stream_track_metrics.h
[modify] https://crrev.com/869faf0b5aee0f32a130bb9afb6fb80424dbbddc/ipc/ipc_message_start.h

Comment 3 by guidou@chromium.org, Mar 19 2018

Status: Fixed (was: Assigned)

Sign in to add a comment