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

Unified Diff: ash/system/chromeos/bluetooth/bluetooth_notification_controller.cc

Issue 684253002: Provide default implementations for NotificationDelegate methods. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 6 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: ash/system/chromeos/bluetooth/bluetooth_notification_controller.cc
diff --git a/ash/system/chromeos/bluetooth/bluetooth_notification_controller.cc b/ash/system/chromeos/bluetooth/bluetooth_notification_controller.cc
index e64bc77c707ba87f5c19709a9ea59266cff94791..146be2c43c8750807442bd216475b8f2deffe69d 100644
--- a/ash/system/chromeos/bluetooth/bluetooth_notification_controller.cc
+++ b/ash/system/chromeos/bluetooth/bluetooth_notification_controller.cc
@@ -58,12 +58,8 @@ class BluetoothPairingNotificationDelegate
virtual ~BluetoothPairingNotificationDelegate();
// message_center::NotificationDelegate overrides.
- virtual void Display() override;
- virtual void Error() override;
- virtual void Close(bool by_user) override;
- virtual bool HasClickedListener() override;
- virtual void Click() override;
- virtual void ButtonClick(int button_index) override;
+ void Close(bool by_user) override;
+ void ButtonClick(int button_index) override;
private:
// Buttons that appear in notifications.
@@ -93,12 +89,6 @@ BluetoothPairingNotificationDelegate::BluetoothPairingNotificationDelegate(
BluetoothPairingNotificationDelegate::~BluetoothPairingNotificationDelegate() {
}
-void BluetoothPairingNotificationDelegate::Display() {
-}
-
-void BluetoothPairingNotificationDelegate::Error() {
-}
-
void BluetoothPairingNotificationDelegate::Close(bool by_user) {
VLOG(1) << "Pairing notification closed. by_user = " << by_user;
// Ignore notification closes generated as a result of pairing completion.
@@ -111,13 +101,6 @@ void BluetoothPairingNotificationDelegate::Close(bool by_user) {
device->CancelPairing();
}
-bool BluetoothPairingNotificationDelegate::HasClickedListener() {
- return false;
-}
-
-void BluetoothPairingNotificationDelegate::Click() {
-}
-
void BluetoothPairingNotificationDelegate::ButtonClick(int button_index) {
VLOG(1) << "Pairing notification, button click: " << button_index;
// If the device object still exists, send the appropriate response either
« no previous file with comments | « ash/display/resolution_notification_controller.cc ('k') | ash/system/chromeos/screen_security/screen_tray_item.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698