Test that 404 subresource bodies with nested message loops don't crash.

Test that 404 subresource bodies with nested message loops don't crash.

This is a browser test because DumpRenderTree doesn't run nested message loops. The failure case was that a nested message triggered from an element that has signalled an error but had an open request would receive a body for the request and crash/fail an assertion.

This test checks the WebKit side fix for this issue, see https://bugs.webkit.org/show_bug.cgi?id=103563

R=sky@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=176882

[gavinp, 2013-01-10 03:13:25]

sky,

What do you think?

[sky, 2013-01-10 16:30:34]

Where is the nested message loop?

[gavinp, 2013-01-10 18:59:17]

On 2013/01/10 16:30:34, sky wrote:
> Where is the nested message loop?

I have copied the comment from webkit_browsertest.cc up into the issue. The
origin of the nested message loop is now documented in all three of the issue
description, the cc file comment and the HTML file comment. Before, it was 2/3,
which isn't bad. :D

The WebKit fix is a bit delayed here, so this test currently crashes. I'm hoping
to land something like this test as soon as a good WebKit fix is in.

[gavinp, 2013-01-10 19:06:03]

And I engrammared the issue description, too.

[sky, 2013-01-10 21:03:53]

LGTM

[commit-bot: I haz the power, 2013-01-14 15:30:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gavinp@chromium.org/11778083/1

[commit-bot: I haz the power, 2013-01-14 16:17:25]

Retried try job too often on linux_aura for step(s) content_browsertests

[gavinp, 2013-01-14 16:52:50]

On 2013/01/14 16:17:25, I haz the power (commit-bot) wrote:
> Retried try job too often on linux_aura for step(s) content_browsertests

That's to be expected; win_aura was running without the webkit test. I am
surprised lkgr didn't apply here, but I'll wait a day and give it another shot.

[gavinp, 2013-01-14 16:53:07]

On 2013/01/14 16:52:50, gavinp wrote:
> On 2013/01/14 16:17:25, I haz the power (commit-bot) wrote:
> > Retried try job too often on linux_aura for step(s) content_browsertests
> 
> That's to be expected; win_aura was running without the webkit test. I am

webkit test --> webkit side fix.

> surprised lkgr didn't apply here, but I'll wait a day and give it another
shot.

[commit-bot: I haz the power, 2013-01-15 10:15:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gavinp@chromium.org/11778083/1

[commit-bot: I haz the power, 2013-01-15 11:16:22]

Change committed as 176882