Always add bookmarks to the mobile node on Android

components/bookmarks/browser/bookmark_utils.cc

 // Copyright 2014 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 "components/bookmarks/browser/bookmark_utils.h"
 
 #include <stdint.h>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 480 matching lines @@
     model->Remove(node);
   }
 }
 
 void AddIfNotBookmarked(BookmarkModel* model,
                         const GURL& url,
                         const base::string16& title) {
   if (IsBookmarkedByUser(model, url))
     return;  // Nothing to do, a user bookmark with that url already exists.
   model->client()->RecordAction(base::UserMetricsAction("BookmarkAdded"));
-  const BookmarkNode* parent = model->GetParentForNewNodes();
+  const BookmarkNode* parent = GetParentForNewNodes(model);
   model->AddURL(parent, parent->child_count(), title, url);
 }
 
 void RemoveAllBookmarks(BookmarkModel* model, const GURL& url) {
   std::vector<const BookmarkNode*> bookmarks;
   model->GetNodesByURL(url, &bookmarks);
 
   // Remove all the user bookmarks.
   for (size_t i = 0; i < bookmarks.size(); ++i) {
     const BookmarkNode* node = bookmarks[i];
@@ skipping 50 matching lines @@
 
 bool HasDescendantsOf(const std::vector<const BookmarkNode*>& list,
                       const BookmarkNode* root) {
   for (const BookmarkNode* node : list) {
     if (IsDescendantOf(node, root))
       return true;
   }
   return false;
 }
 
+const BookmarkNode* GetParentForNewNodes(BookmarkModel* model) {
+  std::vector<const BookmarkNode*> nodes =
+      GetMostRecentlyModifiedUserFolders(model, 1);
+  DCHECK(!nodes.empty());  // This list is always padded with default folders.
+#if defined(OS_ANDROID)
+  // Default cross-platform node. We should instead return the mobile node as
+  // default on Android.
+  if (nodes[0] == model->bookmark_bar_node() && nodes[0]->child_count() == 0) {

[sky, 2016/10/10 21:02:24]

Your check should be more of are all the permance nodes empty? If so, return the
mobile node.

[kraush, 2016/10/10 22:18:05]

Will do!
I'll iterate over all the children of the root node and make sure they each have
0 children. (From my understanding, that would cover all the permanent nodes.
Let me know if that understanding is incorrect!)

+    return model->mobile_node();
+  }
+#endif
+  return nodes[0];
+}
+
 }  // namespace bookmarks