Move the user interaction policy for FirstMeaningfulPaint UMA into renderer

third_party/WebKit/Source/core/paint/FirstMeaningfulPaintDetectorTest.cpp

Index: third_party/WebKit/Source/core/paint/FirstMeaningfulPaintDetectorTest.cpp
diff --git a/third_party/WebKit/Source/core/paint/FirstMeaningfulPaintDetectorTest.cpp b/third_party/WebKit/Source/core/paint/FirstMeaningfulPaintDetectorTest.cpp
index 1eedaf29c68fd9c46c27f40fb557d90bca7c62d4..5a75fc9ecdb2ae825e35f2e714e00cdea51e8d31 100644
--- a/third_party/WebKit/Source/core/paint/FirstMeaningfulPaintDetectorTest.cpp
+++ b/third_party/WebKit/Source/core/paint/FirstMeaningfulPaintDetectorTest.cpp
@@ -56,6 +56,8 @@ class FirstMeaningfulPaintDetectorTest : public testing::Test {
     Detector().Network2QuietTimerFired(nullptr);
   }
 
+  void SimulateUserInput() { Detector().NotifyInputEvent(); }
+
   void SetActiveConnections(int connections) {
     Detector().SetNetworkQuietTimers(connections);
   }
@@ -249,4 +251,13 @@ TEST_F(FirstMeaningfulPaintDetectorTest, Network2QuietTimer) {
   EXPECT_TRUE(HadNetwork2Quiet());
 }
 
+TEST_F(FirstMeaningfulPaintDetectorTest,
+       FirstMeaningfulPaintAfterUserInteraction) {
+  GetPaintTiming().MarkFirstContentfulPaint();
+  SimulateUserInput();
+  SimulateLayoutAndPaint(10);
+  SimulateNetworkStable();
+  EXPECT_EQ(GetPaintTiming().FirstMeaningfulPaint(), 0.0);
+}

[tdresser, 2017/06/07 12:32:54]

Should we have a test showing that we ignore input before FP?

[Kunihiko Sakamoto, 2017/06/08 07:54:49]

Done.

+
 }  // namespace blink