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

Unified Diff: third_party/WebKit/LayoutTests/fast/events/pointerevents/mouse-pointer-event-properties.html

Issue 2296303002: Make a pen in eraser mode visible thru PointerEvent.buttons (Closed)
Patch Set: Incremental patch on crrev.com/2289273002 Created 4 years, 4 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/LayoutTests/fast/events/pointerevents/mouse-pointer-event-properties.html
diff --git a/third_party/WebKit/LayoutTests/fast/events/pointerevents/mouse-pointer-event-properties.html b/third_party/WebKit/LayoutTests/fast/events/pointerevents/mouse-pointer-event-properties.html
index dda1ac0c9cc8dd62c5f36dac8dc59d1673105407..c8a383b4af78c59589aa88f40536f0424e5068e9 100644
--- a/third_party/WebKit/LayoutTests/fast/events/pointerevents/mouse-pointer-event-properties.html
+++ b/third_party/WebKit/LayoutTests/fast/events/pointerevents/mouse-pointer-event-properties.html
@@ -20,6 +20,8 @@ description("Verifies that pointer event parameters are correct when fired throu
var checkKeyModifiers = false;
+var inputPointerType = "";
+
var pointerType = "";
var penId = 0;
var penPressure = 0;
@@ -54,6 +56,31 @@ var numericAttributes = [
"height",
];
+function getExpectedNumericAttributeValueForPE(mouseEvent, attribute) {
+ var expectedValue = eval("mouseEvent." + attribute);
+
+ // Fix expectedValue for the cases where PE & ME differs
+ if (attribute == "button") {
+ if (mouseEvent.type != "mousedown" && mouseEvent.type != "mouseup")
+ expectedValue = -1;
+ } else if (attribute == "buttons") {
+ if (inputPointerType == "eraser" && mouseEvent.type == "mousedown")
+ expectedValue |= 32;
+ } else if (attribute == "width" || attribute == "height") {
+ expectedValue = 1;
+ } else if (attribute == "pressure") {
+ if (pointerType == "mouse")
+ expectedValue = (mouseEvent.buttons == 0)? 0.0 : 0.5;
+ else
+ expectedValue = penPressure;
+ } else if (attribute == "tiltX") {
+ expectedValue = (pointerType == "mouse")? 0 : penTiltX;
+ } else if (attribute == "tiltY") {
+ expectedValue = (pointerType == "mouse")? 0 : penTiltY;
+ }
+ return expectedValue;
+}
+
function init() {
var targetDiv = document.getElementById("target");
@@ -89,24 +116,10 @@ function init() {
shouldBeTrue("lastPointerEvents[0].isPrimary");
numericAttributes.forEach(function(attribute) {
- var expectedValue = eval("event." + attribute);
- if (attribute == "button" && event.type != "mousedown" && event.type != "mouseup")
- expectedValue = -1;
- else if (attribute == "width" || attribute == "height")
- expectedValue = 1;
- else if (attribute == "pressure") {
- if (pointerType == "mouse")
- expectedValue = (event.buttons == 0)? 0.0 : 0.5;
- else
- expectedValue = penPressure;
- }
- else if (attribute == "tiltX")
- expectedValue = (pointerType == "mouse")? 0 : penTiltX;
- else if (attribute == "tiltY")
- expectedValue = (pointerType == "mouse")? 0 : penTiltY;
-
+ var expectedValue = getExpectedNumericAttributeValueForPE(event, attribute);
shouldBeEqualToNumber("lastPointerEvents[0]." + attribute, expectedValue);
});
+
shouldBeEqualToString("lastPointerEvents[0].view.name", "mainWindow");
} else {
@@ -164,29 +177,30 @@ function runMouseTests(x, y) {
debug("");
}
-function runPenTests(x, y) {
- debug("===== pen tests =====");
+function runPenTests(x, y, id, eraseMode) {
+ debug("===== pen tests " + (eraseMode? "(erase mode)" : "(draw mode)") + " =====");
pointerType = "pen";
- penId = 2;
+ inputPointerType = eraseMode? "eraser" : "pen";
+ penId = id;
penPressure = 0.0;
penTiltX = 0;
penTiltY = 0;
debug("--- move pen into target ---");
- eventSender.mouseMoveTo(x + 5, y + 5, [], "pen", penId, penPressure, penTiltX, penTiltY);
+ eventSender.mouseMoveTo(x + 5, y + 5, [], inputPointerType, penId, penPressure, penTiltX, penTiltY);
debug("");
debug("--- move within target & tap ---");
penTiltX = 45;
penTiltY = -34;
- eventSender.mouseMoveTo(x + 15, y + 15, [], "pen", penId, penPressure, penTiltX, penTiltY);
+ eventSender.mouseMoveTo(x + 15, y + 15, [], inputPointerType, penId, penPressure, penTiltX, penTiltY);
penPressure = 0.75;
- eventSender.mouseDown(0, [], "pen", penId, penPressure, penTiltX, penTiltY);
+ eventSender.mouseDown(0, [], inputPointerType, penId, penPressure, penTiltX, penTiltY);
penPressure = 0.0;
- eventSender.mouseUp(0, [], "pen", penId, penPressure, penTiltX, penTiltY);
+ eventSender.mouseUp(0, [], inputPointerType, penId, penPressure, penTiltX, penTiltY);
debug("--- move pen out of target ---");
- eventSender.mouseMoveTo(x - 5, y - 5, [], "pen", penId, penPressure, penTiltX, penTiltY);
+ eventSender.mouseMoveTo(x - 5, y - 5, [], inputPointerType, penId, penPressure, penTiltX, penTiltY);
debug("");
}
@@ -195,7 +209,8 @@ function runAllTests() {
var rect = document.getElementById("target").getBoundingClientRect();
runMouseTests(rect.left, rect.top);
- runPenTests(rect.left, rect.top);
+ runPenTests(rect.left, rect.top, 2, false);
+ runPenTests(rect.left, rect.top, 3, true);
}
init();

Powered by Google App Engine
This is Rietveld 408576698