Remove dom_loading from page_load_metrics

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 74 matching lines @@
     return false;
 
   // If we have a non-empty timing, it should always have a navigation start.
   if (timing.navigation_start.is_null()) {
     NOTREACHED() << "Received null navigation_start.";
     return false;
   }
 
   // Verify proper ordering between the various timings.
 
-  if (!EventsInOrder(timing.response_start, timing.dom_loading)) {
-    // We sometimes get a zero response_start with a non-zero DOM loading. See
-    // crbug.com/590212.
-    DLOG(ERROR) << "Invalid response_start " << timing.response_start
-                << " for dom_loading " << timing.dom_loading;
-    return false;
-  }
-
   if (!EventsInOrder(timing.response_start, timing.parse_start)) {
     // We sometimes get a zero response_start with a non-zero parse start. See
     // crbug.com/590212.
     DLOG(ERROR) << "Invalid response_start " << timing.response_start
                 << " for parse_start " << timing.parse_start;
     return false;
   }
 
   if (!EventsInOrder(timing.parse_start, timing.parse_stop)) {
     NOTREACHED() << "Invalid parse_start " << timing.parse_start
@@ skipping 15 matching lines @@
   if (timing.parse_blocked_on_script_load_from_document_write_duration >
       timing.parse_blocked_on_script_load_duration) {
     NOTREACHED()
         << "Invalid parse_blocked_on_script_load_from_document_write_duration "
         << timing.parse_blocked_on_script_load_from_document_write_duration
         << " for parse_blocked_on_script_load_duration "
         << timing.parse_blocked_on_script_load_duration;
     return false;
   }
 
-  if (!EventsInOrder(timing.dom_loading,
+  if (!EventsInOrder(timing.parse_stop,
                      timing.dom_content_loaded_event_start)) {
-    NOTREACHED() << "Invalid dom_loading " << timing.dom_loading
+    NOTREACHED() << "Invalid parse_stop " << timing.parse_stop
                  << " for dom_content_loaded_event_start "
                  << timing.dom_content_loaded_event_start;
     return false;
   }
 
   if (!EventsInOrder(timing.dom_content_loaded_event_start,
                      timing.load_event_start)) {
     NOTREACHED() << "Invalid dom_content_loaded_event_start "
                  << timing.dom_content_loaded_event_start
                  << " for load_event_start " << timing.load_event_start;
     return false;
   }
 
-  if (!EventsInOrder(timing.dom_loading, timing.first_layout)) {
-    NOTREACHED() << "Invalid dom_loading " << timing.dom_loading
+  if (!EventsInOrder(timing.parse_start, timing.first_layout)) {
+    NOTREACHED() << "Invalid parse_start " << timing.parse_start
                  << " for first_layout " << timing.first_layout;
     return false;
   }
 
   if (!EventsInOrder(timing.first_layout, timing.first_paint)) {
     // This can happen when we process an XHTML document that doesn't contain
     // well formed XML. See crbug.com/627607.
     DLOG(ERROR) << "Invalid first_layout " << timing.first_layout
                 << " for first_paint " << timing.first_paint;
     return false;
@@ skipping 818 matching lines @@
     if (navigation_handle->IsSamePage() || navigation_handle->IsErrorPage())
       return false;
     const std::string& mime_type = web_contents()->GetContentsMimeType();
     if (mime_type != "text/html" && mime_type != "application/xhtml+xml")
       return false;
   }
   return true;
 }
 
 }  // namespace page_load_metrics