Migrate extension notifications to the new NotificationDisplayService

chrome/browser/extensions/api/notifications/extension_notification_handler.cc

Index: chrome/browser/extensions/api/notifications/extension_notification_handler.cc
diff --git a/chrome/browser/extensions/api/notifications/extension_notification_handler.cc b/chrome/browser/extensions/api/notifications/extension_notification_handler.cc
new file mode 100644
index 0000000000000000000000000000000000000000..f5578ffd8a53e293fea92ce68b168741645c0372
--- /dev/null
+++ b/chrome/browser/extensions/api/notifications/extension_notification_handler.cc
@@ -0,0 +1,16 @@
+// Copyright 2017 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.
+
+#include "chrome/browser/extensions/api/notifications/extension_notification_handler.h"
+
+#include "base/logging.h"
+
+ExtensionNotificationHandler::ExtensionNotificationHandler() = default;
+
+ExtensionNotificationHandler::~ExtensionNotificationHandler() = default;
+
+void ExtensionNotificationHandler::OpenSettings(Profile* profile) {
+  // Extension notifications don't display a settings button.
+  NOTREACHED();

[Devlin, 2017/04/10 21:23:55]

What is this class used for?

[Peter Beverloo, 2017/04/10 23:30:06]

Notifications still have delegates, but we'd like to get rid of those in the
mid-term as they make a hard assumption that the browser process will outlive
the notification.

NonPersistentNotificationHandler (the parent class) provides the necessary
forwarding behaviour.

+}