Skip wpt/workers/semantics/encodings until crbug.com/656171 is resolved.

Skip wpt/workers/semantics/encodings until crbug.com/656171 is resolved.

BUG=656171
Committed: https://crrev.com/096c5be32530dd75f9912c0d08230149195cf61d
Cr-Commit-Position: refs/heads/master@{#426118}

[qyearsley, 2016-10-18 05:16:09]

The CQ bit was checked by [email protected] to run a CQ dry run

[qyearsley, 2016-10-18 05:16:19]

[email protected] changed reviewers:
+ [email protected], [email protected]

[commit-bot: I haz the power, 2016-10-18 05:16:31]

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

[falken, 2016-10-18 05:20:13]

I'm in favor of this but should we remove the existing
wpt/workers/semantics/encodings/* files (or is that taken care of on the next
roll?)

Also, should we remove the wpt/workers/semantics/encodings/* entries in
TestExpectations?

[commit-bot: I haz the power, 2016-10-18 06:00:22]

The CQ bit was unchecked by [email protected]

[commit-bot: I haz the power, 2016-10-18 06:00:23]

Dry run: This issue passed the CQ dry run.

[qyearsley, 2016-10-18 16:20:56]

On 2016/10/18 at 05:20:13, falken wrote:
> I'm in favor of this but should we remove the existing
wpt/workers/semantics/encodings/* files (or is that taken care of on the next
roll?)

I think it would be taken care of in the next roll, but there's no harm in doing
it in this CL.

> Also, should we remove the wpt/workers/semantics/encodings/* entries in
TestExpectations?

Good point, now done.

[qyearsley, 2016-10-18 16:21:00]

The CQ bit was checked by [email protected] to run a CQ dry run

[commit-bot: I haz the power, 2016-10-18 16:21:23]

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, 2016-10-18 19:59:06]

The CQ bit was unchecked by [email protected]

[commit-bot: I haz the power, 2016-10-18 19:59:07]

Dry run: This issue passed the CQ dry run.

[falken, 2016-10-19 00:07:39]

lgtm

[qyearsley, 2016-10-19 00:16:21]

The CQ bit was checked by [email protected]

[commit-bot: I haz the power, 2016-10-19 00:16:50]

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, 2016-10-19 01:35:21]

The CQ bit was unchecked by [email protected]

[commit-bot: I haz the power, 2016-10-19 01:35:22]

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

[nhiroki, 2016-10-19 01:41:24]

Thank you for working on this. LGTM

[falken, 2016-10-19 01:57:17]

The CQ bit was checked by [email protected]

[commit-bot: I haz the power, 2016-10-19 01:57: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, 2016-10-19 02:18:22]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-21 13:05:50]

Description was changed from

==========
Skip wpt/workers/semantics/encodings until crbug.com/656171 is resolved.

BUG=656171
==========

to

==========
Skip wpt/workers/semantics/encodings until crbug.com/656171 is resolved.

BUG=656171

Committed: https://crrev.com/096c5be32530dd75f9912c0d08230149195cf61d
Cr-Commit-Position: refs/heads/master@{#426118}
==========

[commit-bot: I haz the power, 2016-10-21 13:05:51]

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