tree 0a9bf2757d59d2ba43dbd62762eba5879ad5310f
parent 8738bcbddfc12bba6be816a7e17bdb4b9dbb9f6c
author Mikel Astiz <mastiz@chromium.org> 1721667391 +0000
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1721667391 +0000

[bookmarks] Relax preconditions for IsLocalOnlyNode()

Some tests run into flakiness and exercise this codepath. Although the
semantics of the predicate are unclear during these edge cases, it seems
better than CHECK-failing to return an arbitrary value, in this case
`true` in consistency with prior precedent for the root node.

Bug: None
Change-Id: Id15ce5e1819a5970a3be4e4c9e11d70287e278d3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5724375
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Justin Cohen <justincohen@chromium.org>
Auto-Submit: Mikel Astiz <mastiz@chromium.org>
Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Reviewed-by: Jérôme Lebel <jlebel@chromium.org>
Code-Coverage: findit-for-me@appspot.gserviceaccount.com <findit-for-me@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#1331135}