[PageLoadMetrics] Reenable AdsMetrics and handle case where navigation aborts

It turns out that a frame can abort and continue to load resources even if it
has no previous navigation. This happens if a doc.write overwrites the frame
navigation while the navigation is still provisional.

This CL now labels all finished frame navigations (regardless of success) and
removes an incorrect dcheck.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

BUG=708570

Review-Url: https://codereview.chromium.org/2861433005
Cr-Commit-Position: refs/heads/master@{#470701}
9 files changed