diff options
author | zork@chromium.org <zork@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-12 05:16:20 +0000 |
---|---|---|
committer | zork@chromium.org <zork@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-12 05:16:20 +0000 |
commit | a6413746c7dc93abcac5573dad08aa6a07826e6e (patch) | |
tree | a0d18464bb79000098b1b5b40a9b5ee3e52feeae /chrome/browser/notifications/desktop_notifications_unittest.cc | |
parent | 8e85884ba4efff3c2189581d9819dfdfcfb35aa6 (diff) | |
download | chromium_src-a6413746c7dc93abcac5573dad08aa6a07826e6e.zip chromium_src-a6413746c7dc93abcac5573dad08aa6a07826e6e.tar.gz chromium_src-a6413746c7dc93abcac5573dad08aa6a07826e6e.tar.bz2 |
Add notifications to allow desktop notification permissions to be synced.
Fix a bug where sites can be added to allow and deny lists multiple times.
BUG=none
TEST=Open two browsers with separate profiles. Request permission for notifications on a site on one. Check that the permissions are the same on the other browser.
Review URL: http://codereview.chromium.org/1578023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44234 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/notifications/desktop_notifications_unittest.cc')
-rw-r--r-- | chrome/browser/notifications/desktop_notifications_unittest.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/notifications/desktop_notifications_unittest.cc b/chrome/browser/notifications/desktop_notifications_unittest.cc index 637d9a1..be7a6f6 100644 --- a/chrome/browser/notifications/desktop_notifications_unittest.cc +++ b/chrome/browser/notifications/desktop_notifications_unittest.cc @@ -82,9 +82,9 @@ void DesktopNotificationsTest::SetUp() { } void DesktopNotificationsTest::TearDown() { + service_.reset(NULL); profile_.reset(NULL); ui_manager_.reset(NULL); - service_.reset(NULL); } TEST_F(DesktopNotificationsTest, TestShow) { |