[System-Keyboard-Lock] Restore keyboard-lock tests

[System-Keyboard-Lock] Restore keyboard-lock tests

The related test cases are removed by change
https://chromium-review.googlesource.com/c/499507/ because of a wrong annotation
in W3CImportExpectations file. The change to correct W3CImprotExpectations will
be submitted by change https://codereview.chromium.org/2916803004/. So this
change restores the original test files.

NOEXPORT=true
BUG=680809
Review-Url: https://codereview.chromium.org/2915203003
Cr-Commit-Position: refs/heads/master@{#476843}
Committed: https://chromium.googlesource.com/chromium/src/+/cc0c8cf835d5656666dbc7266d6f542d6607bc1f

[Hzj_jie, 2017-06-02 17:50:31]

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

[commit-bot: I haz the power, 2017-06-02 17:50:53]

Dry run: CQ is trying da patch.

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

[Hzj_jie, 2017-06-02 17:57:29]

Description was changed from

==========
Restore keyboard-lock tests

BUG=
==========

to

==========
[System-Keyboard-Lock] Restore keyboard-lock tests

The related test cases are removed by change
https://chromium-review.googlesource.com/c/499507/ because of a wrong annotation
in W3CImportExpectations file. The change to correct W3CImprotExpectations will
be submitted by change https://codereview.chromium.org/2916803004/. So this
change restores the original test files.

BUG=680809
==========

[Hzj_jie, 2017-06-02 17:57:30]

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

[Hzj_jie, 2017-06-02 17:58:52]

Description was changed from

==========
[System-Keyboard-Lock] Restore keyboard-lock tests

The related test cases are removed by change
https://chromium-review.googlesource.com/c/499507/ because of a wrong annotation
in W3CImportExpectations file. The change to correct W3CImprotExpectations will
be submitted by change https://codereview.chromium.org/2916803004/. So this
change restores the original test files.

BUG=680809
==========

to

==========
[System-Keyboard-Lock] Restore keyboard-lock tests

The related test cases are removed by change
https://chromium-review.googlesource.com/c/499507/ because of a wrong annotation
in W3CImportExpectations file. The change to correct W3CImprotExpectations will
be submitted by change https://codereview.chromium.org/2916803004/. So this
change restores the original test files.

BUG=680809
==========

[qyearsley, 2017-06-02 18:12:19]

LGTM

Above the BUG= line in the description, could you also add NOEXPORT=true? That
way the wpt-exporter definitely won't consider this exportable.

I *think* that without that line, the exporter will look at this commit, and
check whether the patch applies to web-platform-tests HEAD (and it shouldn't
since these files already exist there), so I think it may be OK anyway, but
either way adding NOEXPORT=true will be correct.

[Hzj_jie, 2017-06-02 18:13:29]

Description was changed from

==========
[System-Keyboard-Lock] Restore keyboard-lock tests

The related test cases are removed by change
https://chromium-review.googlesource.com/c/499507/ because of a wrong annotation
in W3CImportExpectations file. The change to correct W3CImprotExpectations will
be submitted by change https://codereview.chromium.org/2916803004/. So this
change restores the original test files.

BUG=680809
==========

to

==========
[System-Keyboard-Lock] Restore keyboard-lock tests

The related test cases are removed by change
https://chromium-review.googlesource.com/c/499507/ because of a wrong annotation
in W3CImportExpectations file. The change to correct W3CImprotExpectations will
be submitted by change https://codereview.chromium.org/2916803004/. So this
change restores the original test files.

BUG=680809
NOEXPORT=true
==========

[Hzj_jie, 2017-06-02 18:13:49]

Description was changed from

==========
[System-Keyboard-Lock] Restore keyboard-lock tests

The related test cases are removed by change
https://chromium-review.googlesource.com/c/499507/ because of a wrong annotation
in W3CImportExpectations file. The change to correct W3CImprotExpectations will
be submitted by change https://codereview.chromium.org/2916803004/. So this
change restores the original test files.

BUG=680809
NOEXPORT=true
==========

to

==========
[System-Keyboard-Lock] Restore keyboard-lock tests

The related test cases are removed by change
https://chromium-review.googlesource.com/c/499507/ because of a wrong annotation
in W3CImportExpectations file. The change to correct W3CImprotExpectations will
be submitted by change https://codereview.chromium.org/2916803004/. So this
change restores the original test files.

NOEXPORT=true
BUG=680809
==========

[Hzj_jie, 2017-06-02 18:14:24]

On 2017/06/02 18:12:19, qyearsley wrote:
> LGTM
> 
> Above the BUG= line in the description, could you also add NOEXPORT=true? That
> way the wpt-exporter definitely won't consider this exportable.
> 
> I *think* that without that line, the exporter will look at this commit, and
> check whether the patch applies to web-platform-tests HEAD (and it shouldn't
> since these files already exist there), so I think it may be OK anyway, but
> either way adding NOEXPORT=true will be correct.

Sorry for lacking knowledge about the exporter, I will follow your suggestion.

[qyearsley, 2017-06-02 18:16:01]

On 2017/06/02 at 18:14:24, zijiehe wrote:
> On 2017/06/02 18:12:19, qyearsley wrote:
> > LGTM
> > 
> > Above the BUG= line in the description, could you also add NOEXPORT=true?
That
> > way the wpt-exporter definitely won't consider this exportable.
> > 
> > I *think* that without that line, the exporter will look at this commit, and
> > check whether the patch applies to web-platform-tests HEAD (and it shouldn't
> > since these files already exist there), so I think it may be OK anyway, but
> > either way adding NOEXPORT=true will be correct.
> 
> Sorry for lacking knowledge about the exporter, I will follow your suggestion.

No problem, LGTM :-)

[commit-bot: I haz the power, 2017-06-02 19:08:29]

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

[commit-bot: I haz the power, 2017-06-02 19:08:30]

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

[Hzj_jie, 2017-06-02 22:04:18]

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

[commit-bot: I haz the power, 2017-06-02 22:05:06]

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-06-02 23:59:25]

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

[commit-bot: I haz the power, 2017-06-02 23:59:26]

Dry run: This issue passed the CQ dry run.

[Hzj_jie, 2017-06-03 00:22:56]

The CQ bit was checked by zijiehe@chromium.org

[Hzj_jie, 2017-06-03 00:22:57]

The patchset sent to the CQ was uploaded after l-g-t-m from
qyearsley@chromium.org
Link to the patchset: https://codereview.chromium.org/2915203003/#ps20001
(title: "Workaround testharness.js test requirement")

[commit-bot: I haz the power, 2017-06-03 00:23:43]

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-06-03 00:28:13]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1496449376499560,
"parent_rev": "1bcfddf2df553b180d0253f26527f94d38710b0e", "commit_rev":
"cc0c8cf835d5656666dbc7266d6f542d6607bc1f"}

[commit-bot: I haz the power, 2017-06-03 00:28:25]

Description was changed from

==========
[System-Keyboard-Lock] Restore keyboard-lock tests

The related test cases are removed by change
https://chromium-review.googlesource.com/c/499507/ because of a wrong annotation
in W3CImportExpectations file. The change to correct W3CImprotExpectations will
be submitted by change https://codereview.chromium.org/2916803004/. So this
change restores the original test files.

NOEXPORT=true
BUG=680809
==========

to

==========
[System-Keyboard-Lock] Restore keyboard-lock tests

The related test cases are removed by change
https://chromium-review.googlesource.com/c/499507/ because of a wrong annotation
in W3CImportExpectations file. The change to correct W3CImprotExpectations will
be submitted by change https://codereview.chromium.org/2916803004/. So this
change restores the original test files.

NOEXPORT=true
BUG=680809

Review-Url: https://codereview.chromium.org/2915203003
Cr-Commit-Position: refs/heads/master@{#476843}
Committed:
https://chromium.googlesource.com/chromium/src/+/cc0c8cf835d5656666dbc7266d6f...
==========

[commit-bot: I haz the power, 2017-06-03 00:28:26]

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