android: Remove ChildProcessServiceImpl.sContext

android: Remove ChildProcessServiceImpl.sContext

getContext is never called so the only use case left is checking that
create is not called more than once in this process. It's an
AutomicReference, but is not used atomically (ie it's not really calling
compareAndSet). So it can just replace it with a bool instead.

BUG=689758
Review-Url: https://codereview.chromium.org/2861673009
Cr-Commit-Position: refs/heads/master@{#469832}
Committed: https://chromium.googlesource.com/chromium/src/+/56de014ae2e5f80d5a009401d8d513314b3b5a6d

[boliu, 2017-05-05 17:39:21]

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

[boliu, 2017-05-05 17:39:22]

ptal, minor clean up

[boliu, 2017-05-05 17:39:28]

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

[commit-bot: I haz the power, 2017-05-05 17:39:44]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[agrieve, 2017-05-05 17:54:29]

lgtm

[boliu, 2017-05-05 17:56:39]

The CQ bit was unchecked by boliu@chromium.org

[boliu, 2017-05-05 17:56:44]

The CQ bit was checked by boliu@chromium.org

[commit-bot: I haz the power, 2017-05-05 17:57:08]

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-05-05 21:54:59]

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

[commit-bot: I haz the power, 2017-05-05 21:55:00]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[boliu, 2017-05-05 23:01:20]

The CQ bit was checked by boliu@chromium.org

[commit-bot: I haz the power, 2017-05-05 23:03:43]

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-05-06 00:49:53]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494025280673150, "parent_rev":
"85f33e0246ffae9721367c027a974642f036eb03", "commit_rev":
"56de014ae2e5f80d5a009401d8d513314b3b5a6d"}

[commit-bot: I haz the power, 2017-05-06 00:50:04]

Description was changed from

==========
android: Remove ChildProcessServiceImpl.sContext

getContext is never called so the only use case left is checking that
create is not called more than once in this process. It's an
AutomicReference, but is not used atomically (ie it's not really calling
compareAndSet). So it can just replace it with a bool instead.

BUG=689758
==========

to

==========
android: Remove ChildProcessServiceImpl.sContext

getContext is never called so the only use case left is checking that
create is not called more than once in this process. It's an
AutomicReference, but is not used atomically (ie it's not really calling
compareAndSet). So it can just replace it with a bool instead.

BUG=689758

Review-Url: https://codereview.chromium.org/2861673009
Cr-Commit-Position: refs/heads/master@{#469832}
Committed:
https://chromium.googlesource.com/chromium/src/+/56de014ae2e5f80d5a009401d8d5...
==========

[commit-bot: I haz the power, 2017-05-06 00:50:05]

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