diff options
| author | jknotten@chromium.org <jknotten@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-10-09 18:30:37 +0000 |
|---|---|---|
| committer | jknotten@chromium.org <jknotten@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-10-09 18:30:37 +0000 |
| commit | 8c7c42f5cd3d3cab81fad08b1159106184fa0c47 (patch) | |
| tree | 128924ad97a0c561625ab01a471a8bf4f351b6b9 | |
| parent | f59d5c37ba3e0963a23c1cf51672eeb7b5e72ac8 (diff) | |
| download | chromium_src-8c7c42f5cd3d3cab81fad08b1159106184fa0c47.zip chromium_src-8c7c42f5cd3d3cab81fad08b1159106184fa0c47.tar.gz chromium_src-8c7c42f5cd3d3cab81fad08b1159106184fa0c47.tar.bz2 | |
Don't retain reference to ChromeGeolocationPermissionContext
ChromeGeolocationPermissionContext owns
GeolocationInfoBarQueueController, so make sure that the callback
passed to GeolocationInfoBarQueueController doesn't increase the
reference count on ChromeGeolocationPermissionContext (which
https://chromiumcodereview.appspot.com/11072012 accidentally does).
TBR=bulach
BUG=152921
TEST=unittest:chrome_geolocation_permission_context on memory.fyi bot
Review URL: https://chromiumcodereview.appspot.com/11087030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@160881 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/browser/geolocation/chrome_geolocation_permission_context.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/geolocation/chrome_geolocation_permission_context.cc b/chrome/browser/geolocation/chrome_geolocation_permission_context.cc index 8b1c751..9964d30 100644 --- a/chrome/browser/geolocation/chrome_geolocation_permission_context.cc +++ b/chrome/browser/geolocation/chrome_geolocation_permission_context.cc @@ -40,7 +40,7 @@ ChromeGeolocationPermissionContext::ChromeGeolocationPermissionContext( new GeolocationInfoBarQueueController( base::Bind( &ChromeGeolocationPermissionContext::NotifyPermissionSet, - this), + base::Unretained(this)), profile))) { } |