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

Unified Diff: chrome/test/data/extensions/api_test/extension_options/embed_other/embedder/test.js

Issue 429763002: Implement createfailed event for <extensionoptions> (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove console.logs and diff from CL 412093002 Created 6 years, 5 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/extension_options/embed_other/embedder/test.js
diff --git a/chrome/test/data/extensions/api_test/extension_options/embed_other/embedder/test.js b/chrome/test/data/extensions/api_test/extension_options/embed_other/embedder/test.js
new file mode 100644
index 0000000000000000000000000000000000000000..b8169e1a48e420e2f9be907e29c9fa3728dc165e
--- /dev/null
+++ b/chrome/test/data/extensions/api_test/extension_options/embed_other/embedder/test.js
@@ -0,0 +1,30 @@
+// Copyright 2014 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.
+
+chrome.test.runTests([
+ function cannotEmbedOtherExtensionsOptions() {
+ chrome.storage.local.get('embeddedId', function(items) {
not at google - send to devlin 2014/08/04 19:12:03 wrap function(items) in a chrome.test.callbackPass
ericzeng 2014/08/04 21:15:34 Done.
+ var done = chrome.test.callbackAdded();
+ var extensionoptions = document.createElement('extensionoptions');
+ extensionoptions.addEventListener('createfailed', function() {
+ try {
+ chrome.runtime.sendMessage(items['embeddedId'],
+ 'checking for options page',
+ function(response) {
+ chrome.test.assertFalse(response.hasOptionsPage);
+ done();
+ });
+ } finally {
+ document.body.removeChild(extensionoptions);
+ }
+ });
+ extensionoptions.addEventListener('load', function (){
+ document.body.removeChild(extensionoptions);
+ chrome.test.fail();
+ })
+ extensionoptions.setAttribute('extension', items.embeddedId);
+ document.body.appendChild(extensionoptions);
+ });
+ }
+]);

Powered by Google App Engine
This is Rietveld 408576698