Implement support for notification.vibrate

content/browser/notifications/notification_database_data.proto

Index: content/browser/notifications/notification_database_data.proto
diff --git a/content/browser/notifications/notification_database_data.proto b/content/browser/notifications/notification_database_data.proto
index 1fb6808ba80034500a02467364acd6fcda25cf2e..149d0cd8c9103bce3c99be83f22f1a2a6b7fe85d 100644
--- a/content/browser/notifications/notification_database_data.proto
+++ b/content/browser/notifications/notification_database_data.proto
@@ -34,8 +34,9 @@ message NotificationDatabaseDataProto {
     optional string body = 4;
     optional string tag = 5;
     optional string icon = 6;
-    optional bool silent = 7;
-    optional bytes data = 8;
+    repeated uint32 vibrate = 7 [packed=true];

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

uint32 should match the type you choose to use in the //content API.

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

Done.

+    optional bool silent = 8;
+    optional bytes data = 9;

[Avi (use Gerrit), 2015/04/21 17:28:47]

I am definitely no expert in protos, but from my fuzzy recollections, I thought
you never renumbered items. Where is this proto used?

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

Indeed - please don't do this. You can insert the |vibrate| entry right where it
is now, but give it number 9.

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

I see,  I'll use number 9.

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

I didn't know this. I'll add number for vibrate.
This proto is used for serializing and storing in leveldb.

   }
 
   optional NotificationData notification_data = 4;