Remove HasCommitted check in BookmarkTabHelper::DidFinishNavigation

We need to update the starred state even when the navigation finishes
without committing, because we expect the bookmark star should always
show whether the current committed URL is bookmarked or not.

As a first step, this CL removes the check of committing in
BookmarkTabHelper::DidFinishNavigation, then adds a test to ensure
that the starred state is false on the same document navigation.

TEST: BookmarkBrowsertest.SameDocumentNavigation

Bug: 1329720
Change-Id: I49e3d0e6f7d9f6086cc01264c582c874656cf605
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3649399
Commit-Queue: Gyuyoung Kim <gyuyoung@igalia.com>
Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Reviewed-by: Kevin McNee <mcnee@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1011241}