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

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

Issue 1383483007: Add scheme exceptions for isSecureContext (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Nits and fixes Created 5 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/webrtc_from_web_accessible_resource/test.js
diff --git a/chrome/test/data/extensions/api_test/content_scripts/background_page_iframe/background.js b/chrome/test/data/extensions/api_test/webrtc_from_web_accessible_resource/test.js
similarity index 72%
copy from chrome/test/data/extensions/api_test/content_scripts/background_page_iframe/background.js
copy to chrome/test/data/extensions/api_test/webrtc_from_web_accessible_resource/test.js
index 8da0bc66325d6a56011e02db6c9d9ab20a37cfc2..34715d9ff98f8db27a8c9f3f9a7c3e931cd5d1f9 100644
--- a/chrome/test/data/extensions/api_test/content_scripts/background_page_iframe/background.js
+++ b/chrome/test/data/extensions/api_test/webrtc_from_web_accessible_resource/test.js
@@ -4,8 +4,7 @@
chrome.test.getConfig(function(config) {
var iframe = document.createElement('iframe');
robwu 2015/10/08 21:25:18 This variable seems unused, remove it. And you co
jww 2015/10/09 21:39:26 I cleaned this up a bit, but it's now necessary fo
- iframe.src =
+ var url =
'http://a.com:' + config.testServer.port + '/extensions/test_file.html';
- iframe.onload = function() { chrome.test.sendMessage('iframe loaded'); }
- document.body.appendChild(iframe);
+ chrome.tabs.create({ 'url': url })
});

Powered by Google App Engine
This is Rietveld 408576698