Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Don't throw RemoteException from new NLS APIs" into nyc-dev

Browse Source
This commit is contained in:
TreeHugger Robot
2016-05-23 18:54:17 +00:00
committed by Android (Google) Code Review
parent 7c527bf76d cf548bfad6
commit 28d9ef8112
4 changed files with 20 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
api/current.txt
View File

@@ -34633,8 +34633,8 @@ package android.service.notification {
method public void onNotificationRemoved(android.service.notification.StatusBarNotification, android.service.notification.NotificationListenerService.RankingMap); method public void onNotificationRemoved(android.service.notification.StatusBarNotification, android.service.notification.NotificationListenerService.RankingMap);
method public final void requestInterruptionFilter(int); method public final void requestInterruptionFilter(int);
method public final void requestListenerHints(int); method public final void requestListenerHints(int);
method public static void requestRebind(android.content.ComponentName) throws android.os.RemoteException; method public static void requestRebind(android.content.ComponentName);
method public final void requestUnbind() throws android.os.RemoteException; method public final void requestUnbind();
method public final void setNotificationsShown(java.lang.String[]); method public final void setNotificationsShown(java.lang.String[]);
field public static final int HINT_HOST_DISABLE_CALL_EFFECTS = 4; // 0x4 field public static final int HINT_HOST_DISABLE_CALL_EFFECTS = 4; // 0x4
field public static final int HINT_HOST_DISABLE_EFFECTS = 1; // 0x1 field public static final int HINT_HOST_DISABLE_EFFECTS = 1; // 0x1

4
api/system-current.txt
View File

@@ -37359,8 +37359,8 @@ package android.service.notification {
method public void registerAsSystemService(android.content.Context, android.content.ComponentName, int) throws android.os.RemoteException; method public void registerAsSystemService(android.content.Context, android.content.ComponentName, int) throws android.os.RemoteException;
method public final void requestInterruptionFilter(int); method public final void requestInterruptionFilter(int);
method public final void requestListenerHints(int); method public final void requestListenerHints(int);
method public static void requestRebind(android.content.ComponentName) throws android.os.RemoteException; method public static void requestRebind(android.content.ComponentName);
method public final void requestUnbind() throws android.os.RemoteException; method public final void requestUnbind();
method public final void setNotificationsShown(java.lang.String[]); method public final void setNotificationsShown(java.lang.String[]);
method public final void setOnNotificationPostedTrim(int); method public final void setOnNotificationPostedTrim(int);
method public void unregisterAsSystemService() throws android.os.RemoteException; method public void unregisterAsSystemService() throws android.os.RemoteException;

4
api/test-current.txt
View File

@@ -34710,8 +34710,8 @@ package android.service.notification {
method public void onNotificationRemoved(android.service.notification.StatusBarNotification, android.service.notification.NotificationListenerService.RankingMap); method public void onNotificationRemoved(android.service.notification.StatusBarNotification, android.service.notification.NotificationListenerService.RankingMap);
method public final void requestInterruptionFilter(int); method public final void requestInterruptionFilter(int);
method public final void requestListenerHints(int); method public final void requestListenerHints(int);
method public static void requestRebind(android.content.ComponentName) throws android.os.RemoteException; method public static void requestRebind(android.content.ComponentName);
method public final void requestUnbind() throws android.os.RemoteException; method public final void requestUnbind();
method public final void setNotificationsShown(java.lang.String[]); method public final void setNotificationsShown(java.lang.String[]);
field public static final int HINT_HOST_DISABLE_CALL_EFFECTS = 4; // 0x4 field public static final int HINT_HOST_DISABLE_CALL_EFFECTS = 4; // 0x4
field public static final int HINT_HOST_DISABLE_EFFECTS = 1; // 0x1 field public static final int HINT_HOST_DISABLE_EFFECTS = 1; // 0x1

21
core/java/android/service/notification/NotificationListenerService.java
View File

@@ -790,11 +790,14 @@ public abstract class NotificationListenerService extends Service {
* <p>This method will fail for listeners that have * <p>This method will fail for listeners that have
* not been granted the permission by the user. * not been granted the permission by the user.
*/ */
public static void requestRebind(ComponentName componentName) public static void requestRebind(ComponentName componentName) {
throws RemoteException {
INotificationManager noMan = INotificationManager.Stub.asInterface( INotificationManager noMan = INotificationManager.Stub.asInterface(
ServiceManager.getService(Context.NOTIFICATION_SERVICE)); ServiceManager.getService(Context.NOTIFICATION_SERVICE));
noMan.requestBindListener(componentName); try {
noMan.requestBindListener(componentName);
} catch (RemoteException ex) {
throw ex.rethrowFromSystemServer();
}
} }
/** /**
@@ -807,12 +810,16 @@ public abstract class NotificationListenerService extends Service {
* <p>The service should wait for the {@link #onListenerConnected()} event * <p>The service should wait for the {@link #onListenerConnected()} event
* before performing this operation. I know it's tempting, but you must wait. * before performing this operation. I know it's tempting, but you must wait.
*/ */
public final void requestUnbind() throws RemoteException { public final void requestUnbind() {
if (mWrapper != null) { if (mWrapper != null) {
INotificationManager noMan = getNotificationInterface(); INotificationManager noMan = getNotificationInterface();
noMan.requestUnbindListener(mWrapper); try {
// Disable future messages. noMan.requestUnbindListener(mWrapper);
isConnected = false; // Disable future messages.
isConnected = false;
} catch (RemoteException ex) {
throw ex.rethrowFromSystemServer();
}
} }
} }