Implement support for notification.vibrate

chrome/browser/notifications/platform_notification_service_impl.cc

Index: chrome/browser/notifications/platform_notification_service_impl.cc
diff --git a/chrome/browser/notifications/platform_notification_service_impl.cc b/chrome/browser/notifications/platform_notification_service_impl.cc
index 5ecadd35d85819896607b4f7578caab6185eaa34..300f028d2cfb9b4ef9bf7f9f144191c60ba62cbd 100644
--- a/chrome/browser/notifications/platform_notification_service_impl.cc
+++ b/chrome/browser/notifications/platform_notification_service_impl.cc
@@ -295,6 +295,12 @@ Notification PlatformNotificationServiceImpl::CreateNotificationFromData(
   notification.set_context_message(display_source);
   notification.set_silent(notification_data.silent);
 
+#if defined(OS_ANDROID)

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

Sorry, I didn't mean that you had to make this Android-only, I meant that I'm
not concerned about using ~400 bytes in the very worst-case scenario per visible
Notification elsewhere.

Could you revert these changes and keep it as you had it in PS1?

[Sanghyun Park, 2015/04/27 14:00:00]

Okay I'll change to PS1.

+  // This vibrate property currently has no effect on non-Android platforms.
+  // For reducing memory, virate is not set on non-Android platforms.
+  notification.set_vibrate(notification_data.vibrate);
+#endif
+
   // Web Notifications do not timeout.
   notification.set_never_timeout(true);