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

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

Issue 1958523002: Make WebAPK naming consistent 1/2 (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/android/java/src/org/chromium/chrome/browser/notifications/NotificationPlatformBridge.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationPlatformBridge.java b/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationPlatformBridge.java
index 896e6b43592ce0e6afaab31c800ecd9984a7ce0a..fbb08491cf443fa8fa351b97aac2df3695e702f7 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationPlatformBridge.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationPlatformBridge.java
@@ -162,10 +162,10 @@ public class NotificationPlatformBridge {
}
/**
- * Returns the package for the web APK which should handle the URL.
+ * Returns the package for the WebAPK which should handle the URL.
* @param url The url to check.
- * @return Package name of the web APK which should handle the URL. Returns empty string if the
- * URL should not be handled by a web APK.
+ * @return Package name of the WebAPK which should handle the URL. Returns empty string if the
+ * URL should not be handled by a WebAPK.
*/
@CalledByNative
private String queryWebApkPackage(String url) {
@@ -316,8 +316,8 @@ public class NotificationPlatformBridge {
* @param origin The origin to whom the notification belongs.
* @param profileId
* @param incognito
- * @param webApkPackage The package of the web APK associated with the notification.
- * Empty if the notification is not associated with a web APK.
+ * @param webApkPackage The package of the WebAPK associated with the notification.
+ * Empty if the notification is not associated with a WebAPK.
* @param tag The tag of the notification. May be NULL.
* @param actionIndex The zero-based index of the action button, or -1 if not applicable.
*/
@@ -621,7 +621,7 @@ public class NotificationPlatformBridge {
}
/**
- * Bind to the Service of required Web APK, create an notification and display.
+ * Bind to the Service of required WebAPK, create an notification and display.
*/
private void displayNotificationInWebAPK(NotificationBuilderBase notificationBuilder,
long persistentNotificationId, String origin, String profileId,
@@ -778,8 +778,8 @@ public class NotificationPlatformBridge {
* @param origin The origin of the notification.
* @param profileId Id of the profile that showed the notification.
* @param incognito if the profile session was an off the record one.
- * @param webApkPackage The package of the web APK associated with the notification.
- * Empty if the notification is not associated with a web APK.
+ * @param webApkPackage The package of the WebAPK associated with the notification.
+ * Empty if the notification is not associated with a WebAPK.
* @param tag The tag of the notification. May be NULL.
* @param actionIndex
*/

Powered by Google App Engine
This is Rietveld 408576698