Fix clientX/Y properties of touch pointer events

third_party/WebKit/Source/core/input/TouchEventManager.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef TouchEventManager_h
 #define TouchEventManager_h
 
 #include "core/CoreExport.h"
 #include "core/events/PointerEventFactory.h"
 #include "platform/UserGestureIndicator.h"
@@ skipping 19 matching lines @@
     public:
         DEFINE_INLINE_TRACE()
         {
             visitor->trace(touchNode);
             visitor->trace(targetFrame);
         }
 
         PlatformTouchPoint point;
         Member<Node> touchNode;
         Member<LocalFrame> targetFrame;
-        FloatPoint adjustedPagePoint;
+        FloatPoint clientPoint;

[mustaq, 2016/05/09 14:49:05]

bokan@: "client" seems to be an overloaded term throughout our codebase. May be
this is not true for coordinates? Any chance we could be more specific here?

[Navid Zolghadr, 2016/05/09 17:01:43]

bokan@, do you think framePoint is better instead of clientPoint here or maybe
totally something else?

[bokan, 2016/05/09 17:17:47]

Client and Frame are interchangeable. In terms of coordinates its not overloaded
and is a necessity since lots of web facing APIs use clientX/Y so we have to
keep it in some places.

I don't have a strong preference either way. If this isn't web facing though,
framePoint is generally used for chrome internal data. If you do keep it
clientPoint, then you should change contentPoint below to pagePoint to match.

+        FloatPoint contentPoint;
         FloatSize adjustedRadius;
         bool knownTarget;
         bool consumed;
         String region;
     };
 
     explicit TouchEventManager(LocalFrame*);
     ~TouchEventManager();
     DECLARE_TRACE();
 
@@ skipping 42 matching lines @@
     // True if waiting on first touch move after a touch start.
     bool m_waitingForFirstTouchMove;
 
 };
 
 } // namespace blink
 
 WTF_ALLOW_INIT_WITH_MEM_FUNCTIONS(blink::TouchEventManager::TouchInfo);
 
 #endif // TouchEventManager_h