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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationManagerProxyImpl.java

Issue 2832433002: [Android O] Refactor channel definitions into new class (Closed)
Patch Set: rebase 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 side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationManagerProxyImpl.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationManagerProxyImpl.java b/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationManagerProxyImpl.java
index f7092994130e4b2fd3b0c231cbb89be10f68be90..f255ca1f7358254f8504d4d30601c07ca5c775ec 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationManagerProxyImpl.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationManagerProxyImpl.java
@@ -45,7 +45,7 @@ public class NotificationManagerProxyImpl implements NotificationManagerProxy {
}
@Override
- public void createNotificationChannel(ChannelsInitializer.Channel channel) {
+ public void createNotificationChannel(ChannelDefinitions.Channel channel) {
assert BuildInfo.isAtLeastO();
/*
The code in the try-block uses reflection in order to compile as it calls APIs newer than
@@ -85,7 +85,7 @@ public class NotificationManagerProxyImpl implements NotificationManagerProxy {
}
@Override
- public void createNotificationChannelGroup(ChannelsInitializer.ChannelGroup channelGroup) {
+ public void createNotificationChannelGroup(ChannelDefinitions.ChannelGroup channelGroup) {
assert BuildInfo.isAtLeastO();
/*
The code in the try-block uses reflection in order to compile as it calls APIs newer than
@@ -141,7 +141,7 @@ public class NotificationManagerProxyImpl implements NotificationManagerProxy {
}
@Override
- public void deleteNotificationChannel(@ChannelsInitializer.ChannelId String id) {
+ public void deleteNotificationChannel(@ChannelDefinitions.ChannelId String id) {
assert BuildInfo.isAtLeastO();
/*
The code in the try-block uses reflection in order to compile as it calls APIs newer than

Powered by Google App Engine
This is Rietveld 408576698