Implement support for notification.vibrate

chrome/browser/notifications/notification_ui_manager_android.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/notifications/notification_ui_manager_android.h"
 
 #include <utility>
 
 #include "base/android/jni_array.h"
 #include "base/android/jni_string.h"
@@ skipping 100 matching lines @@
       env, notification.title());
   ScopedJavaLocalRef<jstring> body = ConvertUTF16ToJavaString(
       env, notification.message());
 
   ScopedJavaLocalRef<jobject> icon;
 
   SkBitmap icon_bitmap = notification.icon().AsBitmap();
   if (!icon_bitmap.isNull())
     icon = gfx::ConvertToJavaBitmap(&icon_bitmap);
 
+  std::vector<int64> pattern;

[Peter Beverloo, 2015/04/21 17:58:22]

nit: Please use int64_t everywhere. int64 is deprecated.

[Sanghyun Park, 2015/04/23 11:08:26]

Done.

+  if (!notification.vibrate().empty()) {

[Peter Beverloo, 2015/04/21 17:58:22]

We can simplify this block by using:

ScopedJavaLocalRef<jlongArray> vibration_pattern =
    base::android::ToJavaLongArray(env, notification.vibrate());

[Sanghyun Park, 2015/04/23 11:08:25]

We couldn't use ToJavaLongArray function directly.
Because we should add temp value at the begginning of vibrate value.
In Android platform, the first value is waiting time unlike blink.
Please refer the [1].

If we do not insert temp value in this file, I should add this temp value in
java file.
Can I move this logic to NotificationUIManager.java?

[1] http://developer.android.com/reference/android/os/Vibrator.html

[Peter Beverloo, 2015/04/26 23:26:40]

Yes - having this in Java SGTM. (It changes common structures to
platform-specific structures as close as we can to the usage.)

+    // In Android platform, the first value indicates the number of
+    // milliseconds to wait before turning the vibrator on unlike Blink.
+    // So, we need to insert any value at the beginning of vibrate value.
+    pattern.reserve(notification.vibrate().size() + 1);
+    pattern.push_back(0);
+    for (size_t i = 0; i < notification.vibrate().size(); ++i)
+      pattern.push_back(notification.vibrate()[i]);
+  }
+  ScopedJavaLocalRef<jlongArray> vibrate =
+      base::android::ToJavaLongArray(env, pattern.begin(), pattern.size());
+
   Java_NotificationUIManager_displayNotification(
       env,
       java_object_.obj(),
       persistent_notification_id,
       origin.obj(),
       tag.obj(),
       title.obj(),
       body.obj(),
       icon.obj(),
+      vibrate.obj(),
       notification.silent());
 
   regenerated_notification_infos_[persistent_notification_id] =
       std::make_pair(origin_url.spec(), notification.tag());
 
   notification.delegate()->Display();
 }
 
 bool NotificationUIManagerAndroid::Update(const Notification& notification,
                                           Profile* profile) {
@@ skipping 60 matching lines @@
   return false;
 }
 
 void NotificationUIManagerAndroid::CancelAll() {
   NOTREACHED();
 }
 
 bool NotificationUIManagerAndroid::RegisterNotificationUIManager(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }