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

Unified Diff: chrome/test/data/extensions/api_test/websocket/test.js

Issue 11087027: WebSocket test server migration on ExtensionApiTest.WebSocket (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix copyright Created 8 years, 2 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: chrome/test/data/extensions/api_test/websocket/test.js
diff --git a/chrome/test/data/extensions/api_test/websocket/test.js b/chrome/test/data/extensions/api_test/websocket/test.js
index 8041593cb39cadfdf601c3af45b2eec7a0ec2813..2e2adca09901a93bd2d08c115ddb3723e2d4a278 100644
--- a/chrome/test/data/extensions/api_test/websocket/test.js
+++ b/chrome/test/data/extensions/api_test/websocket/test.js
@@ -3,8 +3,7 @@
// found in the LICENSE file.
function echoTest(port) {
- var url =
- "ws://localhost:" + port + "/websocket/tests/hybi/workers/resources/echo";
+ var url = "ws://localhost:" + port + "/echo";
Ryan Sleevi 2012/10/09 19:39:07 nit: For some tests, we also parameterize the host
Takashi Toyoshima 2012/10/10 01:34:48 I see. Thank you for advice.
var ws = new WebSocket(url);
var MESSAGE_A = "message a";
var MESSAGE_B = "message b";
@@ -34,8 +33,11 @@ function echoTest(port) {
};
}
-chrome.test.runTests([
- chrome.test.getConfig(function(config) {
- echoTest(config.testWebSocketPort);
- })
-]);
+chrome.test.getConfig(function(config) {
+ chrome.test.runTests([
+ function runEchoTest() {
+ echoTest(config.testWebSocketPort);
+ }
+ ]);
+});
+

Powered by Google App Engine
This is Rietveld 408576698