Fix functional tests notifications.NotificationsTest(s)

Fix functional tests:
notifications.NotificationsTest.testAllowNotificationsFromAllSites
notifications.NotificationsTest.testCrashRendererNotificationRemain
notifications.NotificationsTest.testCreateDenyCloseNotifications
notifications.NotificationsTest.testCreateHTMLNotification
notifications.NotificationsTest.testCreateSimpleNotification

BUG=76693
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=81383

[markusheintz_, 2011-04-12 13:02:25]

Pls review. THX

[markusheintz_, 2011-04-12 15:53:30]

@kkania: Pleas review my CL. Thanks

[kkania, 2011-04-12 16:42:42]

http://codereview.chromium.org/6823076/diff/2001/chrome/browser/content_setti...
File chrome/browser/content_settings/content_settings_pref_provider.cc (right):

http://codereview.chromium.org/6823076/diff/2001/chrome/browser/content_setti...
chrome/browser/content_settings/content_settings_pref_provider.cc:79:
default_settings->SetInteger(std::string(kTypeNames[i]),
remove std::string

http://codereview.chromium.org/6823076/diff/2001/chrome/browser/content_setti...
chrome/browser/content_settings/content_settings_pref_provider.cc:304: //
expliciatly we set them here.
this comment is a bit wordy yet confusing; part of this is due to 'of' instead
of 'if' in line 301. Please revise

[markusheintz_, 2011-04-12 17:21:49]

http://codereview.chromium.org/6823076/diff/2001/chrome/browser/content_setti...
File chrome/browser/content_settings/content_settings_pref_provider.cc (right):

http://codereview.chromium.org/6823076/diff/2001/chrome/browser/content_setti...
chrome/browser/content_settings/content_settings_pref_provider.cc:79:
default_settings->SetInteger(std::string(kTypeNames[i]),
On 2011/04/12 16:42:42, kkania wrote:
> remove std::string

Done.

http://codereview.chromium.org/6823076/diff/2001/chrome/browser/content_setti...
chrome/browser/content_settings/content_settings_pref_provider.cc:304: //
expliciatly we set them here.
On 2011/04/12 16:42:42, kkania wrote:
> this comment is a bit wordy yet confusing; part of this is due to 'of' instead
> of 'if' in line 301. Please revise

Done.

[kkania, 2011-04-12 18:36:49]

LGTM

it seems many of the other pyauto notifications tests are still failing still
because of 79054

[kkania, 2011-04-12 18:37:31]

crbug.com/79054, to be clear