Fix presentation callback in BrowserTabStripController

`BrowserTabStripController` uses a presentation time callback registered
using `content::Compositor::RequestPresentationTimeForNextFrame()`
function to determine when a tab change is completely presented to the
user for the purpose of reporting peak memory histograms. This callback
is called even if the presentation fails. This CL updates the code to
use a presentation time callback registered using the new function
`content::Compositor::RequestSuccessfulPresentationTimeForNextFrame()`
instead which will be called after the first successful presentation.

Bug: 1378612
Change-Id: I068328b4a2c25fe7b40996c2d09472ab8903595c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4021485
Reviewed-by: Jonathan Ross <jonross@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Benoit Lize <lizeb@chromium.org>
Commit-Queue: Mohsen Izadi <mohsen@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1073504}