Changeset 207823 in webkit

Timestamp:

Oct 25, 2016, 9:34:32 AM (9 years ago)

Author:

Chris Dumez

Message:

Unreviewed, rolling out r207494.

iOS SDK has been updated on the bots now so rollout the
rollout

Reverted changeset:

"Unreviewed, rolling out r207413."
​https://bugs.webkit.org/show_bug.cgi?id=163547
​http://trac.webkit.org/changeset/207494

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• Platform/spi/ios/UIKitSPI.h (modified) (1 diff)
• Shared/ios/NativeWebTouchEventIOS.mm (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2016-10-25  Chris Dumez  <cdumez@apple.com>
+
+        Unreviewed, rolling out r207494.
+
+        iOS SDK has been updated on the bots now so rollout the
+        rollout
+
+        Reverted changeset:
+
+        "Unreviewed, rolling out r207413."
+        https://bugs.webkit.org/show_bug.cgi?id=163547
+        http://trac.webkit.org/changeset/207494
+
 2016-10-25  Jonathan Bedard  <jbedard@apple.com>
 

trunk/Source/WebKit2/Platform/spi/ios/UIKitSPI.h

@@ -655 +655 @@
     UITouchPhase phase;
 #if __IPHONE_OS_VERSION_MIN_REQUIRED > 100000
-    // FIXME: Uncomment this once UIKit is updated (rdar://problem/28555313).
-    // CGFloat majorRadiusInScreenCoordinates;
+    CGFloat majorRadiusInScreenCoordinates;
     CGFloat force;
     CGFloat altitudeAngle;

trunk/Source/WebKit2/Shared/ios/NativeWebTouchEventIOS.mm

@@ -104 +104 @@
         WebPlatformTouchPoint platformTouchPoint = WebPlatformTouchPoint(identifier, location, phase);
 #if ENABLE(IOS_TOUCH_EVENTS)
-        // FIXME: We should use touchPoint.majorRadiusInScreenCoordinates for radiusX / radiusY once it is available (rdar://problem/28555313).
-        platformTouchPoint.setRadiusX(0);
-        platformTouchPoint.setRadiusY(0);
+        platformTouchPoint.setRadiusX(touchPoint.majorRadiusInScreenCoordinates);
+        platformTouchPoint.setRadiusY(touchPoint.majorRadiusInScreenCoordinates);
         platformTouchPoint.setRotationAngle(0); // Not available in _UIWebTouchEvent yet.
         platformTouchPoint.setForce(touchPoint.force);