Remove wpt/encoding/textdecoder-labels.html from SlowTests.

Remove wpt/encoding/textdecoder-labels.html from SlowTests.

After https://github.com/w3c/web-platform-tests/pull/4290, I believe this test should no longer be slow.

BUG=671164
Review-Url: https://codereview.chromium.org/2646883002
Cr-Commit-Position: refs/heads/master@{#445112}
Committed: https://chromium.googlesource.com/chromium/src/+/2c06df626151088d15a0e134dbd7c49350d0f217

[qyearsley, 2017-01-20 00:44:54]

qyearsley@chromium.org changed reviewers:
+ jsbell@chromium.org

[qyearsley, 2017-01-20 00:44:56]

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

[commit-bot: I haz the power, 2017-01-20 00:46:37]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-20 01:44:39]

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

[commit-bot: I haz the power, 2017-01-20 01:44:40]

Dry run: This issue passed the CQ dry run.

[qyearsley, 2017-01-20 17:49:10]

Description was changed from

==========
Tentatively remove .../textdecoder-labels.html from SlowTests.

After https://github.com/w3c/web-platform-tests/pull/4290 this
test may no longer be slow.

BUG=671164
==========

to

==========
Remove wpt/encoding/textdecoder-labels.html from SlowTests.

After https://github.com/w3c/web-platform-tests/pull/4290, I believe this test
should no longer be slow.

BUG=671164
==========

[jsbell, 2017-01-20 17:58:19]

Hrm... that PR never got accepted.

(I filed it then forgot about it)

Are you somehow testing with it? Did someone else land the equivalent?

[qyearsley, 2017-01-20 18:01:31]

On 2017/01/20 at 17:58:19, jsbell wrote:
> Hrm... that PR never got accepted.
> 
> (I filed it then forgot about it)
> 
> Are you somehow testing with it? Did someone else land the equivalent?

Ah, I just tested by running the test once to see whether it seemed particularly
slow :-/ It took 639 ms, well within the regular timeout.

Does it sound OK to try removing it from SlowTests now anyway, and see whether
it times out on the MSAN bot again?

[jsbell, 2017-01-20 18:51:44]

On 2017/01/20 18:01:31, qyearsley wrote:
> Does it sound OK to try removing it from SlowTests now anyway, and see whether
> it times out on the MSAN bot again?

No objection as long as you're watching!

lgtm

[qyearsley, 2017-01-20 18:52:37]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2017-01-20 18:52:58]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-20 18:58:08]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484938357350920, "parent_rev":
"15ef0aa9060364567a6cf2abd20f6d419619cc83", "commit_rev":
"2c06df626151088d15a0e134dbd7c49350d0f217"}

[commit-bot: I haz the power, 2017-01-20 18:58:41]

Description was changed from

==========
Remove wpt/encoding/textdecoder-labels.html from SlowTests.

After https://github.com/w3c/web-platform-tests/pull/4290, I believe this test
should no longer be slow.

BUG=671164
==========

to

==========
Remove wpt/encoding/textdecoder-labels.html from SlowTests.

After https://github.com/w3c/web-platform-tests/pull/4290, I believe this test
should no longer be slow.

BUG=671164

Review-Url: https://codereview.chromium.org/2646883002
Cr-Commit-Position: refs/heads/master@{#445112}
Committed:
https://chromium.googlesource.com/chromium/src/+/2c06df626151088d15a0e134dbd7...
==========

[commit-bot: I haz the power, 2017-01-20 18:58:42]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/2c06df626151088d15a0e134dbd7...