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

Side by Side Diff: chrome/browser/extensions/api/notifications/extension_notification_handler.cc

Issue 2799343003: Add support for native extension notifications (Closed)
Patch Set: Created 3 years, 8 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 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 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 #include "chrome/browser/extensions/api/notifications/extension_notification_han dler.h" 5 #include "chrome/browser/extensions/api/notifications/extension_notification_han dler.h"
6 6
7 #include "base/logging.h" 7 ExtensionNotificationHandler::ExtensionNotificationHandler()
8 8 : NonPersistentNotificationHandler() {}
9 ExtensionNotificationHandler::ExtensionNotificationHandler() = default;
10 9
11 ExtensionNotificationHandler::~ExtensionNotificationHandler() = default; 10 ExtensionNotificationHandler::~ExtensionNotificationHandler() = default;
12 11
13 void ExtensionNotificationHandler::OnClose(Profile* profile,
14 const std::string& origin,
15 const std::string& notification_id,
16 bool by_user) {
17 // TODO(peter): Call EraseDataForNotificationId from here once this event
18 // can be relied upon. (It's only been implemented for native NDSes.)
19 }
20
21 void ExtensionNotificationHandler::OnClick(
22 Profile* profile,
23 const std::string& origin,
24 const std::string& notification_id,
25 int action_index,
26 const base::NullableString16& reply) {}
27
28 void ExtensionNotificationHandler::OpenSettings(Profile* profile) { 12 void ExtensionNotificationHandler::OpenSettings(Profile* profile) {
29 // Extension notifications don't display a settings button. 13 // Extension notifications don't display a settings button.
30 NOTREACHED(); 14 NOTREACHED();
31 } 15 }
32
33 void ExtensionNotificationHandler::RegisterNotification(
34 const std::string& notification_id,
35 NotificationDelegate* delegate) {}
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698