Reland of "Mojo interface/service for Leak Detector on remote process"

Reland of "Mojo interface/service for Leak Detector on remote process"

This interface allows an instance of the Runtime Memory Leak Detector
(components/metrics/leak_detector) to run on a non-browser process, such
as a tab (renderer) process. The interface provides two function APIs:

- GetParams(): Returns the profiling parameters used for initializing
  LeakDetector.
- SendLeakReports(): Passes an array of leak reports to the browser process,
  where they can be attached to the UMA protobuf.

Also added an implementation of the Mojo interface,
LeakDetectorRemoteController, that implements the Mojo interface
LeakDetectorRemote. This class provides a bridge between the browser and
renderer processes. It also allows a single instance of
LeakDetectorController to provide leak detector params to the renderer
process and receive leak reports from the renderer process.

==============================================
Fixes since previous commit:
- Add missing dependency to components_unittests GYP build
- Add dependency on Mojo C++ services code to renderer

BUG=chromium:615223
TEST=Build successfully, unit tests pass
R=wfh@chromium.org

Review-Url: https://codereview.chromium.org/2064463002
Committed: https://crrev.com/0e57a1efd2a6f47e492c7d990eb397dfab5402a0
Cr-Original-Commit-Position: refs/heads/master@{#406466}
Cr-Commit-Position: refs/heads/master@{#406959}

[Simon Que, 2016-07-21 03:16:29]

Description was changed from

==========
Reland of "Mojo interface/service for Leak Detector on remote process"

This interface allows an instance of the Runtime Memory Leak Detector
(components/metrics/leak_detector) to run on a non-browser process, such
as a tab (renderer) process. The interface provides two function APIs:

- GetParams(): Returns the profiling parameters used for initializing
  LeakDetector.
- SendLeakReports(): Passes an array of leak reports to the browser process,
  where they can be attached to the UMA protobuf.

Also added an implementation of the Mojo interface,
LeakDetectorRemoteController, that implements the Mojo interface
LeakDetectorRemote. This class provides a bridge between the browser and
renderer processes. It also allows a single instance of
LeakDetectorController to provide leak detector params to the renderer
process and receive leak reports from the renderer process.

BUG=chromium:615223
TEST=Build successfully, unit tests pass
R=wfh@chromium.org

Review-Url: https://codereview.chromium.org/2064463002
Cr-Commit-Position: refs/heads/master@{#406466}
==========

to

==========
Reland of "Mojo interface/service for Leak Detector on remote process"

This interface allows an instance of the Runtime Memory Leak Detector
(components/metrics/leak_detector) to run on a non-browser process, such
as a tab (renderer) process. The interface provides two function APIs:

- GetParams(): Returns the profiling parameters used for initializing
  LeakDetector.
- SendLeakReports(): Passes an array of leak reports to the browser process,
  where they can be attached to the UMA protobuf.

Also added an implementation of the Mojo interface,
LeakDetectorRemoteController, that implements the Mojo interface
LeakDetectorRemote. This class provides a bridge between the browser and
renderer processes. It also allows a single instance of
LeakDetectorController to provide leak detector params to the renderer
process and receive leak reports from the renderer process.

==============================================
Fixes since previous commit:
- Add missing dependency to components_unittests GYP build

BUG=chromium:615223
TEST=Build successfully, unit tests pass
R=wfh@chromium.org

Review-Url: https://codereview.chromium.org/2064463002
Cr-Commit-Position: refs/heads/master@{#406466}
==========

[Simon Que, 2016-07-21 06:58:58]

Patchset #3 (id:40001) has been deleted

[Simon Que, 2016-07-21 06:59:28]

Description was changed from

==========
Reland of "Mojo interface/service for Leak Detector on remote process"

This interface allows an instance of the Runtime Memory Leak Detector
(components/metrics/leak_detector) to run on a non-browser process, such
as a tab (renderer) process. The interface provides two function APIs:

- GetParams(): Returns the profiling parameters used for initializing
  LeakDetector.
- SendLeakReports(): Passes an array of leak reports to the browser process,
  where they can be attached to the UMA protobuf.

Also added an implementation of the Mojo interface,
LeakDetectorRemoteController, that implements the Mojo interface
LeakDetectorRemote. This class provides a bridge between the browser and
renderer processes. It also allows a single instance of
LeakDetectorController to provide leak detector params to the renderer
process and receive leak reports from the renderer process.

==============================================
Fixes since previous commit:
- Add missing dependency to components_unittests GYP build

BUG=chromium:615223
TEST=Build successfully, unit tests pass
R=wfh@chromium.org

Review-Url: https://codereview.chromium.org/2064463002
Cr-Commit-Position: refs/heads/master@{#406466}
==========

to

==========
Reland of "Mojo interface/service for Leak Detector on remote process"

This interface allows an instance of the Runtime Memory Leak Detector
(components/metrics/leak_detector) to run on a non-browser process, such
as a tab (renderer) process. The interface provides two function APIs:

- GetParams(): Returns the profiling parameters used for initializing
  LeakDetector.
- SendLeakReports(): Passes an array of leak reports to the browser process,
  where they can be attached to the UMA protobuf.

Also added an implementation of the Mojo interface,
LeakDetectorRemoteController, that implements the Mojo interface
LeakDetectorRemote. This class provides a bridge between the browser and
renderer processes. It also allows a single instance of
LeakDetectorController to provide leak detector params to the renderer
process and receive leak reports from the renderer process.

==============================================
Fixes since previous commit:
- Add missing dependency to components_unittests GYP build
- Add dependency on Mojo C++ services code to renderer

BUG=chromium:615223
TEST=Build successfully, unit tests pass
R=wfh@chromium.org

Review-Url: https://codereview.chromium.org/2064463002
Cr-Commit-Position: refs/heads/master@{#406466}
==========

[Simon Que, 2016-07-21 06:59:42]

sque@chromium.org changed reviewers:
+ jochen@chromium.org

[Simon Que, 2016-07-21 07:00:23]

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

[commit-bot: I haz the power, 2016-07-21 07:00:36]

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, 2016-07-21 07:52:10]

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

[commit-bot: I haz the power, 2016-07-21 07:52:11]

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2016-07-21 08:29:20]

lgtm

[Simon Que, 2016-07-21 16:09:24]

The CQ bit was checked by sque@chromium.org

[commit-bot: I haz the power, 2016-07-21 16:09:38]

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, 2016-07-21 16:15:55]

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

[commit-bot: I haz the power, 2016-07-21 16:15:56]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Simon Que, 2016-07-21 17:20:50]

sque@chromium.org changed reviewers:
+ isherman@chromium.org

[Ilya Sherman, 2016-07-21 20:42:30]

lgtm

[Simon Que, 2016-07-21 21:04:57]

The CQ bit was checked by sque@chromium.org

[commit-bot: I haz the power, 2016-07-21 21:05:46]

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, 2016-07-21 21:14:21]

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

[commit-bot: I haz the power, 2016-07-21 21:14:23]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Will Harris, 2016-07-21 21:43:59]

lgtm

[Simon Que, 2016-07-21 21:44:37]

The CQ bit was checked by sque@chromium.org

[commit-bot: I haz the power, 2016-07-21 21:45:09]

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, 2016-07-21 21:53:23]

Description was changed from

==========
Reland of "Mojo interface/service for Leak Detector on remote process"

This interface allows an instance of the Runtime Memory Leak Detector
(components/metrics/leak_detector) to run on a non-browser process, such
as a tab (renderer) process. The interface provides two function APIs:

- GetParams(): Returns the profiling parameters used for initializing
  LeakDetector.
- SendLeakReports(): Passes an array of leak reports to the browser process,
  where they can be attached to the UMA protobuf.

Also added an implementation of the Mojo interface,
LeakDetectorRemoteController, that implements the Mojo interface
LeakDetectorRemote. This class provides a bridge between the browser and
renderer processes. It also allows a single instance of
LeakDetectorController to provide leak detector params to the renderer
process and receive leak reports from the renderer process.

==============================================
Fixes since previous commit:
- Add missing dependency to components_unittests GYP build
- Add dependency on Mojo C++ services code to renderer

BUG=chromium:615223
TEST=Build successfully, unit tests pass
R=wfh@chromium.org

Review-Url: https://codereview.chromium.org/2064463002
Cr-Commit-Position: refs/heads/master@{#406466}
==========

to

==========
Reland of "Mojo interface/service for Leak Detector on remote process"

This interface allows an instance of the Runtime Memory Leak Detector
(components/metrics/leak_detector) to run on a non-browser process, such
as a tab (renderer) process. The interface provides two function APIs:

- GetParams(): Returns the profiling parameters used for initializing
  LeakDetector.
- SendLeakReports(): Passes an array of leak reports to the browser process,
  where they can be attached to the UMA protobuf.

Also added an implementation of the Mojo interface,
LeakDetectorRemoteController, that implements the Mojo interface
LeakDetectorRemote. This class provides a bridge between the browser and
renderer processes. It also allows a single instance of
LeakDetectorController to provide leak detector params to the renderer
process and receive leak reports from the renderer process.

==============================================
Fixes since previous commit:
- Add missing dependency to components_unittests GYP build
- Add dependency on Mojo C++ services code to renderer

BUG=chromium:615223
TEST=Build successfully, unit tests pass
R=wfh@chromium.org

Review-Url: https://codereview.chromium.org/2064463002
Cr-Commit-Position: refs/heads/master@{#406466}
==========

[commit-bot: I haz the power, 2016-07-21 21:53:25]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-07-21 21:55:20]

Description was changed from

==========
Reland of "Mojo interface/service for Leak Detector on remote process"

This interface allows an instance of the Runtime Memory Leak Detector
(components/metrics/leak_detector) to run on a non-browser process, such
as a tab (renderer) process. The interface provides two function APIs:

- GetParams(): Returns the profiling parameters used for initializing
  LeakDetector.
- SendLeakReports(): Passes an array of leak reports to the browser process,
  where they can be attached to the UMA protobuf.

Also added an implementation of the Mojo interface,
LeakDetectorRemoteController, that implements the Mojo interface
LeakDetectorRemote. This class provides a bridge between the browser and
renderer processes. It also allows a single instance of
LeakDetectorController to provide leak detector params to the renderer
process and receive leak reports from the renderer process.

==============================================
Fixes since previous commit:
- Add missing dependency to components_unittests GYP build
- Add dependency on Mojo C++ services code to renderer

BUG=chromium:615223
TEST=Build successfully, unit tests pass
R=wfh@chromium.org

Review-Url: https://codereview.chromium.org/2064463002
Cr-Commit-Position: refs/heads/master@{#406466}
==========

to

==========
Reland of "Mojo interface/service for Leak Detector on remote process"

This interface allows an instance of the Runtime Memory Leak Detector
(components/metrics/leak_detector) to run on a non-browser process, such
as a tab (renderer) process. The interface provides two function APIs:

- GetParams(): Returns the profiling parameters used for initializing
  LeakDetector.
- SendLeakReports(): Passes an array of leak reports to the browser process,
  where they can be attached to the UMA protobuf.

Also added an implementation of the Mojo interface,
LeakDetectorRemoteController, that implements the Mojo interface
LeakDetectorRemote. This class provides a bridge between the browser and
renderer processes. It also allows a single instance of
LeakDetectorController to provide leak detector params to the renderer
process and receive leak reports from the renderer process.

==============================================
Fixes since previous commit:
- Add missing dependency to components_unittests GYP build
- Add dependency on Mojo C++ services code to renderer

BUG=chromium:615223
TEST=Build successfully, unit tests pass
R=wfh@chromium.org

Review-Url: https://codereview.chromium.org/2064463002
Committed: https://crrev.com/0e57a1efd2a6f47e492c7d990eb397dfab5402a0
Cr-Original-Commit-Position: refs/heads/master@{#406466}
Cr-Commit-Position: refs/heads/master@{#406959}
==========

[commit-bot: I haz the power, 2016-07-21 21:55:21]

Patchset 3 (id:??) landed as
https://crrev.com/0e57a1efd2a6f47e492c7d990eb397dfab5402a0
Cr-Commit-Position: refs/heads/master@{#406959}