Bug 220933 - [iOS] improve support for sequential scribble interactions that involving different inputs
Summary: [iOS] improve support for sequential scribble interactions that involving dif...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Devin Rousso
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-01-25 10:57 PST by Devin Rousso
Modified: 2021-01-25 16:36 PST (History)
3 users (show)

See Also:


Attachments
Patch (5.49 KB, patch)
2021-01-25 10:58 PST, Devin Rousso
no flags Details | Formatted Diff | Diff
Patch (6.00 KB, patch)
2021-01-25 11:45 PST, Devin Rousso
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Devin Rousso 2021-01-25 10:57:40 PST
.
Comment 1 Devin Rousso 2021-01-25 10:58:07 PST
<rdar://problem/71911346>
Comment 2 Devin Rousso 2021-01-25 10:58:49 PST
Created attachment 418308 [details]
Patch
Comment 3 Tim Horton 2021-01-25 11:39:22 PST
Comment on attachment 418308 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=418308&action=review

> Source/WebKit/UIProcess/ios/WKContentViewInteraction.h:399
> +    uint64_t _activeTextInteractionCount;

That's a lot of active text interactions!

> Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:5351
> +    --_activeTextInteractionCount;

How sure are you this will never underflow? Maybe an assertion at a minimum?
Comment 4 Devin Rousso 2021-01-25 11:42:45 PST
Comment on attachment 418308 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=418308&action=review

>> Source/WebKit/UIProcess/ios/WKContentViewInteraction.h:399
>> +    uint64_t _activeTextInteractionCount;
> 
> That's a lot of active text interactions!

/me attempts to see how many interactions he could have at the same time

Aside: I guess I could make it an `NSInteger` instead to match other member variables 🤔

>> Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:5351
>> +    --_activeTextInteractionCount;
> 
> How sure are you this will never underflow? Maybe an assertion at a minimum?

Good idea.
Comment 5 Devin Rousso 2021-01-25 11:45:34 PST
Created attachment 418316 [details]
Patch
Comment 6 EWS 2021-01-25 16:36:37 PST
Committed r271868: <https://trac.webkit.org/changeset/271868>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 418316 [details].