Standardize which page loads are tracked

chrome/browser/page_load_metrics/observers/aborts_page_load_metrics_observer_unittest.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/observers/aborts_page_load_metrics_observer.h"
 
 #include "base/memory/ptr_util.h"
 #include "chrome/browser/page_load_metrics/observers/page_load_metrics_observer_test_harness.h"
 
 class AbortsPageLoadMetricsObserverTest
@@ skipping 11 matching lines @@
 };
 
 TEST_F(AbortsPageLoadMetricsObserverTest, UnknownNavigationBeforeCommit) {
   StartNavigation(GURL("https://www.google.com"));
   // Simulate the user performing another navigation before commit.
   NavigateAndCommit(GURL("https://www.example.com"));
   histogram_tester().ExpectTotalCount(
       internal::kHistogramAbortUnknownNavigationBeforeCommit, 1);
 }
 
+TEST_F(AbortsPageLoadMetricsObserverTest,
+       UnknownProvisionalNavigationBeforeCommit) {
+  StartNavigation(GURL("https://www.google.com"));
+  StartNavigation(GURL("https://www.example.com"));
+  histogram_tester().ExpectTotalCount(
+      internal::kHistogramAbortUnknownNavigationBeforeCommit, 1);
+}
+
+TEST_F(AbortsPageLoadMetricsObserverTest,
+       UnknownNavigationBeforeCommitNonTrackedPageLoad) {
+  StartNavigation(GURL("https://www.google.com"));
+  // Simulate the user performing another navigation before commit. Navigate to
+  // an untracked URL, to verify that we still log abort metrics even if the new
+  // navigation isn't tracked.
+  NavigateAndCommit(GURL("about:blank"));
+  histogram_tester().ExpectTotalCount(
+      internal::kHistogramAbortUnknownNavigationBeforeCommit, 1);
+}
+
 TEST_F(AbortsPageLoadMetricsObserverTest, NewNavigationBeforePaint) {
   NavigateAndCommit(GURL("https://www.google.com"));
   SimulateTimingWithoutPaint();
   // Simulate the user performing another navigation before paint.
   NavigateAndCommit(GURL("https://www.example.com"));
   histogram_tester().ExpectTotalCount(
       internal::kHistogramAbortNewNavigationBeforePaint, 1);
   histogram_tester().ExpectTotalCount(
       internal::kHistogramAbortReloadBeforePaint, 0);
   histogram_tester().ExpectTotalCount(
@@ skipping 150 matching lines @@
 
   NavigateAndCommit(GURL("https://www.example.com"));
 
   base::HistogramTester::CountsMap counts_map =
       histogram_tester().GetTotalCountsForPrefix(
           internal::kHistogramAbortNewNavigationBeforePaint);
 
   EXPECT_TRUE(counts_map.empty() ||
               (counts_map.size() == 1 && counts_map.begin()->second == 1));
 }