bookmarks: BookmarkNodeData's size() cleanups.

components/bookmarks/browser/bookmark_node_data.h

Index: components/bookmarks/browser/bookmark_node_data.h
diff --git a/components/bookmarks/browser/bookmark_node_data.h b/components/bookmarks/browser/bookmark_node_data.h
index fd7d13d824980a0dc11bf18dabcc115e753d95bc..28140a86e055d8edd3aec79ab05b55412764edb4 100644
--- a/components/bookmarks/browser/bookmark_node_data.h
+++ b/components/bookmarks/browser/bookmark_node_data.h
@@ -149,11 +149,13 @@ struct BookmarkNodeData {
   // Do we contain valid data?
   bool is_valid() const { return !elements.empty(); }
 
-  // Returns true if there is a single url.
-  bool has_single_url() const { return is_valid() && elements[0].is_url; }
-
   // Number of elements.
-  size_t size() const { return elements.size(); }

[sky, 2015/01/26 16:23:41]

It's more comment to use size() for members like this in chrome code.

[tfarina, 2015/01/28 16:32:54]

Done.

+  size_t num_elements() const { return elements.size(); }
+
+  // Returns true if there is a single url.
+  bool has_single_url() const {
+    return num_elements() == 1 && elements[0].is_url;
+  }
 
   // Clears the data.
   void Clear();