summaryrefslogtreecommitdiffstats
path: root/chrome/browser/extensions/app_notification_manager_unittest.cc
Commit message (Collapse)AuthorAgeFilesLines
* Move scoped_temp_dir from base to base/filesbrettw@chromium.org2012-11-161-1/+1
| | | | | | | | | | Also add to base namespace. BUG= Review URL: https://codereview.chromium.org/11359217 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168281 0039d316-1c4b-4281-b951-d872f2087c98
* Moved the AppNotification system into extensions namespacerdevlin.cronin@chromium.org2012-08-011-1/+4
| | | | | | | | | | | | Moved AppNotification, AppNotificationManager, AppNotificationStorage, AppNotifyChannelSetup, and AppNotifyChannelUI into extensions namespace. BUG=139148, 117261 Review URL: https://chromiumcodereview.appspot.com/10830036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149444 0039d316-1c4b-4281-b951-d872f2087c98
* CPM Extension Uninstall Watchingrdevlin.cronin@chromium.org2012-07-171-3/+9
| | | | | | | | | | | | | Add in watching for extension uninstall events. This also changes NOTIFICATION_EXTENSION_UNINSTALL to have the Extension as details. BUG=130212 TEST=Included browsertest; existing tests to test for regression on the notification change. Review URL: https://chromiumcodereview.appspot.com/10694056 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147025 0039d316-1c4b-4281-b951-d872f2087c98
* Re-enable AppNotificationManagerTest.ExtensionUninstall under ASan.glider@chromium.org2012-07-131-8/+2
| | | | | | | | BUG=100156 TBR=asargent Review URL: https://chromiumcodereview.appspot.com/10704200 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146552 0039d316-1c4b-4281-b951-d872f2087c98
* Move test_browser_thread.h from content\test to content\public\test. This ↵jam@chromium.org2012-06-041-1/+1
| | | | | | | | | | way we can enforce that internal content headers don't leak to embedders. BUG=98716 TBR=phajdan.jr Review URL: https://chromiumcodereview.appspot.com/10500016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140254 0039d316-1c4b-4281-b951-d872f2087c98
* Removing MessageLoop::QuitTask() from chrome/dcheng@chromium.org2011-12-101-1/+1
| | | | | | | | | | | | base/task.h is going away and being replaced by base/bind.h and base/callback.h. This CL was automatically generated by sed. BUG=none TEST=none Review URL: http://codereview.chromium.org/8873032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113896 0039d316-1c4b-4281-b951-d872f2087c98
* Switch to IWYU for files that previously were getting a bunch ofjoi@chromium.org2011-12-011-0/+1
| | | | | | | | | | | header files via io_thread.h or browser_thread.h TBR=owners (previously approved on codereview 8477004) BUG=98716 Review URL: http://codereview.chromium.org/8746023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112455 0039d316-1c4b-4281-b951-d872f2087c98
* Move BrowserThread to content namespace.joi@chromium.org2011-11-021-0/+2
| | | | | | | | | TBR=owners BUG=98716 Review URL: http://codereview.chromium.org/8437002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108270 0039d316-1c4b-4281-b951-d872f2087c98
* Split BrowserThread into public API and private implementation, step 1.joi@chromium.org2011-10-281-4/+5
| | | | | | | | | | | | | | | | | | | | | Only content/ now has the ability to create BrowserThread objects, with the exception that tests can create the content::TestBrowserThread subclass, and (temporarily) code in chrome/ can create the DeprecatedBrowserThread subclass. A follow-up change will make content/ take care of its own thread creation, remove DeprecatedBrowserThread, and move all state and non-trivial constructors from BrowserThread down to BrowserThreadImpl. Also moved BrowserProcessSubThread into content/ namespace. As part of follow-up cleanup, chrome/ will stop using this class. BUG=98716 TEST=existing Review URL: http://codereview.chromium.org/8392042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107718 0039d316-1c4b-4281-b951-d872f2087c98
* Implement SyncableService in AppNotificationsManager: munjal@chromium.org2011-10-211-7/+11
| | | | | | | | | | | - Implement all methods of SyncableService - Modify existing methods that change the model (Add and ClearAll) to push changes to sync. - Add some extra properties to AppNotification: guid and extension id - Disallow operations on model until storage is loaded. - Ton of unit tests for sync methods Review URL: http://codereview.chromium.org/8355030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106678 0039d316-1c4b-4281-b951-d872f2087c98
* Make NotificationService an interface in the content namespace, and switch ↵jam@chromium.org2011-10-191-2/+2
| | | | | | | | | callers to use it. Move the implementation to content/browser. Stop creating it in all child processes since it's only used in the browser. BUG=98716 Review URL: http://codereview.chromium.org/8342048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106403 0039d316-1c4b-4281-b951-d872f2087c98
* Move NotificationObserver, NotificationSource, and NotificationDetails to ↵jam@chromium.org2011-10-191-4/+4
| | | | | | | | | | | content/public/browser. This patch got way bigger than I wanted, but once I moved NotificationDetails, I figured I might as well mvoe the others since they're in the same files. In hindsight, I should have converted a subset of files at a time by leaving a using statement in the header. BUG=98716 TBR=joi git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106196 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 106110 - Implement SyncableService in AppNotificationsManager:sail@chromium.org2011-10-181-11/+7
| | | | | | | | | | | | | | | | - Implement all methods of SyncableService - Modify existing methods that change the model (Add and ClearAll) to push changes to sync. - Add some extra properties to AppNotification: guid and extension id - Disallow operations on model until storage is loaded. - Ton of unit tests for sync methods There is anotehr part for the full notifications sync to work that will be done in a separate CL. Review URL: http://codereview.chromium.org/8263002 TBR=munjal@chromium.org Review URL: http://codereview.chromium.org/8345022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106136 0039d316-1c4b-4281-b951-d872f2087c98
* Implement SyncableService in AppNotificationsManager:munjal@chromium.org2011-10-181-7/+11
| | | | | | | | | | | | | - Implement all methods of SyncableService - Modify existing methods that change the model (Add and ClearAll) to push changes to sync. - Add some extra properties to AppNotification: guid and extension id - Disallow operations on model until storage is loaded. - Ton of unit tests for sync methods There is anotehr part for the full notifications sync to work that will be done in a separate CL. Review URL: http://codereview.chromium.org/8263002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106110 0039d316-1c4b-4281-b951-d872f2087c98
* Disable AppNotificationManagerTest.ExtensionUninstall under AddressSanitizerglider@chromium.org2011-10-131-1/+7
| | | | | | | | BUG=100156 TBR=timurrrr Review URL: http://codereview.chromium.org/8256013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105293 0039d316-1c4b-4281-b951-d872f2087c98
* Replace UninstalledExtensionInfo with just a string for the extension id, ↵kalman@chromium.org2011-10-101-9/+1
| | | | | | | | | | | | all that's needed now. BUG= TEST= Review URL: http://codereview.chromium.org/8165012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104687 0039d316-1c4b-4281-b951-d872f2087c98
* Make AppNotificationStorage create the database on disk lazilyasargent@chromium.org2011-10-051-1/+1
| | | | | | | | | | | | | | I realized I was unconditionally creating the database on disk, instead of waiting until we actually need to write data in. BUG=98138 TEST=Your profile dir should not contain a 'App Notifications' database until you actually have an app notification (see TEST note on http://codereview.chromium.org/8038040 for how to do this). Review URL: http://codereview.chromium.org/8133031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104193 0039d316-1c4b-4281-b951-d872f2087c98
* Persist App Notifications to diskasargent@chromium.org2011-10-051-0/+123
BUG=98138 TEST=Install a packaged app with the experimental permission, and have it call chrome.experimental.app.notify({title:"foo", bodyText:"bar"}) from one of its pages. That should make a notification appear on its icon on the NTP. After restarting chrome, that same notification should still be there. Review URL: http://codereview.chromium.org/8038040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104066 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-03-26 07:28:33 +0000