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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py

Issue 2302653002: Update list of resource files to copy from web-platform-tests. (Closed)
Patch Set: Updated comments Created 4 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py
index 1f0711860bac7d12c3bd19c5c7d4d42e66ce53b9..cb21bcf52e984c0f53872901f3fae5322e2e7711 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py
@@ -107,9 +107,13 @@ class DepsUpdater(object):
There are some resources in our repository that we use instead of the
upstream versions.
+
+ If this list is changed, we should consider also consider changing the
+ lists of files expected to be identical in LayoutTests/PRESUBMIT.py.
"""
resources_to_copy = [
- ('testharnessreport.js', 'resources'),
qyearsley 2016/08/31 23:41:03 Looking at this here, I'm pretty sure that we *don
jsbell 2016/09/01 16:51:05 We *always* want to use "our" copy of testharnessr
+ ('idlharness.js', 'resources'),
+ ('testharness.js', 'resources'),
('WebIDLParser.js', 'resources'),
('vendor-prefix.js', 'common'),
]
« third_party/WebKit/LayoutTests/PRESUBMIT.py ('K') | « third_party/WebKit/LayoutTests/PRESUBMIT.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698