ipc: Rename GetFileHandleForProcess->GetPlatformFileForTransit.

ipc: Rename GetFileHandleForProcess->GetPlatformFileForTransit.

This CL has no intended behavior change. This CL also removes the now
unnecessary parameter |process|.

BUG=493414
Committed: https://crrev.com/09b40032d8de63a01eaf62760f2ebc89a813d99c
Cr-Commit-Position: refs/heads/master@{#385512}

[erikchen, 2016-04-05 01:13:05]

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

[commit-bot: I haz the power, 2016-04-05 01:13:59]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1858973002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1858973002/1

[commit-bot: I haz the power, 2016-04-05 07:15:10]

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

[commit-bot: I haz the power, 2016-04-05 07:15:11]

Dry run: Try jobs failed on following builders:
  android_compile_dbg on tryserver.chromium.android (JOB_TIMED_OUT,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)

[erikchen, 2016-04-05 17:27:49]

Description was changed from

==========
refactor, part 1

BUG=
==========

to

==========
ipc: Rename GetFileHandleForProcess->GetPlatformFileForTransit.

This CL has no intended behavior change. This CL also removes the now
unnecessary parameter |process|.

BUG=493414
==========

[erikchen, 2016-04-05 17:28:24]

Description was changed from

==========
ipc: Rename GetFileHandleForProcess->GetPlatformFileForTransit.

This CL has no intended behavior change. This CL also removes the now
unnecessary parameter |process|.

BUG=493414
==========

to

==========
ipc: Rename GetFileHandleForProcess->GetPlatformFileForTransit.

This CL has no intended behavior change. This CL also removes the now
unnecessary parameter |process|.

BUG=493414
==========

[erikchen, 2016-04-05 17:28:43]

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

[erikchen, 2016-04-05 17:28:44]

tsepez: Please review.

[Tom Sepez, 2016-04-05 17:45:06]

lgtm

https://codereview.chromium.org/1858973002/diff/1/ipc/ipc_platform_file.h
File ipc/ipc_platform_file.h (right):

https://codereview.chromium.org/1858973002/diff/1/ipc/ipc_platform_file.h#new...
ipc/ipc_platform_file.h:59: // passed to the IPC stack, or else it will leak.
nit: "passed to the IPC layer as part of a message" or some such describing how
it gets to IPC-land.

[erikchen, 2016-04-05 19:25:33]

https://codereview.chromium.org/1858973002/diff/1/ipc/ipc_platform_file.h
File ipc/ipc_platform_file.h (right):

https://codereview.chromium.org/1858973002/diff/1/ipc/ipc_platform_file.h#new...
ipc/ipc_platform_file.h:59: // passed to the IPC stack, or else it will leak.
On 2016/04/05 17:45:06, Tom Sepez wrote:
> nit: "passed to the IPC layer as part of a message" or some such describing
how
> it gets to IPC-land.

Done.

[erikchen, 2016-04-05 19:28:35]

erikchen@chromium.org changed reviewers:
+ avi@chromium.org, droger@chromium.org, mseaborn@chromium.org,
piman@chromium.org, sergeyu@chromium.org, thestig@chromium.org

[erikchen, 2016-04-05 19:28:36]

thestig: Please review chrome/
mseaborn: Please review components/nacl
droger: Please review components/translate
avi: Please review content/
piman: Please review ppapi/proxy
sergeyu: Please review remoting/

[piman, 2016-04-05 19:37:19]

lgtm

[Sergey Ulanov, 2016-04-05 19:38:56]

lgtm

[Mark Seaborn, 2016-04-05 19:40:38]

LGTM for components/nacl/

https://codereview.chromium.org/1858973002/diff/20001/ppapi/proxy/ppapi_proxy...
File ppapi/proxy/ppapi_proxy_test.cc (right):

https://codereview.chromium.org/1858973002/diff/20001/ppapi/proxy/ppapi_proxy...
ppapi/proxy/ppapi_proxy_test.cc:257: should_close_source);
Nit: fits on previous line?

[Lei Zhang, 2016-04-05 20:37:29]

chrome/ lgtm

[droger, 2016-04-06 13:52:15]

lgtm

[Avi (use Gerrit), 2016-04-06 16:46:01]

lgtm

[erikchen, 2016-04-06 17:29:07]

The CQ bit was checked by erikchen@chromium.org

[erikchen, 2016-04-06 17:29:07]

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

[commit-bot: I haz the power, 2016-04-06 17:29:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1858973002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1858973002/20001

[commit-bot: I haz the power, 2016-04-06 18:52:06]

Description was changed from

==========
ipc: Rename GetFileHandleForProcess->GetPlatformFileForTransit.

This CL has no intended behavior change. This CL also removes the now
unnecessary parameter |process|.

BUG=493414
==========

to

==========
ipc: Rename GetFileHandleForProcess->GetPlatformFileForTransit.

This CL has no intended behavior change. This CL also removes the now
unnecessary parameter |process|.

BUG=493414
==========

[commit-bot: I haz the power, 2016-04-06 18:52:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-06 18:53:56]

Description was changed from

==========
ipc: Rename GetFileHandleForProcess->GetPlatformFileForTransit.

This CL has no intended behavior change. This CL also removes the now
unnecessary parameter |process|.

BUG=493414
==========

to

==========
ipc: Rename GetFileHandleForProcess->GetPlatformFileForTransit.

This CL has no intended behavior change. This CL also removes the now
unnecessary parameter |process|.

BUG=493414

Committed: https://crrev.com/09b40032d8de63a01eaf62760f2ebc89a813d99c
Cr-Commit-Position: refs/heads/master@{#385512}
==========

[commit-bot: I haz the power, 2016-04-06 18:53:58]

Patchset 2 (id:??) landed as
https://crrev.com/09b40032d8de63a01eaf62760f2ebc89a813d99c
Cr-Commit-Position: refs/heads/master@{#385512}