Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(300)

Side by Side Diff: chrome/browser/notifications/notification_common.h

Issue 2105863002: Verify that the notification response contains sensible data (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_COMMON_H_ 5 #ifndef CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_COMMON_H_
6 #define CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_COMMON_H_ 6 #define CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_COMMON_H_
7 7
8 namespace content { 8 namespace content {
9 class BrowserContext; 9 class BrowserContext;
10 } // namespace content 10 } // namespace content
11 11
12 // Shared functionality for both in page and persistent notification 12 // Shared functionality for both in page and persistent notification
13 class NotificationCommon { 13 class NotificationCommon {
14 public: 14 public:
15 // Things as user can do to a notification. 15 // Things as user can do to a notification.
16 enum Operation { 16 enum Operation {
17 CLICK = 0, 17 CLICK = 0,
18 CLOSE = 1, 18 CLOSE = 1,
19 SETTINGS = 2 19 SETTINGS = 2,
20 OPERATION_MAX = SETTINGS
20 }; 21 };
21 22
22 // Open the Notification settings screen when clicking the right button. 23 // Open the Notification settings screen when clicking the right button.
23 // TODO(miguelg) have it take a Profile instead once NotificationObjectProxy 24 // TODO(miguelg) have it take a Profile instead once NotificationObjectProxy
24 // is updated. 25 // is updated.
25 static void OpenNotificationSettings( 26 static void OpenNotificationSettings(
26 content::BrowserContext* browser_context); 27 content::BrowserContext* browser_context);
27 }; 28 };
28 29
29 #endif // CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_COMMON_H_ 30 #endif // CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_COMMON_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698