a few minor tweaks the bookmarks API:...

chrome/renderer/resources/extension_process_bindings.js

 // Copyright (c) 2009 The chrome Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // -----------------------------------------------------------------------------
 // NOTE: If you change this file you need to touch renderer_resources.grd to
 // have your change take effect.
 // -----------------------------------------------------------------------------
 
 var chrome;
@@ skipping 485 matching lines @@
       type: "object",
       properties: {
         title: chrome.types.optStr
       }
     },
     chrome.types.optFun
   ];
 
   // bookmark events
 
-  // Sends (id, {title, url, parentId, index})
+  // Sends (id, {title, url, parentId, index, dateAdded, dateGroupModified})

[Matt Perry, 2009/06/29 21:28:07]

btw, the convention for optional is [foo] - see [childrenIds] below.

[Erik does not do reviews, 2009/06/29 21:30:00]

normally, I'd agree, but in this case, [] indicates array/list.

[Matt Perry, 2009/06/29 21:30:50]

oh right.. that's unfortunate :(

+  // date values are milliseconds since the UTC epoch.
   chrome.bookmarks.onAdded = new chrome.Event("bookmark-added");
 
-  // Sends ({parentId, index})
+  // Sends (id, {parentId, index})
   chrome.bookmarks.onRemoved = new chrome.Event("bookmark-removed");
 
   // Sends (id, object) where object has list of properties that have changed.
   // Currently, this only ever includes 'title'.
   chrome.bookmarks.onChanged = new chrome.Event("bookmark-changed");
 
   // Sends (id, {parentId, index, oldParentId, oldIndex})
   chrome.bookmarks.onMoved = new chrome.Event("bookmark-moved");
 
   // Sends (id, [childrenIds])
@@ skipping 13 matching lines @@
     window.addEventListener('unload', function() {
         UnregisterExtension(extensionId); }, false);
     delete chrome.self.register_;
   }
 
   chrome.self.getViews = function() {
     return GetViews();
   }
 })();