Title: [239121] tags/Safari-607.1.16.2/Source
- Revision
- 239121
- Author
- [email protected]
- Date
- 2018-12-12 10:52:59 -0800 (Wed, 12 Dec 2018)
Log Message
Cherry-pick r239088. rdar://problem/46615532
WebCore shouldn't have a Objective-C class named NSCursor
https://bugs.webkit.org/show_bug.cgi?id=192602
<rdar://problem/46615532>
Reviewed by Wenson Hsieh.
Source/WebCore:
* platform/ios/wak/WAKAppKitStubs.h:
* platform/ios/wak/WAKAppKitStubs.m:
(+[NSCursor setHiddenUntilMouseMoves:]): Deleted.
Get rid of the class.
Also remove a comment that seems to have detached from wherever it's supposed to be.
Source/WebKitLegacy/mac:
* WebView/WebFrameView.mm:
(-[WebFrameView keyDown:keyDown:]):
* WebView/WebHTMLView.mm:
(-[WebHTMLView keyDown:]):
(-[WebHTMLView performKeyEquivalent:]):
Only use NSCursor on macOS.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@239088 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: tags/Safari-607.1.16.2/Source/WebCore/ChangeLog (239120 => 239121)
--- tags/Safari-607.1.16.2/Source/WebCore/ChangeLog 2018-12-12 18:52:55 UTC (rev 239120)
+++ tags/Safari-607.1.16.2/Source/WebCore/ChangeLog 2018-12-12 18:52:59 UTC (rev 239121)
@@ -1,5 +1,49 @@
2018-12-12 Kocsen Chung <[email protected]>
+ Cherry-pick r239088. rdar://problem/46615532
+
+ WebCore shouldn't have a Objective-C class named NSCursor
+ https://bugs.webkit.org/show_bug.cgi?id=192602
+ <rdar://problem/46615532>
+
+ Reviewed by Wenson Hsieh.
+
+ Source/WebCore:
+
+ * platform/ios/wak/WAKAppKitStubs.h:
+ * platform/ios/wak/WAKAppKitStubs.m:
+ (+[NSCursor setHiddenUntilMouseMoves:]): Deleted.
+ Get rid of the class.
+ Also remove a comment that seems to have detached from wherever it's supposed to be.
+
+ Source/WebKitLegacy/mac:
+
+ * WebView/WebFrameView.mm:
+ (-[WebFrameView keyDown:keyDown:]):
+ * WebView/WebHTMLView.mm:
+ (-[WebHTMLView keyDown:]):
+ (-[WebHTMLView performKeyEquivalent:]):
+ Only use NSCursor on macOS.
+
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@239088 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2018-12-11 Tim Horton <[email protected]>
+
+ WebCore shouldn't have a Objective-C class named NSCursor
+ https://bugs.webkit.org/show_bug.cgi?id=192602
+ <rdar://problem/46615532>
+
+ Reviewed by Wenson Hsieh.
+
+ * platform/ios/wak/WAKAppKitStubs.h:
+ * platform/ios/wak/WAKAppKitStubs.m:
+ (+[NSCursor setHiddenUntilMouseMoves:]): Deleted.
+ Get rid of the class.
+ Also remove a comment that seems to have detached from wherever it's supposed to be.
+
+2018-12-12 Kocsen Chung <[email protected]>
+
Cherry-pick r239046. rdar://problem/46500832
Add SPI to allow the client to set the user-agent at main frame level, from the UIProcess
Modified: tags/Safari-607.1.16.2/Source/WebCore/platform/ios/wak/WAKAppKitStubs.h (239120 => 239121)
--- tags/Safari-607.1.16.2/Source/WebCore/platform/ios/wak/WAKAppKitStubs.h 2018-12-12 18:52:55 UTC (rev 239120)
+++ tags/Safari-607.1.16.2/Source/WebCore/platform/ios/wak/WAKAppKitStubs.h 2018-12-12 18:52:59 UTC (rev 239121)
@@ -23,8 +23,6 @@
* THE POSSIBILITY OF SUCH DAMAGE.
*/
-/* Unicodes we reserve for function keys on the keyboard, OpenStep reserves the range 0xF700-0xF8FF for this purpose. The availability of various keys will be system dependent. */
-
#ifndef WAKAppKitStubs_h
#define WAKAppKitStubs_h
@@ -205,10 +203,6 @@
NSSelectingPrevious
} NSSelectionDirection;
-WEBCORE_EXPORT @interface NSCursor : NSObject
-+ (void)setHiddenUntilMouseMoves:(BOOL)flag;
-@end
-
#endif // TARGET_OS_IPHONE
#endif // WAKAppKitStubs_h
Modified: tags/Safari-607.1.16.2/Source/WebCore/platform/ios/wak/WAKAppKitStubs.m (239120 => 239121)
--- tags/Safari-607.1.16.2/Source/WebCore/platform/ios/wak/WAKAppKitStubs.m 2018-12-12 18:52:55 UTC (rev 239120)
+++ tags/Safari-607.1.16.2/Source/WebCore/platform/ios/wak/WAKAppKitStubs.m 2018-12-12 18:52:59 UTC (rev 239121)
@@ -30,11 +30,4 @@
id NSApp = nil;
-@implementation NSCursor
-+ (void)setHiddenUntilMouseMoves:(BOOL)flag
-{
- UNUSED_PARAM(flag);
-}
-@end
-
#endif // PLATFORM(IOS_FAMILY)
Modified: tags/Safari-607.1.16.2/Source/WebKitLegacy/mac/ChangeLog (239120 => 239121)
--- tags/Safari-607.1.16.2/Source/WebKitLegacy/mac/ChangeLog 2018-12-12 18:52:55 UTC (rev 239120)
+++ tags/Safari-607.1.16.2/Source/WebKitLegacy/mac/ChangeLog 2018-12-12 18:52:59 UTC (rev 239121)
@@ -1,3 +1,48 @@
+2018-12-12 Kocsen Chung <[email protected]>
+
+ Cherry-pick r239088. rdar://problem/46615532
+
+ WebCore shouldn't have a Objective-C class named NSCursor
+ https://bugs.webkit.org/show_bug.cgi?id=192602
+ <rdar://problem/46615532>
+
+ Reviewed by Wenson Hsieh.
+
+ Source/WebCore:
+
+ * platform/ios/wak/WAKAppKitStubs.h:
+ * platform/ios/wak/WAKAppKitStubs.m:
+ (+[NSCursor setHiddenUntilMouseMoves:]): Deleted.
+ Get rid of the class.
+ Also remove a comment that seems to have detached from wherever it's supposed to be.
+
+ Source/WebKitLegacy/mac:
+
+ * WebView/WebFrameView.mm:
+ (-[WebFrameView keyDown:keyDown:]):
+ * WebView/WebHTMLView.mm:
+ (-[WebHTMLView keyDown:]):
+ (-[WebHTMLView performKeyEquivalent:]):
+ Only use NSCursor on macOS.
+
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@239088 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2018-12-11 Tim Horton <[email protected]>
+
+ WebCore shouldn't have a Objective-C class named NSCursor
+ https://bugs.webkit.org/show_bug.cgi?id=192602
+ <rdar://problem/46615532>
+
+ Reviewed by Wenson Hsieh.
+
+ * WebView/WebFrameView.mm:
+ (-[WebFrameView keyDown:keyDown:]):
+ * WebView/WebHTMLView.mm:
+ (-[WebHTMLView keyDown:]):
+ (-[WebHTMLView performKeyEquivalent:]):
+ Only use NSCursor on macOS.
+
2018-12-03 Alan Coon <[email protected]>
Cherry-pick r238815. rdar://problem/46383932
Modified: tags/Safari-607.1.16.2/Source/WebKitLegacy/mac/WebView/WebFrameView.mm (239120 => 239121)
--- tags/Safari-607.1.16.2/Source/WebKitLegacy/mac/WebView/WebFrameView.mm 2018-12-12 18:52:55 UTC (rev 239120)
+++ tags/Safari-607.1.16.2/Source/WebKitLegacy/mac/WebView/WebFrameView.mm 2018-12-12 18:52:59 UTC (rev 239121)
@@ -1087,8 +1087,10 @@
if (callSuper) {
[super keyDown:event];
} else {
- // if we did something useful, get the cursor out of the way
+#if PLATFORM(MAC)
+ // If we did something useful, get the cursor out of the way.
[NSCursor setHiddenUntilMouseMoves:YES];
+#endif
}
}
Modified: tags/Safari-607.1.16.2/Source/WebKitLegacy/mac/WebView/WebHTMLView.mm (239120 => 239121)
--- tags/Safari-607.1.16.2/Source/WebKitLegacy/mac/WebView/WebHTMLView.mm 2018-12-12 18:52:55 UTC (rev 239120)
+++ tags/Safari-607.1.16.2/Source/WebKitLegacy/mac/WebView/WebHTMLView.mm 2018-12-12 18:52:59 UTC (rev 239121)
@@ -4976,8 +4976,11 @@
if (callSuper)
[super keyDown:event];
- else
+ else {
+#if PLATFORM(MAC)
[NSCursor setHiddenUntilMouseMoves:YES];
+#endif
+ }
}
- (void)keyUp:(WebEvent *)event
@@ -5281,9 +5284,11 @@
if (Frame* frame = core([self _frame]))
ret = frame->eventHandler().keyEvent(event);
- if (ret)
+ if (ret) {
+#if PLATFORM(MAC)
[NSCursor setHiddenUntilMouseMoves:YES];
- else
+#endif
+ } else
ret = [self _handleStyleKeyEquivalent:event] || [super performKeyEquivalent:event];
[self release];
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
