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

Side by Side Diff: chrome/test/data/extensions/api_test/permissions/optional/background.html

Issue 7695003: Fix issue with permissions extension API event listeners. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 4 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/browser/extensions/extension_permissions_api.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <script> 1 <script>
2 2
3 var assertEq = chrome.test.assertEq; 3 var assertEq = chrome.test.assertEq;
4 var assertFalse = chrome.test.assertFalse; 4 var assertFalse = chrome.test.assertFalse;
5 var assertTrue = chrome.test.assertTrue; 5 var assertTrue = chrome.test.assertTrue;
6 var fail = chrome.test.callbackFail; 6 var fail = chrome.test.callbackFail;
7 var pass = chrome.test.callbackPass; 7 var pass = chrome.test.callbackPass;
8 var listenOnce = chrome.test.listenOnce; 8 var listenOnce = chrome.test.listenOnce;
9 9
10 var NOT_OPTIONAL_ERROR = 10 var NOT_OPTIONAL_ERROR =
(...skipping 258 matching lines...) Expand 10 before | Expand all | Expand 10 after
269 pass(function(removed) { 269 pass(function(removed) {
270 assertTrue(removed); 270 assertTrue(removed);
271 chrome.experimental.permissions.getAll(pass(function(permissions) { 271 chrome.experimental.permissions.getAll(pass(function(permissions) {
272 assertTrue(checkPermSetsEq(initialPermissions, permissions)); 272 assertTrue(checkPermSetsEq(initialPermissions, permissions));
273 })); 273 }));
274 chrome.experimental.permissions.contains( 274 chrome.experimental.permissions.contains(
275 {origins:['http://*.c.com/*']}, 275 {origins:['http://*.c.com/*']},
276 pass(function(result) { assertFalse(result); })); 276 pass(function(result) { assertFalse(result); }));
277 doReq('http://c.com', pass(function(result) { assertFalse(result); })); 277 doReq('http://c.com', pass(function(result) { assertFalse(result); }));
278 })); 278 }));
279 },
280
281 // Tests that the changed permissions have taken effect from inside the
282 // onAdded and onRemoved event listeners.
283 function eventListenerPermissions() {
284 listenOnce(chrome.experimental.permissions.onAdded,
285 function(permissions) {
286 chrome.windows.getAll({populate: true}, pass(function() {
287 assertTrue(true);
288 }));
289 });
290 listenOnce(chrome.experimental.permissions.onRemoved,
291 function(permissions) {
292 try {
293 chrome.windows.getAll({populate: true}, function() {
294 chrome.test.fail("Should not have tabs API permission.");
295 });
296 } catch (e) {
297 assertTrue(e.message.indexOf(NO_TABS_PERMISSION) == 0);
298 }
299 });
300
301 chrome.experimental.permissions.request(
302 {permissions: ['tabs']}, pass(function(granted) {
303 assertTrue(granted);
304 chrome.experimental.permissions.remove(
305 {permissions: ['tabs']}, pass(function() {
306 assertTrue(true);
307 }));
308 }));
279 } 309 }
280 310
281 ]); 311 ]);
282 }); 312 });
283 </script> 313 </script>
OLDNEW
« no previous file with comments | « chrome/browser/extensions/extension_permissions_api.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698