IPC::MessageFilter::OnFilterAdded: IPC::Sender -> IPC::Channel

IPC::MessageFilter::OnFilterAdded: IPC::Sender -> IPC::Channel

Passes an IPC::Channel* to OnFilterAdded instead of IPC::Sender*.

This is changing so that filters can use more than just
the Sender interface of the Channel, and namely so they can get at
remote associated interfaces.

BUG=612500
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/e0072ebd0fa7c6b465d71ab1032f2e113c7e7695
Cr-Commit-Position: refs/heads/master@{#419329}

[Ken Rockot(use gerrit already), 2016-09-16 19:52:52]

Description was changed from

==========
IPC::MessageFilter::OnFilterAdded: IPC::Sender -> IPC::Channel

Passes an IPC::Channel* to OnFilterAdded instead of IPC::Sender*.

This is changing so that filters can use more than just
the Sender interface of the Channel, and namely so they can get at
remote associated interfaces.

BUG=612500
==========

to

==========
IPC::MessageFilter::OnFilterAdded: IPC::Sender -> IPC::Channel

Passes an IPC::Channel* to OnFilterAdded instead of IPC::Sender*.

This is changing so that filters can use more than just
the Sender interface of the Channel, and namely so they can get at
remote associated interfaces.

BUG=612500
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Ken Rockot(use gerrit already), 2016-09-16 19:52:59]

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

[commit-bot: I haz the power, 2016-09-16 19:53:28]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Ken Rockot(use gerrit already), 2016-09-16 19:54:36]

rockot@chromium.org changed reviewers:
+ jam@chromium.org

[Ken Rockot(use gerrit already), 2016-09-16 19:54:37]

An alternative solution would be to add a GetAssociatedGroup() and
GetRemoteAssociatedInterface() to IPC::Sender. This CL felt slightly cleaner to
me despite the extra churn.

[Ken Rockot(use gerrit already), 2016-09-16 19:55:23]

On 2016/09/16 at 19:54:37, Ken Rockot wrote:
> An alternative solution would be to add a GetAssociatedGroup() and
GetRemoteAssociatedInterface() to IPC::Sender. This CL felt slightly cleaner to
me despite the extra churn.

Also note, no logic is changing in this CL. Only typenames.

[Ken Rockot(use gerrit already), 2016-09-16 19:57:00]

rockot@chromium.org changed reviewers:
+ rjkroege@chromium.org

[Ken Rockot(use gerrit already), 2016-09-16 19:57:01]

+rjkroege for ui/ozone since I added some extra gunk to FakeGpuProcess in the
test support code

[commit-bot: I haz the power, 2016-09-16 20:02:19]

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

[commit-bot: I haz the power, 2016-09-16 20:02:20]

Dry run: Try jobs failed on following builders:
  win_optional_gpu_tests_rel on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_optional_gpu_...)

[Ken Rockot(use gerrit already), 2016-09-16 20:04:03]

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

[commit-bot: I haz the power, 2016-09-16 20:04:42]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jam, 2016-09-16 20:06:56]

lgtm

[Ken Rockot(use gerrit already), 2016-09-16 20:15:57]

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

[commit-bot: I haz the power, 2016-09-16 20:16:10]

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-09-16 20:27:26]

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

[commit-bot: I haz the power, 2016-09-16 20:27:27]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_optional_gpu_tests_rel on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_optional_...)

[Ken Rockot(use gerrit already), 2016-09-16 20:31:13]

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

[commit-bot: I haz the power, 2016-09-16 20:32:01]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Ken Rockot(use gerrit already), 2016-09-16 20:35:00]

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

[commit-bot: I haz the power, 2016-09-16 20:35:08]

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-09-16 22:28:22]

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

[commit-bot: I haz the power, 2016-09-16 22:28:23]

Dry run: This issue passed the CQ dry run.

[Ken Rockot(use gerrit already), 2016-09-16 23:33:46]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2016-09-16 23:33:47]

The patchset sent to the CQ was uploaded after l-g-t-m from jam@chromium.org
Link to the patchset: https://codereview.chromium.org/2349713003/#ps80001
(title: ".")

[commit-bot: I haz the power, 2016-09-16 23:34:17]

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-09-16 23:40:49]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-09-16 23:43:04]

Description was changed from

==========
IPC::MessageFilter::OnFilterAdded: IPC::Sender -> IPC::Channel

Passes an IPC::Channel* to OnFilterAdded instead of IPC::Sender*.

This is changing so that filters can use more than just
the Sender interface of the Channel, and namely so they can get at
remote associated interfaces.

BUG=612500
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
IPC::MessageFilter::OnFilterAdded: IPC::Sender -> IPC::Channel

Passes an IPC::Channel* to OnFilterAdded instead of IPC::Sender*.

This is changing so that filters can use more than just
the Sender interface of the Channel, and namely so they can get at
remote associated interfaces.

BUG=612500
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/e0072ebd0fa7c6b465d71ab1032f2e113c7e7695
Cr-Commit-Position: refs/heads/master@{#419329}
==========

[commit-bot: I haz the power, 2016-09-16 23:43:05]

Patchset 5 (id:??) landed as
https://crrev.com/e0072ebd0fa7c6b465d71ab1032f2e113c7e7695
Cr-Commit-Position: refs/heads/master@{#419329}