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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/serviceworker/clients-get-client-types.html

Issue 2662953002: Fix two tests in http/tests/serviceworker/ to handle the correct behavior of undefined in arrays, a… (Closed)
Patch Set: whoops, fix TestExpectations file again Created 3 years, 11 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/LayoutTests/http/tests/serviceworker/clients-get-client-types.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/serviceworker/clients-get-client-types.html b/third_party/WebKit/LayoutTests/http/tests/serviceworker/clients-get-client-types.html
index 75f54ebe194f344b5c8d9d396249810e862c0262..c0d0b20b3df455b3f4eb58764a051616a592d5cf 100644
--- a/third_party/WebKit/LayoutTests/http/tests/serviceworker/clients-get-client-types.html
+++ b/third_party/WebKit/LayoutTests/http/tests/serviceworker/clients-get-client-types.html
@@ -64,6 +64,6 @@ function wait_for_clientId() {
var expected = [
/* visibilityState, focused, url, frameType */
['visible', true, normalizeURL(scope) + '-frame.html', 'nested'],
- [,,normalizeURL(scope) + '-shared-worker.js', 'none']
+ [undefined, undefined, normalizeURL(scope) + '-shared-worker.js', 'none']
];
</script>

Powered by Google App Engine
This is Rietveld 408576698