[Prerender] first contentful paint histograms.

chrome/test/BUILD.gn

Index: chrome/test/BUILD.gn
diff --git a/chrome/test/BUILD.gn b/chrome/test/BUILD.gn
index f2a022fbb68294609011346284b96cc1740c68e5..cf4164888df8b1fb5e70a7e07dc017ffd6f6d0a0 100644
--- a/chrome/test/BUILD.gn
+++ b/chrome/test/BUILD.gn
@@ -1628,6 +1628,7 @@ test("browser_tests") {
       "../browser/net/spdyproxy/chrome_data_use_group_browsertest.cc",
       "../browser/net/websocket_browsertest.cc",
       "../browser/page_load_metrics/observers/https_engagement_metrics/https_engagement_page_load_metrics_observer_browsertest.cc",
+      "../browser/page_load_metrics/observers/page_load_metrics_observer_test_harness.cc",

[pasko, 2016/11/17 19:52:16]

this file is now referenced in 2 places, can it be moved to some common library
instead? like "page_load_metrics_test_support"?

[mattcary, 2016/11/18 09:21:11]

I have no idea, the build file is not very comprehensible.

I could put it into :test_support directly. 

I could make a new page_load_metrics_test_support which I could then include in
browser_tests and unit_tests. Could this new library could reasonably go in
either browser/BUILD.gn or tests/BUILD.gn?

I could put this new page_load_metrics_test_support library either as a
dependence of test_support, or put it as dependencies of unit_tests and
browser_tests.

There are probably some other options as well...

At any rate, point me in the right direction and I'll implement it.

       "../browser/page_load_metrics/page_load_metrics_browsertest.cc",
       "../browser/password_manager/credential_manager_browsertest.cc",
       "../browser/password_manager/password_manager_browsertest.cc",