Frameworks: Heads up meets SlimKat....SlimKat meets Google IO [1/4]

First introduced from apple, showed as android version on PA as hover and now found
in android code as native implementation we want to do our interpretation of this feature.
Especially thanks PA and CM for some of the UX ideas.
We have basically split this feature in two parts.
1. Googles native integration. We keep it as is and use the native flags for it. Apps
   (in our case Dialer, Mms, Email per account) can force show a heads up notification
   if the notification is important. So basically same what was shown on Google IO the first
   time to the public. Eg the dialer on SlimKat is doing exactly the same now like google want it.
   Due that we use same flags like google every future app with heads up support will automatically
   show as heads up.
2. Give the user an ability to enable heads up mode for third party apps which do not have a native
   integration. Well first thought was on which apps a user would usally use it (keeping in minde the idea from google....heads up
   is for important notifications and for nothing more)? Right on chat, email
   or social apps. On all other apps it simply does not make sense at all. Which leads into the question why the
   current integrations are handling blacklists or whitelists and enable it per default for all apps. This
   is from UX point of view an absolutely no go (just imagine the user blacklists half of the 200 apps installed lol).
   Aside that parsing long lists is a peformance problem.
   So our attempt goes into another direction. Headsup for (native integration is another story) apps is disabled
   by default. We bind the information directly to the application package. So each app carries the information itself.
   The user can either enable it with a longlick in the notification drawer on the notification or in Settings->app->App info
   as well via a check box. As soon it is enabled for this app every notification from this app will show as heads up as long
   it is not an ongoing notification.
   In general on system signed apps the user is not allowed to change the heads up state. Thats nothing the user should
   decide....the system should do. Aside that it prevends user to enable heads up for eg IME selector etc etc.
Thats the basic concept behind. In addition to this concept following things were done.
- Introduce a new expanded notification flag. Apps can send it via extra metadata to force
  a expanded heads up notification is it is needed (eg dialer).
- Add a user option to enable always expanded on heads up notification
- Add a user option to set the timeout for the heads up view.
- Introduce a snooze button. Every heads up notification has a snooze button on the top right. If the user press it
  heads up is disabled for a certain time (like a do not disturb mode). The time length is user configurable in settings.
- We had a bunch of user which want to use heads up in a more intrusive way. Current default behaviour is that a heads up
  shows up if the user did not push the heads up of this app into background. This is due that the system assumes if the user
  put it into background he or she do not want to be disturbed by notifications of this app at the moment. Of course in the moment
  the user interacted with the background notification heads up is activated again. This leads like google suggested in a very
  good non intrusive way. Well but some user use it in another way (especially on immversive mode) and want to be notified on every
  notification update of this app via heads up. Due of this understandable other user behaviour we add an option for it to allow
  this scenario.
- Detect direction of the swipe. Left swipe will dismiss the notification (if it is not a non clearable). Right swipe will just
  hide the heads up and the notification is still in background accessable in the notification drawer.
Other then that some fixes were done as well (eg utilize the back button, correctly handle the hiding of the heads up,
handle action buttons on heads up view correctly, fix several expanded desktop and immersive issues
and some other issues the AOSP implementation currently has)
To summarize...heads up this way is very non intrusive and especially we do not need a bunch of settings or blacklists.
So it is pretty straight forward, fast, clean and user friendly.
Some visual examples:
https://plus.google.com/107859134357373490612/posts/bJABHs1aNMj

Conflicts:
	core/java/android/app/Notification.java
	core/java/android/provider/Settings.java
	core/res/res/values/slim_strings.xml
	packages/SystemUI/res/values/slim_strings.xml
	packages/SystemUI/src/com/android/systemui/statusbar/BaseStatusBar.java
	packages/SystemUI/src/com/android/systemui/statusbar/CommandQueue.java
	packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBar.java
	packages/SystemUI/src/com/android/systemui/statusbar/tv/TvStatusBar.java
	services/java/com/android/server/pm/Settings.java

Author: kufikugel

core/java/android/app/ActivityManagerNative.java

@@ -2056,6 +2056,15 @@ public boolean onTransact(int code, Parcel data, Parcel reply, int flags)
             return true;
         }
 
+        case IS_HEADS_UP_ENABLED_TRANSACTION: {
+            data.enforceInterface(IActivityManager.descriptor);
+            int pid = data.readInt();
+            boolean res = isHeadsUpEnabledForProcess(pid);
+            reply.writeNoException();
+            reply.writeInt(res ? 1 : 0);
+            return true;
+        }
+
         }
 
         return super.onTransact(code, data, reply, flags);
@@ -4740,5 +4749,19 @@ public void notifySplitViewLayoutChanged() throws RemoteException {
         reply.recycle();
     }
 
+
+    public boolean isHeadsUpEnabledForProcess(int pid) throws RemoteException {
+        Parcel data = Parcel.obtain();
+        Parcel reply = Parcel.obtain();
+        data.writeInterfaceToken(IActivityManager.descriptor);
+        data.writeInt(pid);
+        mRemote.transact(IS_HEADS_UP_ENABLED_TRANSACTION, data, reply, 0);
+        reply.readException();
+        int res = reply.readInt();
+        data.recycle();
+        reply.recycle();
+        return res == 1;
+    }
+
     private IBinder mRemote;
 }

core/java/android/app/ApplicationPackageManager.java

@@ -49,6 +49,7 @@
 import android.content.res.XmlResourceParser;
 import android.graphics.drawable.Drawable;
 import android.net.Uri;
+import android.os.Binder;
 import android.os.Process;
 import android.os.RemoteException;
 import android.os.UserHandle;
@@ -1382,6 +1383,31 @@ public boolean getApplicationBlockedSettingAsUser(String packageName, UserHandle
         return false;
     }
 
+    /**
+     * @hide
+     */
+    @Override
+    public void setHeadsUpSetting(String packageName, boolean enabled) {
+        try {
+            mPM.setHeadsUpSetting(packageName, enabled, mContext.getUserId());
+        } catch (RemoteException e) {
+            // Should never happen!
+        }
+    }
+
+    /**
+     * @hide
+     */
+    @Override
+    public boolean getHeadsUpSetting(String packageName) {
+        try {
+            return mPM.getHeadsUpSetting(packageName, mContext.getUserId());
+        } catch (RemoteException e) {
+            // Should never happen!
+        }
+        return false;
+    }
+
     /**
      * @hide
      */

core/java/android/app/ContextImpl.java

@@ -1705,6 +1705,17 @@ private WallpaperManager getWallpaperManager() {
         return new DropBoxManager(service);
     }
 
+    @Override
+    public boolean isHeadsUpEnabled() {
+        try {
+            return ActivityManagerNative.getDefault()
+                    .isHeadsUpEnabledForProcess(Binder.getCallingPid());
+        } catch (RemoteException e) {
+            Log.e(TAG, e.getMessage(), e);
+        }
+        return false;
+    }
+
     @Override
     public int checkPermission(String permission, int pid, int uid) {
         if (permission == null) {

core/java/android/app/IActivityManager.java

@@ -206,6 +206,8 @@ public int handleIncomingUser(int callingPid, int callingUid, int userId, boolea
     public void setProcessForeground(IBinder token, int pid,
             boolean isForeground) throws RemoteException;
 
+    public boolean isHeadsUpEnabledForProcess(int pid) throws RemoteException;
+
     public int checkPermission(String permission, int pid, int uid)
             throws RemoteException;
 
@@ -700,6 +702,7 @@ private WaitResult(Parcel source) {
     int GET_PERSISTED_URI_PERMISSIONS_TRANSACTION = IBinder.FIRST_CALL_TRANSACTION+181;
     int APP_NOT_RESPONDING_VIA_PROVIDER_TRANSACTION = IBinder.FIRST_CALL_TRANSACTION+182;
     int GET_CALLING_PACKAGE_FOR_BROADCAST_TRANSACTION = IBinder.FIRST_CALL_TRANSACTION+183;
+    int IS_HEADS_UP_ENABLED_TRANSACTION = IBinder.FIRST_CALL_TRANSACTION+184;
     /* SPLIT VIEW */
     int GET_ACTIVITY_FOR_TASK_TRANSACTION = IBinder.FIRST_CALL_TRANSACTION+200;
     int NOTIFY_SPLIT_VIEW_LAYOUT_CHANGED = IBinder.FIRST_CALL_TRANSACTION+201;

core/java/android/app/Notification.java

@@ -568,7 +568,7 @@ public class Notification implements Parcelable
     public static final String EXTRA_FORCE_SHOW_LIGHTS = "android.forceShowLights";
 
     /**
-     * Value for {@link #EXTRA_AS_HEADS_UP}.
+     * Default alue for {@link #EXTRA_AS_HEADS_UP}.
      * @hide
      */
     public static final int HEADS_UP_NEVER = 0;
@@ -581,11 +581,29 @@ public class Notification implements Parcelable
 
 
     /**
-     * Value for {@link #EXTRA_AS_HEADS_UP}.
+     * Default value for {@link #EXTRA_AS_HEADS_UP}.
      * @hide
      */
     public static final int HEADS_UP_REQUESTED = 2;
 
+    /**
+     * Not used.
+     * @hide
+     */
+    public static final String EXTRA_HEADS_UP_EXPANDED = "headsupExpanded";
+
+    /**
+     * Value for {@link #EXTRA_HEADS_UP_EXPANDED}.
+     * @hide
+     */
+    public static final int HEADS_UP_EXPANDED = 0;
+
+    /**
+     * Default value for {@link #EXTRA_HEADS_UP_EXPANDED}.
+     * @hide
+     */
+    public static final int HEADS_UP_NOT_EXPANDED = 1;
+
     /**
      * Structure to encapsulate a named action that can be shown as part of this notification.
      * It must include an icon, a label, and a {@link PendingIntent} to be fired when the action is
@@ -1788,9 +1806,7 @@ private RemoteViews generateActionButton(Action action) {
                               : R.layout.notification_action);
             button.setTextViewCompoundDrawablesRelative(R.id.action0, action.icon, 0, 0, 0);
             button.setTextViewText(R.id.action0, action.title);
-            if (!tombstone) {
-                button.setOnClickPendingIntent(R.id.action0, action.actionIntent);
-            }
+            button.setOnClickPendingIntent(R.id.action0, action.actionIntent, tombstone);
             button.setContentDescription(R.id.action0, action.title);
             return button;
         }

core/java/android/content/Context.java

@@ -2439,6 +2439,15 @@ public abstract boolean startInstrumentation(ComponentName className,
      */
     public static final String USER_SERVICE = "user";
 
+    /**
+     * Determine whether the application or calling application has
+     * heads up notification enabled. Non system applications and content providers
+     * can check this value if they wish to honor the heads up feature.
+     *
+     * @hide
+     */
+    public abstract boolean isHeadsUpEnabled();
+
     /**
      * Use with {@link #getSystemService} to retrieve a
      * {@link android.app.AppOpsManager} for tracking application operations

core/java/android/content/ContextWrapper.java

@@ -545,6 +545,12 @@ public int checkPermission(String permission, int pid, int uid) {
         return mBase.checkPermission(permission, pid, uid);
     }
 
+    /** @hide */
+    @Override
+    public boolean isHeadsUpEnabled() {
+        return mBase.isHeadsUpEnabled();
+    }
+
     @Override
     public int checkCallingPermission(String permission) {
         return mBase.checkCallingPermission(permission);

core/java/android/content/pm/IPackageManager.aidl

@@ -238,6 +238,10 @@ interface IPackageManager {
     int getPreferredActivities(out List<IntentFilter> outFilters,
             out List<ComponentName> outActivities, String packageName);
 
+    boolean getHeadsUpSetting(in String packageName, int userId);
+
+    void setHeadsUpSetting(in String packageName, boolean enabled, int userId);
+
     /**
      * Report the set of 'Home' activity candidates, plus (if any) which of them
      * is the current "always use this one" setting.

core/java/android/content/pm/PackageManager.java

@@ -3221,6 +3221,22 @@ public abstract void setApplicationEnabledSetting(String packageName,
      */
     public abstract int getApplicationEnabledSetting(String packageName);
 
+    /**
+     * @param packageName
+     * @return
+     *
+     * @hide
+     */
+    public abstract boolean getHeadsUpSetting(String packageName);
+
+    /**
+     * @param packageName
+     * @param enabled
+     *
+     * @hide
+     */
+    public abstract void setHeadsUpSetting(String packageName, boolean enabled);
+
     /**
      * Puts the package in a blocked state, which is almost like an uninstalled state,
      * making the package unavailable, but it doesn't remove the data or the actual

core/java/android/content/pm/PackageUserState.java

@@ -30,6 +30,7 @@ public class PackageUserState {
     public boolean installed;
     public boolean blocked; // Is the app restricted by owner / admin
     public int enabled;
+    public boolean headsUp;
 
     public String lastDisableAppCaller;
 
@@ -42,13 +43,15 @@ public PackageUserState() {
         installed = true;
         blocked = false;
         enabled = COMPONENT_ENABLED_STATE_DEFAULT;
+        headsUp = false;
     }
 
     public PackageUserState(PackageUserState o) {
         installed = o.installed;
         stopped = o.stopped;
         notLaunched = o.notLaunched;
         enabled = o.enabled;
+        headsUp = o.headsUp;
         blocked = o.blocked;
         lastDisableAppCaller = o.lastDisableAppCaller;
         disabledComponents = o.disabledComponents != null
@@ -60,4 +63,4 @@ public PackageUserState(PackageUserState o) {
         visibleComponents = o.visibleComponents != null
                 ? new HashSet<String>(o.visibleComponents) : null;
     }
-}
+}