Updating modification time of parent when a folder node is added in it in bookmark bar.

chrome/browser/extensions/api/bookmarks/bookmarks_api.cc

Index: chrome/browser/extensions/api/bookmarks/bookmarks_api.cc
diff --git a/chrome/browser/extensions/api/bookmarks/bookmarks_api.cc b/chrome/browser/extensions/api/bookmarks/bookmarks_api.cc
index 59c0c2cfb05fd2e5f02f605e3941294950573289..90c92d204dbcd036b78b7a42463041f6eed61994 100644
--- a/chrome/browser/extensions/api/bookmarks/bookmarks_api.cc
+++ b/chrome/browser/extensions/api/bookmarks/bookmarks_api.cc
@@ -189,6 +189,8 @@ const BookmarkNode* BookmarksFunction::CreateBookmarkNode(
   if (!node) {
     error_ = keys::kNoNodeError;
     return NULL;
+  } else if (!details.url.get()) {
+    model->SetDateFolderModified(parent, base::Time::Now());

[Ken Rockot(use gerrit already), 2015/04/28 03:14:39]

This looks like the wrong code to change. Shouldn't you instead add a call to
parent->set_date_folder_modified from within
BookmarkModel::AddFolderWithMetaInfo?

On an tangential note, the if-block after the DCHECK is effectively unreachable
and should probably be removed.

[Deepak, 2015/04/28 11:23:33]

I have done changes due to 2 reasons:
1) I have followed the way parent node is getting modified when we add folders
via bookmark manager.
It is adding the folder node and then modifying the parent modification time in
BookmarkNode* CreateNewNode().

2)we should not modify parent folder time till new node get added in it. and
AddNode() called last in BookmarkModel::AddFolderWithMetaInfo().

Bookmark model is accessed by both bookmark bar as well as bookmark manager, so
does not wanted to disturb functionality here.

I agree with you that if condition is not reachable as DCHECK() is their.
I am removing this code and putting by code in the else case.

Please correct me if I misunderstood something.
Thanks

   }
 
   return node;