[PageLoadMetrics] Relax invariants and log the exceptions

chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer_unittest.cc

 // Copyright 2017 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/ads_page_load_metrics_observer.h"
 
 #include <string>
 
 #include "base/macros.h"
 #include "base/test/histogram_tester.h"
@@ skipping 223 matching lines @@
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.PerFrame.PercentNetwork", 0,
       1);
   histogram_tester().ExpectBucketCount(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.PerFrame.PercentNetwork", 100,
       2);
   histogram_tester().ExpectBucketCount(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.PerFrame.PercentNetwork", 50,
       1);
 
   // Counts
+  histogram_tester().ExpectBucketCount(
+      "PageLoad.Clients.Ads.Google.ParentExistsForSubFrame", 0, 0);
+  histogram_tester().ExpectTotalCount(
+      "PageLoad.Clients.Ads.Google.ResourceTypeWhenNoFrameFound", 0);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.FrameCounts.AnyParentFrame.AdFrames", 4, 1);
 
   // Page percentages
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.FullPage.Total.PercentAds", 71, 1);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.Aggregate.PercentNetwork", 60,
       1);
   histogram_tester().ExpectUniqueSample(
@@ skipping 70 matching lines @@
   // Individual Ad Frame Metrics
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.PerFrame.Total", 20, 1);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.PerFrame.Network", 20, 1);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.PerFrame.PercentNetwork", 100,
       1);
 
   // Counts
+  histogram_tester().ExpectBucketCount(
+      "PageLoad.Clients.Ads.Google.ParentExistsForSubFrame", 0, 0);
+  histogram_tester().ExpectTotalCount(
+      "PageLoad.Clients.Ads.Google.ResourceTypeWhenNoFrameFound", 0);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.FrameCounts.AnyParentFrame.AdFrames", 1, 1);
 
   // Page percentages
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.FullPage.Total.PercentAds", 66, 1);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.Aggregate.PercentNetwork",
       100, 1);
   histogram_tester().ExpectUniqueSample(
@@ skipping 42 matching lines @@
   // Individual Ad Frame Metrics
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.PerFrame.Total", 10, 2);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.PerFrame.Network", 10, 2);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.PerFrame.PercentNetwork", 100,
       2);
 
   // Counts
+  histogram_tester().ExpectBucketCount(
+      "PageLoad.Clients.Ads.Google.ParentExistsForSubFrame", 0, 0);
+  histogram_tester().ExpectTotalCount(
+      "PageLoad.Clients.Ads.Google.ResourceTypeWhenNoFrameFound", 0);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.FrameCounts.AnyParentFrame.AdFrames", 2, 1);
 
   // Page percentages
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.FullPage.Total.PercentAds", 50, 1);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.Aggregate.PercentNetwork",
       100, 1);
   histogram_tester().ExpectUniqueSample(
@@ skipping 113 matching lines @@
   // Individual Ad Frame Metrics
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.PerFrame.Total", 20, 1);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.PerFrame.Network", 20, 1);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.PerFrame.PercentNetwork", 100,
       1);
 
   // Counts
+  histogram_tester().ExpectBucketCount(
+      "PageLoad.Clients.Ads.Google.ParentExistsForSubFrame", 0, 0);
+  histogram_tester().ExpectUniqueSample(
+      "PageLoad.Clients.Ads.Google.ResourceTypeWhenNoFrameFound",
+      content::RESOURCE_TYPE_SUB_FRAME, 1);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.FrameCounts.AnyParentFrame.AdFrames", 1, 1);
 
   // Page percentages
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.FullPage.Total.PercentAds", 66, 1);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.Aggregate.PercentNetwork",
       100, 1);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.FullPage.Network.PercentAds", 66, 1);
 
   // Page byte counts
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.Aggregate.Total", 20, 1);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.AdFrames.Aggregate.Network", 20, 1);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.FullPage.Total", 30, 1);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.FullPage.Network", 30, 1);
   histogram_tester().ExpectUniqueSample(
       "PageLoad.Clients.Ads.Google.Bytes.NonAdFrames.Aggregate.Total", 10, 1);
 }
+
+// This tests an issue that is believed to be the cause of
+// https://crbug.com/721369. The issue is that a frame from a previous
+// navigation might commit during a new navigation, and the ads metrics won't
+// know about the frame's parent (because it doesn't exist in the page).
+TEST_F(AdsPageLoadMetricsObserverTest, FrameWithNoParent) {
+  RenderFrameHost* main_frame = NavigateMainFrame(kNonAdUrl);
+  RenderFrameHost* sub_frame =
+      CreateAndNavigateSubFrame(kNonAdUrl, kNonAdName, main_frame);
+
+  histogram_tester().ExpectBucketCount(
+      "PageLoad.Clients.Ads.Google.ParentExistsForSubFrame", 0, 0);
+
+  // Renavigate the child, but, while navigating, the main frame renavigates.
+  RenderFrameHost* child_of_subframe =
+      RenderFrameHostTester::For(sub_frame)->AppendChild(kAdName);
+  auto navigation_simulator = NavigationSimulator::CreateRendererInitiated(
+      GURL(kNonAdUrl2), child_of_subframe);
+  navigation_simulator->Start();
+
+  // Main frame renavigates.
+  NavigateMainFrame(kNonAdUrl);
+
+  // Child frame commits.
+  navigation_simulator->Commit();
+  child_of_subframe = navigation_simulator->GetFinalRenderFrameHost();
+  histogram_tester().ExpectBucketCount(
+      "PageLoad.Clients.Ads.Google.ParentExistsForSubFrame", 0, 1);
+
+  // Test that a resource loaded into an unknown frame doesn't cause any
+  // issues.
+  histogram_tester().ExpectTotalCount(
+      "PageLoad.Clients.Ads.Google.ResourceTypeWhenNoFrameFound", 0);
+  LoadResource(child_of_subframe, ResourceCached::NOT_CACHED, 10);
+  histogram_tester().ExpectBucketCount(
+      "PageLoad.Clients.Ads.Google.ResourceTypeWhenNoFrameFound",
+      content::RESOURCE_TYPE_SUB_FRAME, 1);
+}