Simplify permission-related code for Web Notifications.

chrome/browser/notifications/notification_permission_context.h

+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_PERMISSION_CONTEXT_H_
+#define CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_PERMISSION_CONTEXT_H_
+
+#include "chrome/browser/content_settings/permission_context_base.h"
+#include "components/content_settings/core/common/content_settings.h"
+
+class GURL;
+class Profile;
+
+class NotificationPermissionContext : public PermissionContextBase {
+ public:
+  explicit NotificationPermissionContext(Profile* profile);
+  ~NotificationPermissionContext() override;
+
+  // PermissionContextBase implementation (public).

[mlamouri (slow - plz ping), 2015/06/30 14:27:11]

nit: I've never seen "(public)" being mentioned that way. It's also pretty
obvious it is public. I would remove it.

[Peter Beverloo, 2015/06/30 14:35:23]

Done.

+  void ResetPermission(const GURL& requesting_origin,
+                       const GURL& embedder_origin) override;
+
+ private:
+  FRIEND_TEST_ALL_PREFIXES(NotificationPermissionContextTest,
+                           IgnoresEmbedderOrigin);
+  FRIEND_TEST_ALL_PREFIXES(NotificationPermissionContextTest,
+                           NoSecureOriginRequirement);

[mlamouri (slow - plz ping), 2015/06/30 14:27:11]

I think having a NotificationPermissionContextTest class as friend would be
better than listing all the tests.

[Peter Beverloo, 2015/06/30 14:35:23]

Acknowledged.

+
+  // PermissionContextBase implementation (private).

[mlamouri (slow - plz ping), 2015/06/30 14:27:11]

ditto.

[Peter Beverloo, 2015/06/30 14:35:23]

Done.

+  void UpdateContentSetting(const GURL& requesting_origin,
+                            const GURL& embedder_origin,
+                            ContentSetting content_setting) override;
+  bool IsRestrictedToSecureOrigins() const override;
+};
+
+#endif  // CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_PERMISSION_CONTEXT_H_