Fix flakiness of ExecuteScriptApiTest.FrameWithHttp204

Fix flakiness of ExecuteScriptApiTest.FrameWithHttp204

BUG=585471
Committed: https://crrev.com/91e1a50d7a74d1a8d9ac6f81cca394a826d14354
Cr-Commit-Position: refs/heads/master@{#374798}

[robwu, 2016-02-10 21:51:59]

rob@robwu.nl changed reviewers:
+ creis@chromium.org

[Charlie Reis, 2016-02-10 22:00:11]

creis@chromium.org changed reviewers:
+ nasko@chromium.org

[Charlie Reis, 2016-02-10 22:00:12]

I'm pretty rusty on extension test code.  Nasko, can you review?

(Side note: Any chance this will help with https://crbug.com/583031, or does the
test still fail when run with --isolate-extensions?)

[Charlie Reis, 2016-02-10 22:00:21]

Description was changed from

==========
Fix flakiness of ExecuteScriptApiTest.FrameWithHttp204

BUG=585471
==========

to

==========
Fix flakiness of ExecuteScriptApiTest.FrameWithHttp204

BUG=585471
==========

[Charlie Reis, 2016-02-10 22:00:21]

creis@chromium.org changed reviewers:
- creis@chromium.org

[robwu, 2016-02-10 22:14:56]

On 2016/02/10 22:00:12, Charlie Reis wrote:
> I'm pretty rusty on extension test code.  Nasko, can you review?
> 
> (Side note: Any chance this will help with https://crbug.com/583031, or does
the
> test still fail when run with --isolate-extensions?)

This does not help against https://crbug.com/583031. To fix that,
--site-per-process and --isolate-extensions have to be separate flags. But Nasko
was working on webNavigation code, which might fix this issue. If it is
important to unblock the tests, then I can submit a simple patch that separates
the flags, otherwise let's just wait for Nasko's work on webNavigation.

[ncarter (slow), 2016-02-10 22:21:56]

nick@chromium.org changed reviewers:
+ nick@chromium.org
- nasko@chromium.org

[ncarter (slow), 2016-02-10 22:21:57]

lgtm (though it would be nicer, of course, to force the race to resolve in a
deterministic way, I don't have any ideas for how we'd do that, short of adding
server-side delay)

[robwu, 2016-02-10 22:23:21]

The CQ bit was checked by rob@robwu.nl

[commit-bot: I haz the power, 2016-02-10 22:25:55]

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

[commit-bot: I haz the power, 2016-02-11 00:12:01]

Description was changed from

==========
Fix flakiness of ExecuteScriptApiTest.FrameWithHttp204

BUG=585471
==========

to

==========
Fix flakiness of ExecuteScriptApiTest.FrameWithHttp204

BUG=585471
==========

[commit-bot: I haz the power, 2016-02-11 00:12:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-16 22:33:02]

Description was changed from

==========
Fix flakiness of ExecuteScriptApiTest.FrameWithHttp204

BUG=585471
==========

to

==========
Fix flakiness of ExecuteScriptApiTest.FrameWithHttp204

BUG=585471

Committed: https://crrev.com/91e1a50d7a74d1a8d9ac6f81cca394a826d14354
Cr-Commit-Position: refs/heads/master@{#374798}
==========

[commit-bot: I haz the power, 2016-02-16 22:33:03]

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