Chromium Code Reviews Chromium Code Reviews
Issue 1895983002:
Adding UMA metrics to track usage of pointerevents. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: third_party/WebKit/LayoutTests/virtual/pointerevent/fast/events/pointerevents/pointer-use-count.html |
| diff --git a/third_party/WebKit/LayoutTests/virtual/pointerevent/fast/events/pointerevents/pointer-use-count.html b/third_party/WebKit/LayoutTests/virtual/pointerevent/fast/events/pointerevents/pointer-use-count.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..65e284520943b57d74e6d5fab71a6a2a8d2aacf8 |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/virtual/pointerevent/fast/events/pointerevents/pointer-use-count.html |
| @@ -0,0 +1,78 @@ |
| +<!DOCTYPE HTML> |
| +<script src="../../../../../resources/js-test.js"></script> |
| +<style> |
| +div.box { |
| + margin: 5px; |
| + padding: 20px; |
| + float: left; |
| +} |
| +#grey { |
| + width: 50px; |
| + height: 50px; |
| +} |
| +</style> |
| + |
| +<div id="grey" class="box" style="background-color:grey"> |
| +</div> |
| + |
| +<div id="console"></div> |
| + |
| +<script> |
| +description("Verifies that use of pointerEvents is measured by use counters."); |
| + |
| +var PointerEventCount = 1306; |
| + |
| +var rect = document.getElementById("grey").getBoundingClientRect(); |
| +var x1 = rect.left + 5; |
| +var y1 = rect.top + 5; |
| + |
| + |
| +function init() { |
| + var eventList = ["pointerenter", "pointerleave", "pointerover", "pointerout", "pointermove", "pointerdown", "pointerup", |
| + "gotpointercapture", "lostpointercapture"]; |
|
tdresser
2016/04/19 12:43:20
I don't think we need to test every event type for
sahel
2016/04/19 17:57:06
I omitted some of the events from the list.
|
| + |
| + document.addEventListener("lostpointercapture", function(event) { |
| + if (event.eventPhase == Event.AT_TARGET) { |
| + debug("document received " + event.type); |
| + } |
| + }); |
|
tdresser
2016/04/19 12:43:20
We don't need to explicitly test behavior of "lost
sahel
2016/04/19 17:57:06
Done.
|
| + ["grey"].forEach(function(id) { |
|
tdresser
2016/04/19 12:43:20
This forEach looks unnecessary.
sahel
2016/04/19 17:57:06
Done.
|
| + var targetDiv = document.getElementById(id); |
| + eventList.forEach(function(eventName) { |
| + targetDiv.addEventListener(eventName, function(event) { |
| + if (event.eventPhase == Event.AT_TARGET) { |
| + debug(id + " received " + event.type); |
| + if (event.type == "pointerdown") { |
| + debug(" **** Set Pointer Capture ***** "); |
| + targetDiv.setPointerCapture(event.pointerId); |
|
tdresser
2016/04/19 12:43:20
We don't need to test pointer capture logic in thi
sahel
2016/04/19 17:57:06
No setPointerCapture anymore, just accessing point
|
| + } |
| + } |
| + }); |
| + }); |
| + }); |
| +} |
| + |
| +function runTest() { |
| + debug(" **** Move to grey box & mouse press & jiggle ***** "); |
| + eventSender.mouseMoveTo(x1, y1); |
| + eventSender.mouseDown(1); |
| + eventSender.mouseMoveTo(x1+1, y1+1); |
| + |
| + debug(" **** Move to (0,0) ***** "); |
| + eventSender.mouseMoveTo(0, 0); |
| + |
| + debug(" **** Move back to grey & mouse release ***** "); |
| + eventSender.mouseMoveTo(x1, y1); |
| + eventSender.mouseUp(1); |
| + |
| + debug(" **** Print PointerEventCount usage ***** "); |
| + debug(internals.isUseCounted(document, PointerEventCount)); |
| +} |
| + |
| +init(); |
| +if (window.eventSender) |
| + runTest(); |
| +else |
| + debug("This test requires eventSender"); |
| + |
| +</script> |
