Upstream WebAPK notification related code

chrome/android/webapk/libs/runtime_library/src/org/chromium/webapk/lib/runtime_library/IWebApkApi.aidl

Index: chrome/android/webapk/libs/runtime_library/src/org/chromium/webapk/lib/runtime_library/IWebApkApi.aidl
diff --git a/chrome/android/webapk/libs/runtime_library/src/org/chromium/webapk/lib/runtime_library/IWebApkApi.aidl b/chrome/android/webapk/libs/runtime_library/src/org/chromium/webapk/lib/runtime_library/IWebApkApi.aidl
new file mode 100644
index 0000000000000000000000000000000000000000..e63fd4df57614519b3ac757d8a447abe9a75bd4f
--- /dev/null
+++ b/chrome/android/webapk/libs/runtime_library/src/org/chromium/webapk/lib/runtime_library/IWebApkApi.aidl
@@ -0,0 +1,21 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.

[Peter Beverloo, 2016/05/19 15:59:00]

nit: year++

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+package org.chromium.webapk.lib.runtime_library;
+
+import android.app.Notification;
+
+/**
+ * Interface for communicating between WebAPK service and Chrome.

[Peter Beverloo, 2016/05/19 15:59:00]

How frozen is this API? Does changing anything here require a rebuild of all
existing WebAPKs?

My reason for asking is that we currently check whether notifications are
disabled for the entire app, and log it using a histogram. In the future,
however, we would like to actually act on this by unsubscribing push
subscriptions, and taking away the battery-using functionality backing
notifications too. Android now has a formal API to check for this too (N+).

Is determineAppNotificationStatus() something we'd want to include in the WebAPK
library at some point?

[1] NotificationSystemStatusUtil.determineAppNotificationStatus

[pkotwicz, 2016/05/20 00:45:19]

We store the implementation of this library in a separate dex file (web_apk.dex)
within the Chrome APK. The WebAPK extracts the dex file from Chrome and uses it.
The only truly expensive thing to change is the WebAPK's Android manifest.
Changing the WebAPK's Android manifest requires the WebAPK to be regenerated by
the server and Google Play to push an updated WebAPK to the device

Yes, if determineAppNotificationStatus() does not work out of the box with a
remote android.content.Context we would need to add it to the API at some point.

+ */
+interface IWebApkApi {
+    // Gets the id of the icon to represent WebAPK notifications in status bar.
+    int getSmallIconId();
+
+    // Display a notification.
+    void displayNotification(String platformTag, int platformID, in Notification notification);

[Peter Beverloo, 2016/05/19 15:59:00]

suggestion: I'd keep these as identical as possible to the
android.app.NotificationManager API, maybe even call out that that's what it's
based on. I like that we're using that API :-).

void notifyNotification(String tag, int id, in Notification notification);
void closeNotification(String tag, int id);

+
+    // Close a notification.
+    void closeNotification(String platformTag, int platformID);
+}