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

Unified Diff: chrome/test/data/extensions/api_test/cross_origin_xhr/content_script/test.js

Issue 16268017: GTTF: convert some tests in chrome to use EmbeddedTestServer patch nr 1 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 6 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/cross_origin_xhr/content_script/test.js
diff --git a/chrome/test/data/extensions/api_test/cross_origin_xhr/content_script/test.js b/chrome/test/data/extensions/api_test/cross_origin_xhr/content_script/test.js
index b3f12ed8cc686dafae7b32ff02bfaad5d664cddd..fa5893600e88c4e790267fa8bdaf845f228d30ca 100644
--- a/chrome/test/data/extensions/api_test/cross_origin_xhr/content_script/test.js
+++ b/chrome/test/data/extensions/api_test/cross_origin_xhr/content_script/test.js
@@ -12,7 +12,7 @@ chrome.test.getConfig(function(config) {
}
function doReq(domain, expectSuccess) {
- var url = rewriteURL(domain + ':PORT/files/extensions/test_file.txt');
+ var url = rewriteURL(domain + ':PORT/extensions/test_file.txt');
chrome.tabs.sendRequest(testTabId, url, function(response) {
if (expectSuccess) {
@@ -28,7 +28,7 @@ chrome.test.getConfig(function(config) {
}
chrome.tabs.create({
- url: rewriteURL('http://localhost:PORT/files/extensions/test_file.html')},
+ url: rewriteURL('http://localhost:PORT/extensions/test_file.html')},
function(tab) {
testTabId = tab.id;
});
@@ -52,9 +52,10 @@ chrome.test.getConfig(function(config) {
function disallowedSubdomain() {
doReq('http://foob.com', false);
},
- function disallowedSSL() {
- doReq('https://a.com', false);
- },
+ // TODO(asargent): Explicitly create SSL test server and enable the test.
+ // function disallowedSSL() {
+ // doReq('https://a.com', false);
+ // },
function targetPageAlwaysAllowed() {
// Even though localhost does not show up in the host permissions, we
// can still make requests to it since it's the page that the content

Powered by Google App Engine
This is Rietveld 408576698