[Extensions] Chromium crashed in single-process mode on fresh launch.

[Extensions] Chromium crashed in single-process mode on fresh launch.

With current implementation, in-process utility thread clean up is taken care by IO thread on detecting that concern IPC communication channel been closed. And utility thread has ThreadRestrictions to disallowed IO. Apparently it asserts (SIG-ABORT) before performing join in pre-check.

With this CL we are making sure that before IO thread waits for utility thread's cleanup it enables ThreadRestrictions to allow IO for utility thread so that it can join. This avoids chromium crash on fresh launch.

BUG=421007
R=mpcomplete@chromium.org

Committed: https://crrev.com/e82a809ff6040ab6a752eef2b97502bfbf9e8108
Cr-Commit-Position: refs/heads/master@{#302112}

[Pritam Nikam, 2014-10-29 06:23:14]

Patchset #1 (id:1) has been deleted

[Pritam Nikam, 2014-10-29 06:25:55]

pritam.nikam@samsung.com changed reviewers:
+ rockot@chromium.org

[Pritam Nikam, 2014-10-29 06:27:25]

Hi Ken,
PTAL, Thanks!

[Ken Rockot(use gerrit already), 2014-10-29 14:04:15]

On 2014/10/29 06:27:25, Pritam Nikam wrote:
> Hi Ken,
> PTAL, Thanks!

I don't think we want to do this, but in any case I am not an OWNER of this
file.

[Pritam Nikam, 2014-10-29 14:32:52]

On 2014/10/29 14:04:15, Ken Rockot wrote:
> On 2014/10/29 06:27:25, Pritam Nikam wrote:
> > Hi Ken,
> > PTAL, Thanks!
> 
> I don't think we want to do this, but in any case I am not an OWNER of this
> file.

Thanks Ken.

What would be appropriate way to approach this problem?
Any pointers would be great help.

Thanks!

[Pritam Nikam, 2014-10-30 08:50:51]

pritam.nikam@samsung.com changed reviewers:
+ miket@chromium.org, mpcomplete@chromium.org

[Pritam Nikam, 2014-10-30 08:55:17]

+ Matt & miket.

I encounter this crash (SIG-ABORT) on fresh launch chromium in single process
mode. With this CL, seems solving the problem. Please review this patch set. 

Thanks!

[Matt Perry, 2014-10-30 16:26:40]

lgtm

[Pritam Nikam, 2014-10-30 17:31:40]

The CQ bit was checked by pritam.nikam@samsung.com

[commit-bot: I haz the power, 2014-10-30 17:33:25]

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

[commit-bot: I haz the power, 2014-10-30 18:26:00]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2014-10-30 18:26:35]

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