Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(133)

Side by Side Diff: Tools/Scripts/webkitpy/thirdparty/wpt/README.chromium

Issue 1154373005: Introduce WPTServe for running W3C Blink Layout tests (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Add additional README for maintenance. Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 W3C Web Platform Tests in Blink Layout Tests
2
3 Design Doc: https://goo.gl/iXUaZd
4
5 This directory contains checked out and reduced code from web-platform-tests
jsbell 2015/06/12 17:13:17 We should get jgraham (W3C) to review this doc at
burnik 2015/06/12 17:23:55 Agreed. That's why I explicitly state the HEADs at
6 (https://github.com/w3c/web-platform-tests/) required to run WPT tests as part
7 of Blink Layout Tests.
8
9 For licensing, see README.chromium in parent directory
10 (Tools/Scripts/webkitpy/thirdparty/README.chromium).
11
12 Other files present in this directories are as follows:
13
14 wpt.config.json
15 ===============
16 The configuration file used when running WPTServe. Note that this file loads
17 after wpt/config.default.json and this configuration gets merged onto it. When
18 changing the ports (HTTP/S, WS/S), make sure to update the python code too.
Dirk Pranke 2015/06/12 22:25:02 This is actually a file you wrote, right? Can we m
burnik 2015/06/15 10:35:10 I would actually like to keep it in thirdparty for
19
20 checkout.sh
21 ===========
22 Run this script to delete the third party wpt directory (thirdparty/wpt/wpt)
23 and perform a fresh checkout of the web-platform-tests code. When rolling in new
jsbell 2015/06/12 17:13:17 Can you rewrite this more imperatively, i.e. keep
burnik 2015/06/15 10:35:10 Done.
24 versions of WPT support, use WPTHeads to adjust the HEAD positions before
25 running ./checkout.sh. The script will also delete everything not listed in
26 WPTWhiteList from the checked out directory.
Dirk Pranke 2015/06/12 22:25:02 From what I can tell, we only actually need the
burnik 2015/06/15 10:35:10 Actually, that's not all we need. We need the wpt
27
28 WPTHeads
29 ========
30 List of git commit-ish for the WPT repositories. File format is as follows:
31 First line: HEAD position for web-platform-tests.
32 Rest of lines: parent directory, submodule name, HEAD position for submodule.
33 The submodule checkout is performed in order when running ./checkout.sh.
34
35 WPTWhiteList
36 ============
37 The explicit list of files being kept, everything else not on this list is
38 deleted when running ./checkout.sh. Use this file to control what gets checked
39 in and try to keep the list as small as possible (use what you need).
40
41 **
42
43 Running web-platform tests with enabled WPTServe on a local machine
44
45 Starting run-webkit-tests with the --enable-wptserve flag will use the WPT
46 serve for tests which live in LayoutTests/imported/web-platform-tests.
47
48 WPTServe starts HTTP/S and WS/S servers as separate processes.
49
50 The content_shell used to run the tests will receive the URL of each test
51 (instead of a filename). The document root http://web-platform.test/ maps to
52 LayoutTests/imported/web-platform-tests. HTTPS tests are enabled by default.
53
54 Example run:
55
56 ./Tools/Scripts/run-webkit-tests \
57 --debug \
58 --enable-wptserve \
59 --no-new-test-results \
60 imported/web-platform-tests
61
62 For setting up SSL, refer to Tools/Scripts/webkitpy/thirdparty/wpt/wpt/_certs
63 (which gets generated on the first run) and Web Platform Tests Documentation.
jsbell 2015/06/12 17:13:17 Include URL of the WPT docs?
burnik 2015/06/12 17:23:54 When I find them, I'll make sure to add. So far, n
burnik 2015/06/15 10:35:10 I've rewritten this in the "Configuration" section
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698