Moving IChildProcessService to base/

Moving IChildProcessService to base/

As part of the effort of moving Android process launching to base/,
moving IChildProcessService.aidl to base/.

BUG=702316
Review-Url: https://codereview.chromium.org/2777013005
Cr-Commit-Position: refs/heads/master@{#461020}
Committed: https://chromium.googlesource.com/chromium/src/+/ce6ef2968b6785ccc8c29bf8b5954b5242845552

[Jay Civelli, 2017-03-28 21:07:38]

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

[commit-bot: I haz the power, 2017-03-28 21:08:19]

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, 2017-03-28 21:09:00]

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

[commit-bot: I haz the power, 2017-03-28 21:09:02]

Dry run: CQ has no permission to schedule in bucket
master.tryserver.chromium.linux

[Jay Civelli, 2017-03-28 21:24:10]

Patchset #1 (id:1) has been deleted

[Jay Civelli, 2017-03-28 21:24:16]

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

[commit-bot: I haz the power, 2017-03-28 21:24:26]

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, 2017-03-28 21:24:40]

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

[commit-bot: I haz the power, 2017-03-28 21:24:41]

Dry run: CQ has no permission to schedule in bucket
master.tryserver.chromium.win

[Jay Civelli, 2017-03-28 22:03:17]

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

[commit-bot: I haz the power, 2017-03-28 22:03:42]

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, 2017-03-29 00:06:00]

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

[commit-bot: I haz the power, 2017-03-29 00:06:01]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_TIMED_OUT, build hasn't
started yet, builder probably lacks capacity)

[Jay Civelli, 2017-03-29 00:26:58]

jcivelli@chromium.org changed reviewers:
+ boliu@chromium.org

[boliu, 2017-03-29 01:07:02]

lgtm

https://codereview.chromium.org/2777013005/diff/20001/base/android/java/src/o...
File
base/android/java/src/org/chromium/base/process_launcher/IChildProcessService.aidl
(right):

https://codereview.chromium.org/2777013005/diff/20001/base/android/java/src/o...
base/android/java/src/org/chromium/base/process_launcher/IChildProcessService.aidl:5:
package org.chromium.base.process_launcher;
are security owners set up for base?

[Jay Civelli, 2017-03-29 16:12:17]

https://codereview.chromium.org/2777013005/diff/20001/base/android/java/src/o...
File
base/android/java/src/org/chromium/base/process_launcher/IChildProcessService.aidl
(right):

https://codereview.chromium.org/2777013005/diff/20001/base/android/java/src/o...
base/android/java/src/org/chromium/base/process_launcher/IChildProcessService.aidl:5:
package org.chromium.base.process_launcher;
On 2017/03/29 01:07:01, boliu wrote:
> are security owners set up for base?

Yes, this directory has AIDL files OWNERS set as IPC security owners.

[Jay Civelli, 2017-03-29 16:14:24]

Description was changed from

==========
Moving IChildProcessService to base/

As part of the effort of moving Android process launching to base/,
moving IChildProcessService.aidl to base/.

BUG=702316
==========

to

==========
Moving IChildProcessService to base/

As part of the effort of moving Android process launching to base/,
moving IChildProcessService.aidl to base/.

BUG=702316
==========

[Jay Civelli, 2017-03-29 16:14:24]

jcivelli@chromium.org changed reviewers:
+ palmer@chromium.org

[Jay Civelli, 2017-03-29 16:14:49]

Adding palmer@ for security review of AIDL changes.

[palmer, 2017-03-30 01:25:54]

lgtm

[Jay Civelli, 2017-03-30 01:43:03]

The CQ bit was checked by jcivelli@chromium.org

[commit-bot: I haz the power, 2017-03-30 01:44: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, 2017-03-30 01:55:29]

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

[commit-bot: I haz the power, 2017-03-30 01:55:34]

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...)

[Jay Civelli, 2017-03-30 03:28:10]

jcivelli@chromium.org changed reviewers:
+ nyquist@chromium.org

[Jay Civelli, 2017-03-30 03:28:11]

Adding nyquist@ for base/android OWNERS review

[nyquist, 2017-03-30 20:44:17]

lgtm

[Jay Civelli, 2017-03-30 20:44:56]

The CQ bit was checked by jcivelli@chromium.org

[commit-bot: I haz the power, 2017-03-30 20:46:10]

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, 2017-03-30 23:08:17]

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

[commit-bot: I haz the power, 2017-03-30 23:08:19]

Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Jay Civelli, 2017-03-31 01:46:46]

The CQ bit was checked by jcivelli@chromium.org

[commit-bot: I haz the power, 2017-03-31 01:47:31]

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, 2017-03-31 03:58:32]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490924806286890,
"parent_rev": "829705bfdb5c38eb06f1ab014b9f7b0f317f72b9", "commit_rev":
"ce6ef2968b6785ccc8c29bf8b5954b5242845552"}

[commit-bot: I haz the power, 2017-03-31 03:59:21]

Description was changed from

==========
Moving IChildProcessService to base/

As part of the effort of moving Android process launching to base/,
moving IChildProcessService.aidl to base/.

BUG=702316
==========

to

==========
Moving IChildProcessService to base/

As part of the effort of moving Android process launching to base/,
moving IChildProcessService.aidl to base/.

BUG=702316

Review-Url: https://codereview.chromium.org/2777013005
Cr-Commit-Position: refs/heads/master@{#461020}
Committed:
https://chromium.googlesource.com/chromium/src/+/ce6ef2968b6785ccc8c29bf8b595...
==========

[commit-bot: I haz the power, 2017-03-31 03:59:22]

Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/ce6ef2968b6785ccc8c29bf8b595...