Refactor notification operation functionality out of PNS

chrome/browser/notifications/notification_common.h

Index: chrome/browser/notifications/notification_common.h
diff --git a/chrome/browser/notifications/notification_common.h b/chrome/browser/notifications/notification_common.h
new file mode 100644
index 0000000000000000000000000000000000000000..236cbe966e8312ba808f3863260808c14988fc5b
--- /dev/null
+++ b/chrome/browser/notifications/notification_common.h
@@ -0,0 +1,29 @@
+// Copyright 2016 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_COMMON_H_
+#define CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_COMMON_H_
+
+namespace content {
+class BrowserContext;
+}  // namespace content
+
+// Shared functionality for both in page and persistent notification
+class NotificationCommon {
+ public:
+  // Things as user can do to a notification.
+  enum Operation {
+    NOTIFICATION_CLICK = 0,
+    NOTIFICATION_CLOSE = 1,
+    NOTIFICATION_SETTINGS = 2

[Peter Beverloo, 2016/06/29 17:32:29]

nit: since it's scoped to the NotificationCommon class, no need to prefix with
Notification again. What about {CLICK, CLOSE, SETTINGS}? You could prefix with
OPERATION_ if you so desire.

[Miguel Garcia, 2016/07/01 10:49:08]

The shorter the better, so I went with CLICK,CLOSE,SETTINGS.

+  };
+
+  // Open the Notification settings screen when clicking the right button.
+  // TODO(miguelg) have it take a Profile instead once NotificationObjectProxy
+  // is updated.
+  static void OpenNotificationSettings(
+      content::BrowserContext* browser_context);
+};
+
+#endif  // CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_COMMON_H_