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

Unified Diff: chrome/test/data/extensions/api_test/notifications/has_permission_prefs/background.html

Issue 8763008: Move yet another block of 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/notifications/has_permission_prefs/background.html
===================================================================
--- chrome/test/data/extensions/api_test/notifications/has_permission_prefs/background.html (revision 112170)
+++ chrome/test/data/extensions/api_test/notifications/has_permission_prefs/background.html (working copy)
@@ -1,37 +1,6 @@
-<script>
-var notification = null;
-
-// Shows the notification window using the specified URL.
-// Control continues at onNotificationDone().
-function showNotification(url) {
- notification = window.webkitNotifications.createHTMLNotification(url);
- notification.onerror = function() {
- chrome.test.fail("Failed to show notification.");
- };
- notification.show();
-}
-
-// Called by the notification when it is done with its tests.
-function onNotificationDone(notificationWindow) {
- var views = chrome.extension.getViews();
- chrome.test.assertEq(2, views.length);
- notificationWindow.onunload = function() {
- chrome.test.succeed();
- }
- notification.cancel();
-}
-
-chrome.test.runTests([
- function hasPermission() {
- chrome.test.assertEq(0, // allowed
- webkitNotifications.checkPermission());
- chrome.test.succeed();
- },
- function absoluteURL() {
- showNotification(chrome.extension.getURL("notification.html"));
- },
- function relativeURL() {
- showNotification("notification.html");
- }
-]);
-</script>
+<!--
+ * 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.
+-->
+<script src="background.js"></script>

Powered by Google App Engine
This is Rietveld 408576698