Android: Fix GPU.GPUProcessTerminationStatus UMA stat

Android: Fix GPU.GPUProcessTerminationStatus UMA stat

ChildProcessConnectionImpl.mAlwaysInForeground is used
for the GPU process and indicates that it does not participate
in hiding like renderers. The 'important' binding gives it the
same priority as the main application, so upon termination we
should just use the app's foreground status to determine whether
the GPU process was in the foreground or not.

BUG=462422
Committed: https://crrev.com/479947d99fd817737f517c068a77d74cd08183f8
Cr-Commit-Position: refs/heads/master@{#319768}

[no sievers, 2015-03-09 22:29:53]

sievers@chromium.org changed reviewers:
+ tedchoc@chromium.org

[no sievers, 2015-03-09 22:31:51]

Ted, patl.

A quick fix for the UMA reporting (to correctly differentiate in between crashed
vs. was killed by the system).

I'm also reworking this a bit more to subclass the ManagedConnections instead of
these hacks (there might also be a few other bugs, and the code is becoming
confusing), but that patch won't be mergeable.

[Ted C, 2015-03-09 22:35:03]

lgtm

https://codereview.chromium.org/987193003/diff/20001/content/public/android/j...
File
content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java
(right):

https://codereview.chromium.org/987193003/diff/20001/content/public/android/j...
content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java:432:
private static boolean sApplicationInForeground = true;
move this up with the other variables (i.e. right under
sSurfaceTextureSurfaceMap)

[no sievers, 2015-03-09 22:38:37]

https://codereview.chromium.org/987193003/diff/20001/content/public/android/j...
File
content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java
(right):

https://codereview.chromium.org/987193003/diff/20001/content/public/android/j...
content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java:432:
private static boolean sApplicationInForeground = true;
On 2015/03/09 22:35:03, Ted C wrote:
> move this up with the other variables (i.e. right under
> sSurfaceTextureSurfaceMap)

Done.

[no sievers, 2015-03-09 22:39:02]

The CQ bit was checked by sievers@chromium.org

[no sievers, 2015-03-09 22:39:03]

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

[commit-bot: I haz the power, 2015-03-09 22:39:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/987193003/40001

[commit-bot: I haz the power, 2015-03-09 23:29:48]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-03-09 23:31:12]

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