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

Unified Diff: chrome/browser/notifications/notification_ui_manager_android.cc

Issue 1054573002: Implement support for notification.vibrate (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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_ui_manager_android.cc
diff --git a/chrome/browser/notifications/notification_ui_manager_android.cc b/chrome/browser/notifications/notification_ui_manager_android.cc
index 1d8f4822e78e3240f9cd14f45621e4c2f1851f9d..4ee7219df7e0b02fb0b85e76a958d742c5fd208e 100644
--- a/chrome/browser/notifications/notification_ui_manager_android.cc
+++ b/chrome/browser/notifications/notification_ui_manager_android.cc
@@ -130,6 +130,9 @@ void NotificationUIManagerAndroid::Add(const Notification& notification,
if (!icon_bitmap.isNull())
icon = gfx::ConvertToJavaBitmap(&icon_bitmap);
+ ScopedJavaLocalRef<jintArray> vibration_pattern =
+ base::android::ToJavaIntArray(env, notification.vibration_pattern());
+
Java_NotificationUIManager_displayNotification(
env,
java_object_.obj(),
@@ -139,6 +142,7 @@ void NotificationUIManagerAndroid::Add(const Notification& notification,
title.obj(),
body.obj(),
icon.obj(),
+ vibration_pattern.obj(),
notification.silent());
regenerated_notification_infos_[persistent_notification_id] =

Powered by Google App Engine
This is Rietveld 408576698