CSSPaint: Move layout tests in csspaint/ to http/tests/csspaint/

CSSPaint: Move layout tests in csspaint/ to http/tests/csspaint/

After ES6 modules loading for PaintWorklet (issue 627945) is introduced, layout
tests for PaintWorklet are no longer able to run on non-http environment because
ES6 modules does not support "file://" url (See https://crbug.com/594639#c86).

This CL moves layout tests in csspaint/ to http/tests/csspaint/, and copies
resources/run-after-layout-and-paint.js into http/tests/resources/.

BUG=719857
Review-Url: https://codereview.chromium.org/2868003002
Cr-Commit-Position: refs/heads/master@{#474174}
Committed: https://chromium.googlesource.com/chromium/src/+/a71bdb09d0b4e789a045820895ab3535bffddabf

[nhiroki, 2017-05-09 08:35:02]

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

[commit-bot: I haz the power, 2017-05-09 08:35:38]

Dry run: CQ is trying da patch.

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

[nhiroki, 2017-05-09 08:36:27]

Description was changed from

==========
CSSPaint: Move layout tests in csspaint/ to http/tests/csspaint/

After ES6 modules loading for PaintWorklet (issue 627945) is introduced, layout
tests for PaintWorklet are no longer able to run on non-http environment because
ES6 modules does not support "file://" url (See https://crbug.com/594639#c86).

This CL moves layout tests in csspaint/ to http/tests/csspaint/, and ...

BUG=719857
==========

to

==========
CSSPaint: Move layout tests in csspaint/ to http/tests/csspaint/

After ES6 modules loading for PaintWorklet (issue 627945) is introduced, layout
tests for PaintWorklet are no longer able to run on non-http environment because
ES6 modules does not support "file://" url (See https://crbug.com/594639#c86).

This CL moves layout tests in csspaint/ to http/tests/csspaint/, and copies
resources/run-after-layout-and-paint.js into http/tests/resources/.

BUG=719857
==========

[nhiroki, 2017-05-09 08:45:44]

nhiroki@chromium.org changed reviewers:
+ ikilpatrick@chromium.org, junov@chromium.org

[nhiroki, 2017-05-09 08:45:45]

+ikilpatrick@, +junov@ (owners of modules/csspaint/)

Hi, could you review this? Thanks.

https://codereview.chromium.org/2868003002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/FlagExpectations/enable-slimming-paint-v2
(left):

https://codereview.chromium.org/2868003002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/FlagExpectations/enable-slimming-paint-v2:19:
Bug(none) csspaint/ [ Skip ]
Just removed because "http/ [ Skip ]" covers this.

[commit-bot: I haz the power, 2017-05-09 08:48:01]

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

[commit-bot: I haz the power, 2017-05-09 08:48:02]

Dry run: 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, 2017-05-09 08:49:42]

Presubmit checkes failed with following errors:

** Presubmit ERRORS **
* The following files are passing testharness results without console error
messages, they should be removed:

/b/build/slave/linux/build/src/third_party/WebKit/LayoutTests/http/tests/csspaint/invalidation-background-image-expected.txt

/b/build/slave/linux/build/src/third_party/WebKit/LayoutTests/http/tests/csspaint/invalidation-border-image-expected.txt

/b/build/slave/linux/build/src/third_party/WebKit/LayoutTests/http/tests/csspaint/invalidation-content-image-expected.txt
ERROR: found passing testharness results without console error messages.

I'll tweak these expectation files (maybe in a separate CL?) if this migration
patch looks good to you.

[nhiroki, 2017-05-10 23:33:01]

Ping? :)

[nhiroki, 2017-05-11 00:53:05]

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

[commit-bot: I haz the power, 2017-05-11 00:54:42]

Dry run: CQ is trying da patch.

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

[nhiroki, 2017-05-11 00:57:10]

On 2017/05/09 08:49:42, nhiroki wrote:
> Presubmit checkes failed with following errors:
> 
> ** Presubmit ERRORS **
> * The following files are passing testharness results without console error
> messages, they should be removed:
> 
>
/b/build/slave/linux/build/src/third_party/WebKit/LayoutTests/http/tests/csspaint/invalidation-background-image-expected.txt
> 
>
/b/build/slave/linux/build/src/third_party/WebKit/LayoutTests/http/tests/csspaint/invalidation-border-image-expected.txt
> 
>
/b/build/slave/linux/build/src/third_party/WebKit/LayoutTests/http/tests/csspaint/invalidation-content-image-expected.txt
> ERROR: found passing testharness results without console error messages.
> 
> I'll tweak these expectation files (maybe in a separate CL?) if this migration
> patch looks good to you.

Removed these expectation files (didn't move them to http/tests/) because their
contents should be covered by assertions in
csspaint/resources/test-runner-invalidation-logging.js

[commit-bot: I haz the power, 2017-05-11 03:27:15]

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

[commit-bot: I haz the power, 2017-05-11 03:27:16]

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_...)

[nhiroki, 2017-05-12 00:21:06]

Ping? (this is the last blocker to enabling module loading for PaintWorklet [*])

[*] https://codereview.chromium.org/2826313003/

[nhiroki, 2017-05-22 00:33:48]

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

[commit-bot: I haz the power, 2017-05-22 00:33:56]

Dry run: CQ is trying da patch.

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

[nhiroki, 2017-05-22 00:36:18]

ikilpatrick@, junov@: I'm now back from OOO and resuming this work. Could you
review this?

[commit-bot: I haz the power, 2017-05-22 01:46:12]

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

[commit-bot: I haz the power, 2017-05-22 01:46:14]

Dry run: This issue passed the CQ dry run.

[nhiroki, 2017-05-23 10:53:54]

nhiroki@chromium.org changed reviewers:
+ style-dev@chromium.org

[nhiroki, 2017-05-23 10:53:55]

+style-dev@, anyone can review this? Thank you.

[ikilpatrick, 2017-05-24 03:39:42]

lgtm

[nhiroki, 2017-05-24 04:00:20]

Thank you!

[nhiroki, 2017-05-24 04:00:28]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2017-05-24 04:01:39]

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-05-24 05:37:35]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1495598428701700,
"parent_rev": "b303c28a8002db42825e3b12f3fd956f3c32ecab", "commit_rev":
"a71bdb09d0b4e789a045820895ab3535bffddabf"}

[commit-bot: I haz the power, 2017-05-24 05:37:49]

Description was changed from

==========
CSSPaint: Move layout tests in csspaint/ to http/tests/csspaint/

After ES6 modules loading for PaintWorklet (issue 627945) is introduced, layout
tests for PaintWorklet are no longer able to run on non-http environment because
ES6 modules does not support "file://" url (See https://crbug.com/594639#c86).

This CL moves layout tests in csspaint/ to http/tests/csspaint/, and copies
resources/run-after-layout-and-paint.js into http/tests/resources/.

BUG=719857
==========

to

==========
CSSPaint: Move layout tests in csspaint/ to http/tests/csspaint/

After ES6 modules loading for PaintWorklet (issue 627945) is introduced, layout
tests for PaintWorklet are no longer able to run on non-http environment because
ES6 modules does not support "file://" url (See https://crbug.com/594639#c86).

This CL moves layout tests in csspaint/ to http/tests/csspaint/, and copies
resources/run-after-layout-and-paint.js into http/tests/resources/.

BUG=719857

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

[commit-bot: I haz the power, 2017-05-24 05:37:51]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/a71bdb09d0b4e789a045820895ab...