REGRESSION: fast/events/watchos/double-tap-to-zoom* tests crashing in com.apple.WebKit: -[WKContentView(WKInteraction) _singleTapDidReset:]
Created attachment 374254 [details] Patch
<rdar://problem/52856531>
Comment on attachment 374254 [details] Patch Looks find to me. Is there a reason to prefer this over making _lastActiveTouchIdentifier an ivar of type RetainPtr<NSNumber>, and then exposing a getter that returns _lastActiveTouchIdentifier.get()? I think this is how most of our ObjC properties in WebKit are implemented.
(In reply to Wenson Hsieh from comment #3) > Comment on attachment 374254 [details] > Patch > > Looks find to me. Is⦠s/find/fine/
Created attachment 374258 [details] Patch for landing
(In reply to Wenson Hsieh from comment #3) > Comment on attachment 374254 [details] > Patch > > Looks find to me. Is there a reason to prefer this over making > _lastActiveTouchIdentifier an ivar of type RetainPtr<NSNumber>, and then > exposing a getter that returns _lastActiveTouchIdentifier.get()? I think > this is how most of our ObjC properties in WebKit are implemented. No reason, I just wasn't aware. Thanks for bringing this up, I think I did what you suggested in the patch for landing. Let me know if it isn't quite right.
Created attachment 374259 [details] Patch for landing
Comment on attachment 374259 [details] Patch for landing Clearing flags on attachment: 374259 Committed r247502: <https://trac.webkit.org/changeset/247502>
All reviewed patches have been landed. Closing bug.
*** Bug 199911 has been marked as a duplicate of this bug. ***