<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - New force-related DOM events should fire in WK1 views"
href="https://bugs.webkit.org/show_bug.cgi?id=144663#c6">Comment # 6</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - New force-related DOM events should fire in WK1 views"
href="https://bugs.webkit.org/show_bug.cgi?id=144663">bug 144663</a>
from <span class="vcard"><a class="email" href="mailto:bdakin@apple.com" title="Beth Dakin <bdakin@apple.com>"> <span class="fn">Beth Dakin</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=144663#c5">comment #5</a>)
<span class="quote">> Comment on <span class=""><a href="attachment.cgi?id=252527&action=diff" name="attach_252527" title="Patch">attachment 252527</a> <a href="attachment.cgi?id=252527&action=edit" title="Patch">[details]</a></span>
> Patch
>
> View in context:
> <a href="https://bugs.webkit.org/attachment.cgi?id=252527&action=review">https://bugs.webkit.org/attachment.cgi?id=252527&action=review</a>
>
> > Source/WebCore/page/mac/EventHandlerMac.mm:86
> > + static NeverDestroyed<RetainPtr<NSEvent>> event;
>
> Is it really right for this to be stored between views? That seems weird.
> </span >
Hmm, maybe not. I was copying what's done for currentNSEventSlot(). I think it makes as much sense in this new case as it did in the old, but I'm a little unsure here.
<span class="quote">> > Source/WebCore/platform/mac/PlatformEventFactoryMac.mm:428
> > + if ([event type] == NSEventTypePressure) {
>
> You check this three times in this function. Maybe put it in a local?
> </span >
Sure, I can fix this up.
<span class="quote">> > Source/WebCore/platform/mac/PlatformEventFactoryMac.mm:439
> > + UNUSED_PARAM(lastPressureEvent);
>
> Indentation is wrong I think.
> </span >
Will look.
<span class="quote">> > Source/WebKit/mac/WebView/WebHTMLView.mm:-3844
> > - [[[self _webView] _immediateActionController] webView:[self _webView] willHandleMouseDown:event];
>
> Whaaaaaa?
> </span >
This is because mouseDown used to come after the immediate action! Remember? We used to have shenanigans like this in WK2 as well, and we were able to remove them when we switched to delaysPrimaryMouseEvents:NO. This code was there purely because we were getting mouse down at a weird time.
<span class="quote">> > Source/WebKit/mac/WebView/WebViewData.h:178
> > + NSEvent *pressureEvent;
>
> Maybe lastPressureEvent?
> </span >
Sure.
<span class="quote">> Could you not keep this in EventHandler instead and avoid keeping it here?</span >
Hmmmmmmm. I'll have to think this over. This is a closer mirror to what we do in WK2. If we moved it to EventHandler, we would have WK1 only member variables there, which we actually manage to avoid at this point.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>