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

Unified Diff: LayoutTests/fast/repaint/event-while-loading.html

Issue 14859016: Don't force layout for mouse event hit tests (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Add a test and check didFirstLayout() instead of shouldScheduleLayout(). And don't force layouts wh… Created 7 years, 7 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: LayoutTests/fast/repaint/event-while-loading.html
diff --git a/LayoutTests/fast/repaint/event-while-loading.html b/LayoutTests/fast/repaint/event-while-loading.html
new file mode 100644
index 0000000000000000000000000000000000000000..712161bad2d1c43214741c72fc2602abe4392532
--- /dev/null
+++ b/LayoutTests/fast/repaint/event-while-loading.html
@@ -0,0 +1,27 @@
+<!DOCTYPE html>
+<script>
+ var gotMouseEvent = false;
+ function mouseHandler() { gotMouseEvent = true; }
+ document.documentElement.addEventListener('click', mouseHandler, false);
+
+ if (window.testRunner && window.internals) {
+ window.testRunner.dumpAsText(false);
+
+ window.internals.startTrackingRepaints(document);
+ if (window.eventSender) {
+ window.eventSender.mouseMoveTo(1, 1);
+ window.eventSender.mouseDown();
+ window.eventSender.mouseUp();
+ }
+ var repaintRects = window.internals.repaintRectsAsText(document);
+ window.internals.stopTrackingRepaints(document);
+
+ window.onload = function() {
+ var pre = document.createElement('pre');
+ document.body.appendChild(pre);
+ pre.innerHTML = repaintRects;
ojan 2013/05/14 01:55:22 I think you also meant to check !gotMouseEvent her
cbiesinger 2013/05/14 18:09:56 Good catch - done
+ }
+ }
+</script>
+<body>
+
« no previous file with comments | « no previous file | LayoutTests/fast/repaint/event-while-loading-expected.txt » ('j') | Source/core/page/EventHandler.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698