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

Unified Diff: chrome/test/data/extensions/api_test/content_scripts/extension_iframe/iframe.js

Issue 9653022: Revert 125811 - Convert app_bindings.js to the schema_generated_bindings.js infrastructure. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 9 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/extension_iframe/iframe.js
===================================================================
--- chrome/test/data/extensions/api_test/content_scripts/extension_iframe/iframe.js (revision 125812)
+++ chrome/test/data/extensions/api_test/content_scripts/extension_iframe/iframe.js (working copy)
@@ -28,15 +28,11 @@
success = false;
}
-// Ditto chrome.tabs, though it's special because it's a dependency of the
-// partially unprivileged chrome.extension.
-if (chrome.tabs) {
- console.log('Error: chrome.tabs exists, it shouldn\'t.');
- success = false;
-}
-
-// Parts of chrome.extension are unavailable.
+// Parts of chrome.extension and chrome.tabs (which get included because it's
+// a dependency of chrome.extension) are unavailable.
if (!runsWithException(function() { return chrome.extension.getViews; }))
success = false;
+if (!runsWithException(function() { return chrome.tabs.create; }))
+ success = false;
chrome.extension.sendRequest({success: success});
« no previous file with comments | « chrome/renderer/resources/extensions/setup_bindings.js ('k') | chrome/test/data/extensions/api_test/stubs/content_script.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698