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

Unified Diff: chrome/browser/notifications/notification_platform_bridge_android.h

Issue 1969303002: Upstream WebAPK notification related code (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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/browser/notifications/notification_platform_bridge_android.h
diff --git a/chrome/browser/notifications/notification_platform_bridge_android.h b/chrome/browser/notifications/notification_platform_bridge_android.h
index 28fa0336b294d0edcbddb58a4082318dd7b2d6a4..a53ff3b659438cc3ec6a614e34d50071e5b1abc7 100644
--- a/chrome/browser/notifications/notification_platform_bridge_android.h
+++ b/chrome/browser/notifications/notification_platform_bridge_android.h
@@ -41,6 +41,7 @@ class NotificationPlatformBridgeAndroid : public NotificationPlatformBridge {
const base::android::JavaParamRef<jstring>& java_profile_id,
jboolean incognito,
const base::android::JavaParamRef<jstring>& java_tag,
+ const base::android::JavaParamRef<jstring>& java_webapk_package,
jint action_index);
// Called by the Java implementation when the notification has been closed.
@@ -69,11 +70,21 @@ class NotificationPlatformBridgeAndroid : public NotificationPlatformBridge {
static bool RegisterNotificationPlatformBridge(JNIEnv* env);
private:
- // Pair containing the information necessary in order to enable closing
- // notifications that were not created by this instance of the manager: the
- // notification's origin and tag. This list may not contain the notifications
- // that have not been interacted with since the last restart of Chrome.
- using RegeneratedNotificationInfo = std::pair<std::string, std::string>;
+ // Contains information necessary in order to enable closing notifications
+ // that were not created by this instance of the manager. This list may not
+ // contain the notifications that have not been interacted with since the last
+ // restart of Chrome.
+ struct RegeneratedNotificationInfo {
+ RegeneratedNotificationInfo();
+ RegeneratedNotificationInfo(const std::string& origin,
+ const std::string& tag,
+ const std::string& webapk_package);
+ ~RegeneratedNotificationInfo();
+
+ std::string origin;
+ std::string tag;
+ std::string webapk_package;
+ };
// Mapping of a persistent notification id to renegerated notification info.
// TODO(peter): Remove this map once notification delegate ids for Web

Powered by Google App Engine
This is Rietveld 408576698