| commit | 9afacd27ad2436db5efce5a71febe8d8b169bee7 | |
| author | dewittj@chromium.org <dewittj@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | |
| Wed, 13 Nov 2013 20:23:31 +0000 (13 20:23 +0000) | ||
| committer | dewittj@chromium.org <dewittj@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | |
| Wed, 13 Nov 2013 20:23:31 +0000 (13 20:23 +0000) | ||
| tree | 0432b0902c76cac3b065946a5053ae03a8256d25 | treesnapshot (tar.gz zip) |
| parent | 33843bba42f92361e817339e3ba8ac2153dee056 | commitdiff |
Fix broken threading model in CheckDesktopNotificationPermission
Over time, this function (which is called on the IO thread) was updated
to use DesktopNotificationService. While the methods in that service
were enforced to run on the correct thread, it's not possible to get
the notification service from the ProfileIOData safely. This removes the
dependency on DesktopNotificationService from the IO-thread notification
functions.
BUG=256638
Review URL: https://codereview.chromium.org/61323002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@234886 0039d316-1c4b-4281-b951-d872f2087c98
Over time, this function (which is called on the IO thread) was updated
to use DesktopNotificationService. While the methods in that service
were enforced to run on the correct thread, it's not possible to get
the notification service from the ProfileIOData safely. This removes the
dependency on DesktopNotificationService from the IO-thread notification
functions.
BUG=256638
Review URL: https://codereview.chromium.org/61323002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@234886 0039d316-1c4b-4281-b951-d872f2087c98
14 files changed: