Factor management of metrics updates into its own class.

chrome/browser/page_load_metrics/metrics_web_contents_observer.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/page_load_metrics/metrics_web_contents_observer.h"
 
 #include <algorithm>
 #include <ostream>
 #include <string>
 #include <utility>
@@ skipping 591 matching lines @@
   }
 
   if (!web_contents()->GetLastCommittedURL().SchemeIsHTTPOrHTTPS()) {
     RecordInternalError(ERR_IPC_FROM_BAD_URL_SCHEME);
     error = true;
   }
 
   if (error)
     return;
 
+  committed_load_->UpdateTiming(render_frame_host, timing, metadata);
+
   const bool is_main_frame = (render_frame_host->GetParent() == nullptr);
-  if (is_main_frame) {
-    committed_load_->UpdateTiming(timing, metadata);
-  } else {
-    committed_load_->UpdateSubFrameTiming(render_frame_host, timing, metadata);
-  }
-
   for (auto& observer : testing_observers_)
     observer.OnTimingUpdated(is_main_frame, timing, metadata);
 }
 
 void MetricsWebContentsObserver::OnUpdateTimingOverIPC(
     content::RenderFrameHost* render_frame_host,
     const mojom::PageLoadTiming& timing,
     const mojom::PageLoadMetadata& metadata) {
   DCHECK(!base::FeatureList::IsEnabled(features::kPageLoadMetricsMojofication));
   OnTimingUpdated(render_frame_host, timing, metadata);
@@ skipping 46 matching lines @@
     observer_->RemoveTestingObserver(this);
     observer_ = nullptr;
   }
 }
 
 void MetricsWebContentsObserver::TestingObserver::OnGoingAway() {
   observer_ = nullptr;
 }
 
 }  // namespace page_load_metrics