Update touch-action hit-testing to match latest spec changes.

Source/core/rendering/RenderObject.h

 /*
  * Copyright (C) 2000 Lars Knoll (knoll@kde.org)
  *           (C) 2000 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Dirk Mueller (mueller@kde.org)
  *           (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 929 matching lines @@
 
     virtual void imageChanged(ImageResource*, const IntRect* = 0) OVERRIDE FINAL;
     virtual void imageChanged(WrappedImagePtr, const IntRect* = 0) { }
     virtual bool willRenderImage(ImageResource*) OVERRIDE FINAL;
 
     void selectionStartEnd(int& spos, int& epos) const;
 
     void remove() { if (parent()) parent()->removeChild(this); }
 
     bool isInert() const;
-    virtual bool visibleForTouchAction() const { return false; }
+
+    // touch-action applies to all elements with both width AND height properties.
+    // According to the CSS Box Model Spec (http://dev.w3.org/csswg/css-box/#the-width-and-height-properties)
+    // width applies to all elements but non-replaced inline elements, table rows, and row groups and
+    // height applies to all elements but non-replaced inline elements, table columns, and column groups.
+    bool visibleForTouchAction() const

[ojan, 2014/04/09 22:20:13]

This function body is long for a header file. Can you move it to the cpp file? I
expect the compiler will inline it either way since it's only called in one
place.

+    {
+        if (isInline() && !isReplaced())

[ojan, 2014/04/09 22:20:13]

Makes me a little sad that we can't use inheritance for this, but looking at the
places we call setInline and setReplaced...it's a total mess that doesn't match
our inheritance hierarchy, so lgtm.

+            return false;
+        if (isTableRow() || isRenderTableCol())
+            return false;
+
+        return true;
+    }
+
     bool visibleToHitTestRequest(const HitTestRequest& request) const
     {
         if (request.touchAction() && !visibleForTouchAction())
             return false;
         return style()->visibility() == VISIBLE && (request.ignorePointerEventsNone() || style()->pointerEvents() != PE_NONE) && !isInert();
     }
 
     bool visibleToHitTesting() const { return style()->visibility() == VISIBLE && style()->pointerEvents() != PE_NONE && !isInert(); }
 
     // Map points and quads through elements, potentially via 3d transforms. You should never need to call these directly; use
@@ skipping 483 matching lines @@
 void showTree(const WebCore::RenderObject*);
 void showLineTree(const WebCore::RenderObject*);
 void showRenderTree(const WebCore::RenderObject* object1);
 // We don't make object2 an optional parameter so that showRenderTree
 // can be called from gdb easily.
 void showRenderTree(const WebCore::RenderObject* object1, const WebCore::RenderObject* object2);
 
 #endif
 
 #endif // RenderObject_h