From c2632ab128e13820d4203e2973a85033df9fbd38 Mon Sep 17 00:00:00 2001 From: "jhawkins@chromium.org" Date: Thu, 17 Nov 2011 23:21:58 +0000 Subject: Valgrind: Fix a leak in ContentSettingsProvider. If the call to SetWebsiteSetting() fails, the value is still owned by the caller and must be deleted. BUG=none TEST=none R=bauerb@chromium.org Review URL: http://codereview.chromium.org/8592001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110594 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/content_settings/content_settings_provider.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'chrome/browser/content_settings/content_settings_provider.h') diff --git a/chrome/browser/content_settings/content_settings_provider.h b/chrome/browser/content_settings/content_settings_provider.h index 1ec0851..0b0a64e 100644 --- a/chrome/browser/content_settings/content_settings_provider.h +++ b/chrome/browser/content_settings/content_settings_provider.h @@ -42,7 +42,7 @@ class ProviderInterface { const ResourceIdentifier& resource_identifier, bool incognito) const = 0; - // Askes the provider to set the website setting for a particular + // Asks the provider to set the website setting for a particular // |primary_pattern|, |secondary_pattern|, |content_type| tuple. If the // provider accepts the setting it returns true and takes the ownership of the // |value|. Otherwise false is returned and the ownership of the |value| stays -- cgit v1.1