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

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

Issue 8758008: Move another block of extension tests to manifest_version 2. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 1 month 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/events/background.js
===================================================================
--- chrome/test/data/extensions/api_test/events/background.js (revision 0)
+++ chrome/test/data/extensions/api_test/events/background.js (revision 0)
@@ -0,0 +1,55 @@
+// Copyright (c) 2011 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([
+ // Tests that attaching and detaching to an event for which we don't have
+ // permission acts as expected (e.g. we don't DCHECK!).
+ function attachAndDetachNoPermisssions() {
+ function dummy() {};
+ try {
+ chrome.tabs.onUpdated.addListener(dummy);
+ chrome.test.fail();
+ } catch (e) {
+ chrome.test.assertTrue(
+ e.message.search("You do not have permission") >= 0,
+ e.message);
+ }
+ chrome.test.assertFalse(chrome.tabs.onUpdated.hasListeners());
+ chrome.tabs.onUpdated.removeListener(dummy); // browser should not DCHECK
+ chrome.test.succeed();
+ },
+
+ // Tests that attaching a named event twice will fail.
+ function doubleAttach() {
+ function dummy() {};
+ var onClicked = new chrome.Event("browserAction.onClicked");
+ var onClicked2 = new chrome.Event("browserAction.onClicked");
+ onClicked.addListener(dummy);
+ chrome.test.assertTrue(onClicked.hasListeners());
+ try {
+ onClicked2.addListener(dummy);
+ chrome.test.fail();
+ } catch (e) {
+ chrome.test.assertTrue(
+ e.message.search("already attached") >= 0,
+ e.message);
+ }
+ chrome.test.assertFalse(onClicked2.hasListeners());
+ onClicked2.removeListener(dummy);
+
+ onClicked.removeListener(dummy);
+ chrome.test.assertFalse(onClicked.hasListeners());
+ chrome.test.succeed();
+ },
+
+ // Tests that 2 pages attaching to the same event does not trigger a DCHECK.
+ function twoPageAttach() {
+ // Test harness should already have opened tab.html, which registers this
+ // listener.
+ chrome.browserAction.onClicked.addListener(function() {});
+
+ // Test continues in twoPageAttach.html.
+ window.open("twoPageAttach.html");
+ },
+]);
« no previous file with comments | « chrome/test/data/extensions/api_test/events/background.html ('k') | chrome/test/data/extensions/api_test/events/manifest.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698