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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/inspector/network-test.js

Issue 2626553002: [Devtools][Code health] Changed NetworkRequest.url to proper getter/setter (Closed)
Patch Set: changes 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/inspector/network-test.js
diff --git a/third_party/WebKit/LayoutTests/http/tests/inspector/network-test.js b/third_party/WebKit/LayoutTests/http/tests/inspector/network-test.js
index 92ccdd6fa936ce7de68cf1bd036eb4796230e336..f19b8a7a554a06978a79cc2f92ce14a1a1fdce0e 100644
--- a/third_party/WebKit/LayoutTests/http/tests/inspector/network-test.js
+++ b/third_party/WebKit/LayoutTests/http/tests/inspector/network-test.js
@@ -69,17 +69,17 @@ InspectorTest.networkRequests = function()
InspectorTest.dumpNetworkRequests = function()
{
var requests = InspectorTest.networkRequests();
- requests.sort(function(a, b) {return a.url.localeCompare(b.url);});
+ requests.sort(function(a, b) {return a.url().localeCompare(b.url());});
InspectorTest.addResult("resources count = " + requests.length);
for (i = 0; i < requests.length; i++)
- InspectorTest.addResult(requests[i].url);
+ InspectorTest.addResult(requests[i].url());
}
// |url| must be a regular expression to match request URLs.
InspectorTest.findRequestsByURLPattern = function(urlPattern)
{
return InspectorTest.networkRequests().filter(function(value) {
- return urlPattern.test(value.url)
+ return urlPattern.test(value.url());
});
}

Powered by Google App Engine
This is Rietveld 408576698