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

Unified Diff: third_party/WebKit/LayoutTests/external/wpt/resource-timing/test_resource_timing.js

Issue 2962113002: Updates to Server-Timing in accordance with with spec changes (Closed)
Patch Set: fix web-platform-tests Created 3 years, 5 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/external/wpt/server-timing/test_server_timing.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/external/wpt/resource-timing/test_resource_timing.js
diff --git a/third_party/WebKit/LayoutTests/external/wpt/resource-timing/test_resource_timing.js b/third_party/WebKit/LayoutTests/external/wpt/resource-timing/test_resource_timing.js
index d632a0d5f579d0f56dad6e20db9e87b9548c9083..51a3d2c1fc57da145a40e55efc71446f0466a5b9 100644
--- a/third_party/WebKit/LayoutTests/external/wpt/resource-timing/test_resource_timing.js
+++ b/third_party/WebKit/LayoutTests/external/wpt/resource-timing/test_resource_timing.js
@@ -162,7 +162,11 @@ function resource_load(expected)
assert_equals(entries_by_name_type.length, 1, "should have a single entry for each resource (with type)");
assert_not_equals(entries_by_name, entries_by_name_type, "values should be copies");
for (p in entries_by_name[0]) {
- assert_equals(entries_by_name[0][p], entries_by_name_type[0][p], "Property " + p + " should match");
+ var assertMethod = assert_equals
+ if (Array.isArray(entries_by_name[0][p]) && Array.isArray(entries_by_name_type[0][p])) {
+ assertMethod = assert_array_equals
+ }
+ assertMethod(entries_by_name[0][p], entries_by_name_type[0][p], "Property " + p + " should match");
}
this.done();
});
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/external/wpt/server-timing/test_server_timing.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698