Don't choose a background process when choosing a process for starting a Service Worker.

Don't choose a background process when choosing a process for starting a Service Worker.

This is a quick fix for https://crbug.com/531345.
We need to revisit the behavior of ServiceWorker and the timer suspension of background processes which was intoruced by https://codereview.chromium.org/1133143003.

BUG=531345
Committed: https://crrev.com/3ddd919b0f5550377e178ba6d4aa5256701d53ff
Cr-Commit-Position: refs/heads/master@{#351530}

[horo, 2015-09-30 08:45:27]

horo@chromium.org changed reviewers:
+ kinuko@chromium.org

[horo, 2015-09-30 08:45:28]

kinuko@
Could you please review this?

[kinuko, 2015-09-30 09:26:15]

Thanks for making this... Were you able to test the actual behavior?

https://codereview.chromium.org/1373853003/diff/1/content/browser/service_wor...
File content/browser/service_worker/service_worker_process_manager.cc (right):

https://codereview.chromium.org/1373853003/diff/1/content/browser/service_wor...
content/browser/service_worker/service_worker_process_manager.cc:31: if
(static_cast<RenderProcessHostImpl*>(rph)->backgrounded())
nit: could we add a comment (like what you wrote in the issue's description) and
link to the bug if this is intended to be a short term fix?

[horo, 2015-09-30 09:45:37]

> Were you able to test the actual behavior?

Yes. Done manually.

https://codereview.chromium.org/1373853003/diff/1/content/browser/service_wor...
File content/browser/service_worker/service_worker_process_manager.cc (right):

https://codereview.chromium.org/1373853003/diff/1/content/browser/service_wor...
content/browser/service_worker/service_worker_process_manager.cc:31: if
(static_cast<RenderProcessHostImpl*>(rph)->backgrounded())
On 2015/09/30 09:26:15, kinuko wrote:
> nit: could we add a comment (like what you wrote in the issue's description)
and
> link to the bug if this is intended to be a short term fix?

Done.

[kinuko, 2015-09-30 09:53:44]

lgtm.
lgtm

[horo, 2015-09-30 10:09:19]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2015-09-30 10:09:50]

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

[commit-bot: I haz the power, 2015-09-30 11:04:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-30 11:05:27]

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

[horo, 2015-10-01 03:43:43]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1380023002/ by horo@chromium.org.

The reason for reverting is: - This patch will increase memory pressure to
launch a new SW process.
- More reasonable patch (crrev.com/351714) landed..