Remove special touch-action hit test mode

Source/core/page/EventHandler.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2012 Digia Plc. and/or its subsidiary(-ies)
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
@@ skipping 3430 matching lines @@
             }
 
             // Ideally we'd ASSERT(!m_targetForTouchID.contains(point.id())
             // since we shouldn't get a touchstart for a touch that's already
             // down. However EventSender allows this to be violated and there's
             // some tests that take advantage of it. There may also be edge
             // cases in the browser where this happens.
             // See http://crbug.com/345372.
             m_targetForTouchID.set(point.id(), node);
 
-            TouchAction effectiveTouchAction = computeEffectiveTouchAction(pagePoint);
+            TouchAction effectiveTouchAction = computeEffectiveTouchAction(*node);
             if (effectiveTouchAction != TouchActionAuto)
                 m_frame->page()->chrome().client().setTouchAction(effectiveTouchAction);
         }
     }
 
     m_touchPressed = !allTouchReleased;
 
     // If there's no document receiving touch events, or no handlers on the
     // document set to receive the events, then we can skip all the rest of
     // this work.
@@ skipping 153 matching lines @@
         return TouchActionNone;
     if (action1 == TouchActionAuto)
         return action2;
     if (action2 == TouchActionAuto)
         return action1;
     if (!(action1 & action2))
         return TouchActionNone;
     return action1 & action2;
 }
 
-TouchAction EventHandler::computeEffectiveTouchAction(const LayoutPoint& point)
+TouchAction EventHandler::computeEffectiveTouchAction(const Node& node)
 {
     // Optimization to minimize risk of this new feature (behavior should be identical
     // since there's no way to get non-default touch-action values).
     if (!RuntimeEnabledFeatures::cssTouchActionEnabled())
         return TouchActionAuto;
 
-    HitTestResult taResult = hitTestResultAtPoint(point, HitTestRequest::TouchEvent | HitTestRequest::ReadOnly | HitTestRequest::Active | HitTestRequest::TouchAction);
-    Node* node = taResult.innerNode();
-    if (!node)
+    // if the hittest node doesn't support touchAction, then bailout immediately.
+    if (node.renderer() && !node.renderer()->supportsTouchAction())

[gnana, 2014/06/12 14:02:08]

This check is added for below testcase.

<div class='ta-none' style='height: 0; margin-bottom: 50px'>
  <span expected-action='auto'>
    touch-action should not be inherited by inline elements
  </span>
</div>

[Rick Byers, 2014/06/12 19:27:25]

This is the case we need to change the test for.  Remove this code (it'll break
common cases like inline text inside of a DIV that has touch-action applied).

This test case should be updated to say "SHOULD BE inherited by inline elements"
and I'd explicitly add something like "(note this is different than IE, see
http://crbug.com/380203)".

         return TouchActionAuto;
 
     // Start by permitting all actions, then walk the elements supporting
     // touch-action from the target node up to the nearest scrollable ancestor
     // and exclude any prohibited actions.
     TouchAction effectiveTouchAction = TouchActionAuto;
-    for (const Node* curNode = node; curNode; curNode = NodeRenderingTraversal::parent(curNode)) {
+    for (const Node* curNode = &node; curNode; curNode = NodeRenderingTraversal::parent(curNode)) {
         if (RenderObject* renderer = curNode->renderer()) {
-            if (renderer->visibleForTouchAction()) {
+            if (renderer->supportsTouchAction()) {
                 TouchAction action = renderer->style()->touchAction();
                 effectiveTouchAction = intersectTouchAction(action, effectiveTouchAction);
                 if (effectiveTouchAction == TouchActionNone)
                     break;
             }
 
             // If we've reached an ancestor that supports a touch action, search no further.
             if (renderer->isBox() && toRenderBox(renderer)->scrollsOverflow())
                 break;
         }
@@ skipping 79 matching lines @@
 unsigned EventHandler::accessKeyModifiers()
 {
 #if OS(MACOSX)
     return PlatformEvent::CtrlKey | PlatformEvent::AltKey;
 #else
     return PlatformEvent::AltKey;
 #endif
 }
 
 } // namespace WebCore