a few minor tweaks the bookmarks API:...

chrome/browser/extensions/extension_bookmarks_module.cc

 // Copyright (c) 2009 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/extensions/extension_bookmarks_module.h"
 
 #include "base/json_writer.h"
 #include "chrome/browser/bookmarks/bookmark_codec.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
 #include "chrome/browser/bookmarks/bookmark_utils.h"
@@ skipping 13 matching lines @@
   // Convert |node| into a JSON value
   static DictionaryValue* GetNodeDictionary(const BookmarkNode* node,
                                             bool recurse) {
     DictionaryValue* dict = new DictionaryValue();
     dict->SetInteger(keys::kIdKey, node->id());
 
     const BookmarkNode* parent = node->GetParent();
     if (parent)
       dict->SetInteger(keys::kParentIdKey, parent->id());
 
-    if (!node->is_folder())
+    if (!node->is_folder()) {
       dict->SetString(keys::kUrlKey, node->GetURL().spec());
+    } else {
+      // Javascript Date wants milliseconds since the epoch, ToDoubleT is
+      // seconds.
+      base::Time t = node->date_group_modified();
+      if (!t.is_null())
+        dict->SetReal(keys::kDateGroupModifiedKey, floor(t.ToDoubleT() * 1000));
+    }
 
     dict->SetString(keys::kTitleKey, node->GetTitle());
+    if (!node->date_added().is_null()) {
+      // Javascript Date wants milliseconds since the epoch, ToDoubleT is
+      // seconds.
+      dict->SetReal(keys::kDateAddedKey,
+                    floor(node->date_added().ToDoubleT() * 1000));
+    }
 
     int childCount = node->GetChildCount();
     ListValue* children = new ListValue();
     for (int i = 0; i < childCount; ++i) {
       const BookmarkNode* child = node->GetChild(i);
       if (recurse) {
         DictionaryValue* dict = GetNodeDictionary(child, true);
         children->Append(dict);
       }
     }
@@ skipping 113 matching lines @@
   JSONWriter::Write(&args, false, &json_args);
   DispatchEvent(model->profile(), keys::kOnBookmarkMoved, json_args);
 }
 
 void ExtensionBookmarkEventRouter::BookmarkNodeAdded(BookmarkModel* model,
                                                      const BookmarkNode* parent,
                                                      int index) {
   ListValue args;
   const BookmarkNode* node = parent->GetChild(index);
   args.Append(new FundamentalValue(node->id()));
-  DictionaryValue* object_args = new DictionaryValue();
-  object_args->SetString(keys::kTitleKey, node->GetTitle());
-  object_args->SetString(keys::kUrlKey, node->GetURL().spec());
-  object_args->SetInteger(keys::kParentIdKey, parent->id());
-  object_args->SetInteger(keys::kIndexKey, index);
-  args.Append(object_args);
+  DictionaryValue* obj = ExtensionBookmarks::GetNodeDictionary(node, false);
+
+  // Remove id since it's already being passed as the first argument.
+  obj->Remove(keys::kIdKey, NULL);
+  args.Append(obj);
 
   std::string json_args;
   JSONWriter::Write(&args, false, &json_args);
   DispatchEvent(model->profile(), keys::kOnBookmarkAdded, json_args);
 }
 
 void ExtensionBookmarkEventRouter::BookmarkNodeRemoved(
     BookmarkModel* model,
     const BookmarkNode* parent,
     int index) {
+  // TODO(erikkay) can this version ever be called?
+  NOTREACHED();
+}
+
+void ExtensionBookmarkEventRouter::BookmarkNodeRemoved(
+    BookmarkModel* model,
+    const BookmarkNode* parent,
+    int index,
+    const BookmarkNode* node) {
   ListValue args;
+  args.Append(new FundamentalValue(node->id()));
   DictionaryValue* object_args = new DictionaryValue();
   object_args->SetInteger(keys::kParentIdKey, parent->id());
   object_args->SetInteger(keys::kIndexKey, index);
   args.Append(object_args);
 
   std::string json_args;
   JSONWriter::Write(&args, false, &json_args);
   DispatchEvent(model->profile(), keys::kOnBookmarkRemoved, json_args);
 }
 
@@ skipping 283 matching lines @@
   }
   if (node == model->root_node() ||
       node == model->other_node() ||
       node == model->GetBookmarkBarNode()) {
     error_ = keys::kModifySpecialError;
     return false;
   }
   model->SetTitle(node, title);
   return true;
 }