Fix observing loading behavior in DocumentLoader

Regressed by crrev.com/83d5d416.

DidObserveLoadingBehavior should come after DispatchDidCommitLoad,
and the order got reversed while moving DispatchDidCommitLoad call
a bit later in the navigation commit process.

Bug: 1020082
Change-Id: I0c636f8d64b917a49902e560a38b1f0cb3c7bfe8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1893688Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Commit-Queue: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#711622}

third_party/blink/renderer/core/loader/document_loader.cc

@@ -1420,15 +1420,6 @@ void DocumentLoader::DidCommitNavigation() {
   // resumed.
   frame_->ResumeSubresourceLoading();
 
-  // DidObserveLoadingBehavior() must be called after DispatchDidCommitLoad() is
-  // called for the metrics tracking logic to handle it properly.
-  if (service_worker_network_provider_ &&
-      service_worker_network_provider_->GetControllerServiceWorkerMode() ==
-          blink::mojom::ControllerServiceWorkerMode::kControlled) {
-    GetLocalFrameClient().DidObserveLoadingBehavior(
-        kLoadingBehaviorServiceWorkerControlled);
-  }
-
   Document* document = frame_->GetDocument();
   InteractiveDetector* interactive_detector =
       InteractiveDetector::From(*document);
@@ -1600,6 +1591,15 @@ void DocumentLoader::InstallNewDocument(
 void DocumentLoader::CreateParserPostCommit() {
   Document* document = frame_->GetDocument();
 
+  // DidObserveLoadingBehavior() must be called after DispatchDidCommitLoad() is
+  // called for the metrics tracking logic to handle it properly.
+  if (service_worker_network_provider_ &&
+      service_worker_network_provider_->GetControllerServiceWorkerMode() ==
+          blink::mojom::ControllerServiceWorkerMode::kControlled) {
+    GetLocalFrameClient().DidObserveLoadingBehavior(
+        kLoadingBehaviorServiceWorkerControlled);
+  }
+
   // Links with media values need more information (like viewport information).
   // This happens after the first chunk is parsed in HTMLDocumentParser.
   DispatchLinkHeaderPreloads(base::nullopt /* viewport */,