...
<rdar://problem/76785859>
Created attachment 426505 [details] Patch
Created attachment 426508 [details] Patch
Comment on attachment 426508 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=426508&action=review > Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:8382 > + RetainPtr<UITargetedPreview> targetedPreview = createFallbackTargetedPreview(self, self.containerForContextMenuHintPreviews, _focusedElementInformation.interactionRect, backgroundColor); Nit - `auto`. > Source/WebKit/UIProcess/ios/forms/WKDateTimeInputControl.mm:71 > +- (instancetype)initWithDelegate:(id<WKDateTimePickerViewControllerDelegate>)delegate Nit - space between `id` and `<`. (AFAIK, the rule we typically follow is space before protocols, but no space before concrete types like in `NSArray<Foo *> *`). > Source/WebKit/UIProcess/ios/forms/WKDateTimeInputControl.mm:94 > + RetainPtr<UIVisualEffectView> backgroundView = adoptNS([[UIVisualEffectView alloc] initWithEffect:[UIBlurEffect effectWithStyle:UIBlurEffectStyleSystemMaterial]]); Nit - `auto` > Source/WebKit/UIProcess/ios/forms/WKDateTimeInputControl.mm:110 > + RetainPtr<UIView> toolbarView = adoptNS([[UIView alloc] init]); Nit - `auto` > Source/WebKit/UIProcess/ios/forms/WKDateTimeInputControl.mm:121 > + RetainPtr<UIView> separatorView = adoptNS([[UIView alloc] init]); (Ditto) > Source/WebKit/UIProcess/ios/forms/WKDateTimeInputControl.mm:524 > + [_view updateFocusedElementValueAsNumber:([date timeIntervalSince1970] + [self _timeZoneOffsetFromGMT:date]) * kMillisecondsPerSecond]; Nit - `date.timeIntervalSince1970`
Created attachment 426597 [details] Patch for landing
Committed r276325 (236805@main): <https://commits.webkit.org/236805@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 426597 [details].