Fails QUnit.test() after a timeout of 5 seconds.

Fails QUnit.test() after a timeout of 5 seconds.

Summary of changes:
1. Fails QUnit.test() it doesn't return within 5 seconds.
   Timing dependent tests should use fake timers instead.
2. Adds an option to disable the timeout when debugging failed tests
   locally.
3. Logs the current test to the console to ease debugging.

NOTRY=true

Trybots failing on tests that has already been deleted.

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

[kelvinp, 2016-01-26 22:36:45]

Patchset #1 (id:1) has been deleted

[kelvinp, 2016-01-26 22:38:11]

Description was changed from

==========
Fails QUnit tests after a timeout of 5 seconds.

This CL
1. Fails a QUnit browser tests if an individual test fails to return
   within 5 seconds.  Timing dependent tests should use fake timers
   instead.
2. Adds an option to disable the timeout when debugging failed tests
   locally.
3. Logs test output to the console.
==========

to

==========
Fails QUnit.test() after a timeout of 5 seconds.

Summary of changes:
1. Fails QUnit.test() it doesn't return within 5 seconds.  
   Timing dependent tests should use fake timers instead.
2. Adds an option to disable the timeout when debugging failed tests
   locally.
3. Logs the current test to the console to ease debugging.
==========

[kelvinp, 2016-01-26 22:38:11]

kelvinp@chromium.org changed reviewers:
+ jamiewalch@chromium.org

[kelvinp, 2016-01-26 22:57:13]

The CQ bit was checked by kelvinp@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-01-26 22:57:48]

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

[commit-bot: I haz the power, 2016-01-27 00:15:58]

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

[commit-bot: I haz the power, 2016-01-27 00:15:59]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[kelvinp, 2016-01-27 00:46:52]

The CQ bit was checked by kelvinp@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-01-27 00:48:29]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1638753003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1638753003/40001

[commit-bot: I haz the power, 2016-01-27 01:38:01]

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

[commit-bot: I haz the power, 2016-01-27 01:38:01]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[kelvinp, 2016-01-27 01:54:18]

The CQ bit was checked by kelvinp@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-01-27 01:55:22]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1638753003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1638753003/60001

[commit-bot: I haz the power, 2016-01-27 02:41:12]

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

[commit-bot: I haz the power, 2016-01-27 02:41:13]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[kelvinp, 2016-01-27 18:03:04]

Patchset #2 (id:40001) has been deleted

[kelvinp, 2016-01-27 18:03:09]

Patchset #2 (id:60001) has been deleted

[kelvinp, 2016-01-27 18:04:33]

On 2016/01/27 02:41:13, commit-bot: I haz the power wrote:
> Dry run: Try jobs failed on following builders:
>   linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

PTAL

[Jamie, 2016-01-27 18:16:12]

lgtm

[kelvinp, 2016-01-27 18:23:11]

The CQ bit was checked by kelvinp@chromium.org

[commit-bot: I haz the power, 2016-01-27 18:23:54]

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

[commit-bot: I haz the power, 2016-01-27 19:22:45]

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

[commit-bot: I haz the power, 2016-01-27 19:22:51]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[kelvinp, 2016-01-27 21:52:05]

Description was changed from

==========
Fails QUnit.test() after a timeout of 5 seconds.

Summary of changes:
1. Fails QUnit.test() it doesn't return within 5 seconds.  
   Timing dependent tests should use fake timers instead.
2. Adds an option to disable the timeout when debugging failed tests
   locally.
3. Logs the current test to the console to ease debugging.
==========

to

==========
Fails QUnit.test() after a timeout of 5 seconds.

Summary of changes:
1. Fails QUnit.test() it doesn't return within 5 seconds.  
   Timing dependent tests should use fake timers instead.
2. Adds an option to disable the timeout when debugging failed tests
   locally.
3. Logs the current test to the console to ease debugging.

NOTRY=true

Trybots failing on tests that has already been deleted.
==========

[kelvinp, 2016-01-27 21:53:56]

Description was changed from

==========
Fails QUnit.test() after a timeout of 5 seconds.

Summary of changes:
1. Fails QUnit.test() it doesn't return within 5 seconds.  
   Timing dependent tests should use fake timers instead.
2. Adds an option to disable the timeout when debugging failed tests
   locally.
3. Logs the current test to the console to ease debugging.

NOTRY=true

Trybots failing on tests that has already been deleted.
==========

to

==========
Fails QUnit.test() after a timeout of 5 seconds.

Summary of changes:
1. Fails QUnit.test() it doesn't return within 5 seconds.  
   Timing dependent tests should use fake timers instead.
2. Adds an option to disable the timeout when debugging failed tests
   locally.
3. Logs the current test to the console to ease debugging.

NO_TRY=true

Trybots failing on tests that has already been deleted.
==========

[kelvinp, 2016-01-27 21:54:03]

The CQ bit was checked by kelvinp@chromium.org

[commit-bot: I haz the power, 2016-01-27 21:55:36]

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

[kelvinp, 2016-01-27 22:04:03]

The CQ bit was unchecked by kelvinp@chromium.org

[kelvinp, 2016-01-27 22:04:16]

Description was changed from

==========
Fails QUnit.test() after a timeout of 5 seconds.

Summary of changes:
1. Fails QUnit.test() it doesn't return within 5 seconds.  
   Timing dependent tests should use fake timers instead.
2. Adds an option to disable the timeout when debugging failed tests
   locally.
3. Logs the current test to the console to ease debugging.

NO_TRY=true

Trybots failing on tests that has already been deleted.
==========

to

==========
Fails QUnit.test() after a timeout of 5 seconds.

Summary of changes:
1. Fails QUnit.test() it doesn't return within 5 seconds.  
   Timing dependent tests should use fake timers instead.
2. Adds an option to disable the timeout when debugging failed tests
   locally.
3. Logs the current test to the console to ease debugging.

NOTRY=true

Trybots failing on tests that has already been deleted.
==========

[kelvinp, 2016-01-27 22:04:21]

The CQ bit was checked by kelvinp@chromium.org

[commit-bot: I haz the power, 2016-01-27 22:07:15]

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

[commit-bot: I haz the power, 2016-01-27 22:15:25]

Description was changed from

==========
Fails QUnit.test() after a timeout of 5 seconds.

Summary of changes:
1. Fails QUnit.test() it doesn't return within 5 seconds.  
   Timing dependent tests should use fake timers instead.
2. Adds an option to disable the timeout when debugging failed tests
   locally.
3. Logs the current test to the console to ease debugging.

NOTRY=true

Trybots failing on tests that has already been deleted.
==========

to

==========
Fails QUnit.test() after a timeout of 5 seconds.

Summary of changes:
1. Fails QUnit.test() it doesn't return within 5 seconds.  
   Timing dependent tests should use fake timers instead.
2. Adds an option to disable the timeout when debugging failed tests
   locally.
3. Logs the current test to the console to ease debugging.

NOTRY=true

Trybots failing on tests that has already been deleted.
==========

[commit-bot: I haz the power, 2016-01-27 22:15:26]

Committed patchset #2 (id:80001)

[commit-bot: I haz the power, 2016-01-27 22:17:42]

Description was changed from

==========
Fails QUnit.test() after a timeout of 5 seconds.

Summary of changes:
1. Fails QUnit.test() it doesn't return within 5 seconds.  
   Timing dependent tests should use fake timers instead.
2. Adds an option to disable the timeout when debugging failed tests
   locally.
3. Logs the current test to the console to ease debugging.

NOTRY=true

Trybots failing on tests that has already been deleted.
==========

to

==========
Fails QUnit.test() after a timeout of 5 seconds.

Summary of changes:
1. Fails QUnit.test() it doesn't return within 5 seconds.
   Timing dependent tests should use fake timers instead.
2. Adds an option to disable the timeout when debugging failed tests
   locally.
3. Logs the current test to the console to ease debugging.

NOTRY=true

Trybots failing on tests that has already been deleted.

Committed: https://crrev.com/ed48a6910aace9ad86ef10c2ecdfeae1bb31a40c
Cr-Commit-Position: refs/heads/master@{#371890}
==========

[commit-bot: I haz the power, 2016-01-27 22:17:43]

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

[kelvinp, 2016-01-27 23:38:19]

On 2016/01/27 22:17:43, commit-bot: I haz the power wrote:
> Patchset 2 (id:??) landed as
> https://crrev.com/ed48a6910aace9ad86ef10c2ecdfeae1bb31a40c
> Cr-Commit-Position: refs/heads/master@{#371890}

This change might require a clobber for the bots to cycle green. I have already
clobbered the linux build bot.

[kelvinp, 2016-01-28 01:59:51]

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

The reason for reverting is: Test failing on Linux and Mac..

[battre, 2016-01-28 08:38:34]

battre@chromium.org changed reviewers:
+ battre@chromium.org

[battre, 2016-01-28 08:38:35]

Please always reference a bug number in your change list description.

[kelvinp, 2016-01-29 00:55:04]

Description was changed from

==========
Fails QUnit.test() after a timeout of 5 seconds.

Summary of changes:
1. Fails QUnit.test() it doesn't return within 5 seconds.
   Timing dependent tests should use fake timers instead.
2. Adds an option to disable the timeout when debugging failed tests
   locally.
3. Logs the current test to the console to ease debugging.

NOTRY=true

Trybots failing on tests that has already been deleted.

Committed: https://crrev.com/ed48a6910aace9ad86ef10c2ecdfeae1bb31a40c
Cr-Commit-Position: refs/heads/master@{#371890}
==========

to

==========
Fails QUnit.test() after a timeout of 5 seconds.

Summary of changes:
1. Fails QUnit.test() it doesn't return within 5 seconds.
   Timing dependent tests should use fake timers instead.
2. Adds an option to disable the timeout when debugging failed tests
   locally.
3. Logs the current test to the console to ease debugging.

NOTRY=true

Trybots failing on tests that has already been deleted.

Committed: https://crrev.com/ed48a6910aace9ad86ef10c2ecdfeae1bb31a40c
Cr-Commit-Position: refs/heads/master@{#371890}
==========