Added option to bookmarkManagerPrivate.getMetaInfo() to get meta info from all bookmarks in a one f…

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

 // Copyright (c) 2012 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/api/bookmarks/bookmark_api_helpers.h"
 
 #include <math.h>  // For floor()
 #include <vector>
 
 #include "base/strings/string_number_conversions.h"
@@ skipping 116 matching lines @@
   if (node->is_folder() && !node->empty() && !recursive) {
     *error = keys::kFolderNotEmptyError;
     return false;
   }
 
   const BookmarkNode* parent = node->parent();
   model->Remove(parent, parent->GetIndexOf(node));
   return true;
 }
 
+void GetMetaInfo(const BookmarkNode* node,

[not at google - send to devlin, 2014/06/18 17:05:01]

better to pass in a const ref here to imply non-nullability

[yefimt, 2014/06/18 17:29:28]

Done.

+                 base::DictionaryValue* id_to_meta_info_map) {
+  if (node->IsVisible()) {

[not at google - send to devlin, 2014/06/18 17:05:01]

nit: can you early-return here? rather that than the whole function being
indented.

[yefimt, 2014/06/18 17:29:28]

Done.

+    const BookmarkNode::MetaInfoMap* meta_info = node->GetMetaInfoMap();
+    base::DictionaryValue* value = new base::DictionaryValue();
+    if (meta_info) {
+      BookmarkNode::MetaInfoMap::const_iterator itr;
+      for (itr = meta_info->begin(); itr != meta_info->end(); itr++) {

[not at google - send to devlin, 2014/06/18 17:05:01]

++itr not itr++

also I personally don't like declaring iterators anywhere other than inside the
for-declarative, but, up to you I suppose.

[yefimt, 2014/06/18 17:29:28]

Done
Then I need to split for statement into two lines, I think the way I have it now
is more readable.

+        value->SetStringWithoutPathExpansion(itr->first, itr->second);
+      }
+    }
+    id_to_meta_info_map->Set(base::Int64ToString(node->id()), value);
+
+    if (node->is_folder()) {
+      for (int i = 0; i < node->child_count(); ++i) {
+        GetMetaInfo(node->GetChild(i), id_to_meta_info_map);
+      }
+    }
+  }
+}
+
 }  // namespace bookmark_api_helpers
 }  // namespace extensions