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

Unified Diff: chrome/test/data/extensions/api_test/content_scripts/permissions/background.js

Issue 10073011: Fix the optional permissions API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 8 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/content_scripts/permissions/background.js
diff --git a/chrome/test/data/extensions/api_test/content_scripts/permissions/background.js b/chrome/test/data/extensions/api_test/content_scripts/permissions/background.js
new file mode 100644
index 0000000000000000000000000000000000000000..20172abfaa0f975c9a5ba25cee40d0f7921ae341
--- /dev/null
+++ b/chrome/test/data/extensions/api_test/content_scripts/permissions/background.js
@@ -0,0 +1,112 @@
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+var assertEq = chrome.test.assertEq;
+var assertTrue = chrome.test.assertTrue;
+var pass = chrome.test.callbackPass;
+var callbackFail = chrome.test.callbackFail;
+var listenForever = chrome.test.listenForever;
+
+var testTabId;
+var port;
+
+function testUrl(domain) {
+ return 'http://' + domain + ':' + port +
+ '/files/extensions/test_file.html';
+}
+
+function error(domain) {
+ return 'Cannot access contents of url "' + testUrl(domain) + '".' +
+ ' Extension manifest must request permission to access this host.';
+}
+
+// Creates a new tab, navigated to the specified |domain|.
+function createTestTab(domain, callback) {
+ var createdTabId = -1;
+ var done = listenForever(
+ chrome.tabs.onUpdated,
+ function(tabId, changeInfo, tab) {
+ if (tabId == createdTabId && changeInfo.status != 'loading') {
+ callback(tab);
+ done();
+ }
+ });
+
+ chrome.tabs.create({url: testUrl(domain)}, pass(function(tab) {
+ createdTabId = tab.id;
+ }));
+}
+
+chrome.test.getConfig(function(config) {
+ port = config.testServer.port;
+ chrome.test.runTests([
+
+ // Before enabling the optional host permission, we shouldn't be able to
+ // inject content scripts.
+ function noAccess() {
+ createTestTab('a.com', pass(function(tab) {
+ testTabId = tab.id;
+ chrome.tabs.executeScript(
+ tab.id, {code: 'document.title = "success"'},
+ callbackFail(error('a.com')));
+ }));
+ },
+
+ // Add the host permission and see if we can inject a content script into
+ // existing and new tabs.
+ function addPermission() {
+ chrome.permissions.request(
+ {origins: ["http://*/*"]},
+ pass(function(granted) {
+ assertTrue(granted);
+
+ // Try accessing the existing tab.
+ chrome.tabs.executeScript(
+ testTabId, {code: 'document.title = "success"'},
+ pass(function() {
+ chrome.tabs.get(testTabId, pass(function(tab) {
+ assertEq('success', tab.title);
+ }));
+ }));
+
+ // Make sure we can inject a script into a new tab with that host.
+ createTestTab('a.com', pass(function(tab) {
+ chrome.tabs.executeScript(
+ tab.id, {code: 'document.title = "success"'},
+ pass(function() {
+ chrome.tabs.get(tab.id, pass(function(tab) {
+ assertEq('success', tab.title);
+ }));
+ }));
+ }));
+ }));
+ },
+
+ // Try the host again, except outside of the permissions.request callback.
+ function sameHost() {
+ createTestTab('a.com', pass(function(tab) {
+ chrome.tabs.executeScript(
+ tab.id, {code: 'document.title = "success"'},
+ pass(function() {
+ chrome.tabs.get(tab.id, pass(function(tab) {
+ assertEq('success', tab.title);
+ }));
+ }));
+ }));
+ },
+
+ // Try injecting the script into a new tab with a new host.
+ function newHost() {
+ createTestTab('b.com', pass(function(tab) {
+ chrome.tabs.executeScript(
+ tab.id, {code: 'document.title = "success"'},
+ pass(function() {
+ chrome.tabs.get(tab.id, pass(function(tab) {
+ assertEq('success', tab.title);
+ }));
+ }));
+ }));
+ }
+ ]);
+});

Powered by Google App Engine
This is Rietveld 408576698