GitLab

Prior to this CL, Startup.FirstWebContents.NonEmptyPaint2 was recorded for the
active WebContents in the first browser of the BrowserList. However, tests
highlighted that the first browser is not necessarily visible in session restore.
This CL changes the behavior to instead record
Startup.FirstWebContents.NonEmptyPaint2 for the active WebContents in the
first *visible* browser. It also adds a FinishReason for when no browser is
initially visible. This would allow us to track if there is a change in how often
Chrome starts with no initially visible browser (e.g. if this CL lands
https://chromium-review.googlesource.com/c/chromium/src/+/1888405/3/chrome/browser/sessions/session_restore.cc#477
we would start more often with no initially visible browser).

This CL also adds some DCHECKs to make sure that navigation events occur in the
order expected by FirstWebContentsProfiler.

Bug: 1020549
Change-Id: I0733a2b5af2f3617102232e9996459ad3b1c4c5b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1894417Reviewed-by: Gabriel Charette <gab@chromium.org>
Commit-Queue: François Doray <fdoray@chromium.org>
Cr-Commit-Position: refs/heads/master@{#713192}