Remove the blocking touch handlers for the input[type=range] and add touch-action instead

third_party/WebKit/Source/core/html/forms/RangeInputType.cpp

Index: third_party/WebKit/Source/core/html/forms/RangeInputType.cpp
diff --git a/third_party/WebKit/Source/core/html/forms/RangeInputType.cpp b/third_party/WebKit/Source/core/html/forms/RangeInputType.cpp
index e91b6744e21b01b7d9fde5abab14d158326bc22a..53145075ae359ac8311e71bddd980fd77f6ca83d 100644
--- a/third_party/WebKit/Source/core/html/forms/RangeInputType.cpp
+++ b/third_party/WebKit/Source/core/html/forms/RangeInputType.cpp
@@ -186,9 +186,9 @@ void RangeInputType::handleTouchEvent(TouchEvent* event)
     }
 }
 
-bool RangeInputType::hasTouchEventHandler() const
+TouchHandlerType RangeInputType::hasTouchEventHandler() const
 {
-    return true;
+    return Passive;
 }
 
 void RangeInputType::handleKeydownEvent(KeyboardEvent* event)
@@ -259,6 +259,7 @@ void RangeInputType::createShadowSubtree()
     HTMLDivElement* track = HTMLDivElement::create(document);
     track->setShadowPseudoId(AtomicString("-webkit-slider-runnable-track"));
     track->setAttribute(idAttr, ShadowElementNames::sliderTrack());
+    track->setAttribute(styleAttr, "pan-y pinch-zoom");

[majidvp, 2016/08/03 18:01:06]

Using the style attribute seems reasonable.

However I have a few notes about the attributes used: 
1- pinch-zoom: This one is still is not in the spec. I am not against using it
but chrome
currently does not support it fully [1], [2].

2- what about double-tap-zoom, similar to (1) it is not in spec but I think we
should allow it.

3- What happens if the slider reaches its extent? should we in that case allow
the user agent to
scroll in that direction? There is an interesting example of doing this in the
pointer-events spec
which I think applies here. [3]


[1] http://crbug.com/626086 
[2] https://github.com/w3c/pointerevents/issues/29 
[3] https://w3c.github.io/pointerevents/#the-touch-action-css-property

     track->appendChild(SliderThumbElement::create(document));
     HTMLElement* container = SliderContainerElement::create(document);
     container->appendChild(track);