Refactor PageLoadExtraInfo::committed_url to url and did_commit fields.

chrome/browser/page_load_metrics/metrics_web_contents_observer.cc

Index: chrome/browser/page_load_metrics/metrics_web_contents_observer.cc
diff --git a/chrome/browser/page_load_metrics/metrics_web_contents_observer.cc b/chrome/browser/page_load_metrics/metrics_web_contents_observer.cc
index 58f4856a62e20a7424b33c4f64c7f561460547b2..a2e344789f29ff11a2364df8c901bcee6ac8534e 100644
--- a/chrome/browser/page_load_metrics/metrics_web_contents_observer.cc
+++ b/chrome/browser/page_load_metrics/metrics_web_contents_observer.cc
@@ -157,7 +157,7 @@ void MetricsWebContentsObserver::WillStartNavigationRequest(
           ? web_contents()->GetOpener()->GetLastCommittedURL()
           : GURL::EmptyGURL();
   const GURL& currently_committed_url =
-      committed_load_ ? committed_load_->committed_url() : opener_url;
+      committed_load_ ? committed_load_->url() : opener_url;
   has_navigated_ = true;
 
   // We can have two provisional loads in some cases. E.g. a same-site
@@ -345,6 +345,7 @@ void MetricsWebContentsObserver::HandleCommittedNavigationForTrackedLoad(
 
   committed_load_ = std::move(tracker);
   committed_load_->Commit(navigation_handle);
+  DCHECK(committed_load_->did_commit());
 }
 
 void MetricsWebContentsObserver::NavigationStopped() {