Make the default testharness.js timeout longer than the LayoutTests timeout

Make the default testharness.js timeout longer than the LayoutTests timeout

With the previous defaults, tests could timeout before the LayoutTests
timeout, which would be reported as a failure. Making the timeout very
long should fix tests that are flaky with the 2 second timeout and will
make the SlowTests mechanism work for testharness.js tests, should the
need arise.

BUG=355489
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170136

[philipj_slow, 2014-03-26 17:24:28]

Instead of https://codereview.chromium.org/209383013/

[acolwell GONE FROM CHROMIUM, 2014-03-26 17:54:31]

If we do this, I think we should also update
LayoutTests/http/tests/w3c/resources/testharness.js with the same values as
well.

looks good to me, but I definitely think Dirk should have the final say.

[philipj_slow, 2014-03-26 18:00:29]

Oh, I didn't realize we have multiple copies :) Will have to look into that if
Dirk thinks this is a sane approach.

[acolwell GONE FROM CHROMIUM, 2014-03-26 18:22:34]

On 2014/03/26 18:00:29, philipj wrote:
> Oh, I didn't realize we have multiple copies :) Will have to look into that if
> Dirk thinks this is a sane approach.

Unfortunately yes for various historical reasons. I don't remember the exact
reason why we don't have one copy yet. I'm pretty sure it's on a TODO list
somewhere. :)

[Dirk Pranke, 2014-03-26 19:41:05]

lgtm.

whenever I (or someone else) gets around to fully integrating the http-requiring
tests from the w3c as part of the import step, we'll de-dup the testharness.js
copies.

[philipj_slow, 2014-03-27 01:36:08]

OK, I'll make the same edit to all testharness.js and give it a go then!

[philipj_slow, 2014-03-27 02:35:16]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2014-03-27 02:35:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/philipj@opera.com/213283002/40001

[commit-bot: I haz the power, 2014-03-27 02:45:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-27 02:45:57]

Try jobs failed on following builders:
  tryserver.blink on win_blink_rel

[philipj_slow, 2014-03-27 03:02:04]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2014-03-27 03:02:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/philipj@opera.com/213283002/40001

[commit-bot: I haz the power, 2014-03-27 03:45:27]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-27 03:45:27]

Try jobs failed on following builders:
  tryserver.blink on linux_blink_dbg

[philipj_slow, 2014-03-27 04:17:41]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2014-03-27 04:17:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/philipj@opera.com/213283002/40001

[commit-bot: I haz the power, 2014-03-27 05:19:52]

Change committed as 170136