Commit 3b4b3d6f authored by Wei-Yin Chen (陳威尹)'s avatar Wei-Yin Chen (陳威尹) Committed by Commit Bot

Skip deleted partner bookmarks when counting

Bug: 823924
Change-Id: Ic024512bdd6dfe3fe3695459f15ecf926ddab99a
Reviewed-on: https://chromium-review.googlesource.com/c/1383904Reviewed-by: default avatarTheresa <twellington@chromium.org>
Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#618271}
parent acef8023
...@@ -458,7 +458,11 @@ jint BookmarkBridge::GetTotalBookmarkCount( ...@@ -458,7 +458,11 @@ jint BookmarkBridge::GetTotalBookmarkCount(
nodes.pop(); nodes.pop();
for (int i = 0; i < node->child_count(); ++i) { for (int i = 0; i < node->child_count(); ++i) {
// Empty title means deleted partner bookmarks or folders. See
// PartnerBookmarksShim::RemoveBookmark().
const BookmarkNode* child = node->GetChild(i); const BookmarkNode* child = node->GetChild(i);
if (GetTitle(child).empty())
continue;
if (child->is_folder()) { if (child->is_folder()) {
nodes.push(child); nodes.push(child);
} else { } else {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment