Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(86)

Unified Diff: third_party/WebKit/Source/web/WebViewImpl.cpp

Issue 2883273003: Move the user interaction policy for FirstMeaningfulPaint UMA into renderer (Closed)
Patch Set: crash fix Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/web/WebViewImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebViewImpl.cpp b/third_party/WebKit/Source/web/WebViewImpl.cpp
index 551f67011420acccf0890636f865eaa8f557914d..ab501b28ec3d3e90bf347393bfa8063b34f91ec6 100644
--- a/third_party/WebKit/Source/web/WebViewImpl.cpp
+++ b/third_party/WebKit/Source/web/WebViewImpl.cpp
@@ -95,6 +95,7 @@
#include "core/page/TouchDisambiguation.h"
#include "core/page/ValidationMessageClientImpl.h"
#include "core/page/scrolling/TopDocumentRootScrollerController.h"
+#include "core/paint/FirstMeaningfulPaintDetector.h"
#include "core/paint/LinkHighlightImpl.h"
#include "core/paint/PaintLayer.h"
#include "core/timing/DOMWindowPerformance.h"
@@ -2175,6 +2176,12 @@ WebInputEventResult WebViewImpl::HandleInputEvent(
return WebInputEventResult::kHandledSystem;
}
+ if (input_event.GetType() != WebInputEvent::kMouseMove) {
+ FirstMeaningfulPaintDetector::From(
+ *MainFrameImpl()->GetFrame()->GetDocument())
+ .NotifyInputEvent();
+ }
+
if (mouse_capture_node_ &&
WebInputEvent::IsMouseEventType(input_event.GetType())) {
TRACE_EVENT1("input", "captured mouse event", "type",

Powered by Google App Engine
This is Rietveld 408576698