diff options
| author | jianli@chromium.org <jianli@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-07-30 22:24:31 +0000 |
|---|---|---|
| committer | jianli@chromium.org <jianli@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-07-30 22:24:31 +0000 |
| commit | 3076173cb3b3755c762603cb61e5231a565b1c69 (patch) | |
| tree | c61b1646ffd28b547ab921faf0799289a26a2e7d /chrome/browser/extensions/notifications_apitest.cc | |
| parent | b8cf75134c0789789d4b178d07db0167dd795a14 (diff) | |
| download | chromium_src-3076173cb3b3755c762603cb61e5231a565b1c69.zip chromium_src-3076173cb3b3755c762603cb61e5231a565b1c69.tar.gz chromium_src-3076173cb3b3755c762603cb61e5231a565b1c69.tar.bz2 | |
Only allows extensions to create HTML notifications.
createHTMLNotification has been removed from the W3C Notifications spec. We do not want it to be exposed for web pages. However, since quite a few extensions are still using it, we would still keep it to be exposed for extensions.
BUG=98061
TEST=existing tests
Review URL: https://chromiumcodereview.appspot.com/10808113
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149054 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/notifications_apitest.cc')
| -rw-r--r-- | chrome/browser/extensions/notifications_apitest.cc | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/chrome/browser/extensions/notifications_apitest.cc b/chrome/browser/extensions/notifications_apitest.cc index d3640aa..c63f6e5 100644 --- a/chrome/browser/extensions/notifications_apitest.cc +++ b/chrome/browser/extensions/notifications_apitest.cc @@ -1,4 +1,4 @@ -// Copyright (c) 2011 The Chromium Authors. All rights reserved. +// Copyright (c) 2012 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. @@ -9,7 +9,7 @@ #include "chrome/browser/profiles/profile.h" #include "chrome/browser/ui/browser.h" -IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_NotificationsNoPermission) { +IN_PROC_BROWSER_TEST_F(ExtensionApiTest, NotificationsNoPermission) { #if defined(OS_LINUX) && defined(TOOLKIT_VIEWS) // Notifications not supported on linux/views yet. #else @@ -17,8 +17,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_NotificationsNoPermission) { #endif } -IN_PROC_BROWSER_TEST_F(ExtensionApiTest, - DISABLED_NotificationsHasPermissionManifest) { +IN_PROC_BROWSER_TEST_F(ExtensionApiTest, NotificationsHasPermissionManifest) { #if defined(OS_LINUX) && defined(TOOLKIT_VIEWS) // Notifications not supported on linux/views yet. #else @@ -27,8 +26,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, #endif } -// http://crbug.com/98061 -IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_NotificationsHasPermission) { +IN_PROC_BROWSER_TEST_F(ExtensionApiTest, NotificationsHasPermission) { #if defined(OS_LINUX) && defined(TOOLKIT_VIEWS) // Notifications not supported on linux/views yet. #else |