| Index: chrome/browser/subresource_filter/subresource_filter_browsertest.cc
|
| diff --git a/chrome/browser/subresource_filter/subresource_filter_browsertest.cc b/chrome/browser/subresource_filter/subresource_filter_browsertest.cc
|
| index 38c6a7f5f14a83b040cca42d4bc314c139f9f659..e3096adadf462e9b35cbf2e43479bfee257360d3 100644
|
| --- a/chrome/browser/subresource_filter/subresource_filter_browsertest.cc
|
| +++ b/chrome/browser/subresource_filter/subresource_filter_browsertest.cc
|
| @@ -698,9 +698,12 @@ void ExpectHistogramsAreRecordedForTestFrameSet(
|
| } // namespace
|
|
|
| IN_PROC_BROWSER_TEST_F(SubresourceFilterBrowserTest,
|
| - ExpectHistogramsAreRecorded) {
|
| + ExpectHistogramsAreRecordedForFilteredChildFrames) {
|
| ASSERT_NO_FATAL_FAILURE(
|
| SetRulesetToDisallowURLsWithPathSuffix("included_script.js"));
|
| +
|
| + // Navigate to a URL which doesn't include any filtered resources in the
|
| + // top-level document, but which includes filtered resources in child frames.
|
| const GURL url = GetTestUrl(kTestFrameSetPath);
|
| ConfigureAsPhishingURL(url);
|
|
|
| @@ -709,6 +712,14 @@ IN_PROC_BROWSER_TEST_F(SubresourceFilterBrowserTest,
|
|
|
| ExpectHistogramsAreRecordedForTestFrameSet(
|
| tester, false /* expect_performance_measurements */);
|
| +
|
| + // Force a navigation to another page, which flushes page load metrics for the
|
| + // previous page load.
|
| + ui_test_utils::NavigateToURL(browser(), GURL(url::kAboutBlankURL));
|
| +
|
| + // Make sure that pages that have subresource filtered in child frames are
|
| + // also counted.
|
| + tester.ExpectTotalCount(internal::kHistogramSubresourceFilterCount, 1);
|
| }
|
|
|
| IN_PROC_BROWSER_TEST_F(SubresourceFilterWithPerformanceMeasurementBrowserTest,
|
|
|