The logic added above doesn't make sense! It skips all the in between folders and goes right to the root! That is not acceptable at all since the chain could be broken anywhere in between.

All parent "containers" need to be checked. Also, why are you switching from checking:

`folderId != BookmarksFolderConstants.DEFAULT_PARENT_FOLDER_ID`
for
`!folder.isRoot()` 

The way it's coded means that the parent folder has to be "fetched" before it's actually checked, where before it could be checked before it was fetched in a fail fast approach.