Unreviewed, rolling out r167700.
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 25 Apr 2014 01:24:27 +0000 (01:24 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 25 Apr 2014 01:24:27 +0000 (01:24 +0000)
https://bugs.webkit.org/show_bug.cgi?id=132142

Incorrectly reverted the change in r167547 for
webkit.org/b/131898 (Requested by rniwa on #webkit).

Reverted changeset:

"Cursor doesn't change back to pointer when leaving the Safari
window"
https://bugs.webkit.org/show_bug.cgi?id=132038
http://trac.webkit.org/changeset/167700

Patch by Commit Queue <commit-queue@webkit.org> on 2014-04-24

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@167788 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/page/EventHandler.cpp
Source/WebKit/mac/ChangeLog
Source/WebKit/mac/WebCoreSupport/WebChromeClient.mm
Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/mac/PageClientImpl.mm

index 31890dd489ab6215d40689b085260a1aac423ee8..d3da664235dae8a94494456d4db3008c14708ee1 100644 (file)
@@ -1,3 +1,18 @@
+2014-04-24  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r167700.
+        https://bugs.webkit.org/show_bug.cgi?id=132142
+
+        Incorrectly reverted the change in r167547 for
+        webkit.org/b/131898 (Requested by rniwa on #webkit).
+
+        Reverted changeset:
+
+        "Cursor doesn't change back to pointer when leaving the Safari
+        window"
+        https://bugs.webkit.org/show_bug.cgi?id=132038
+        http://trac.webkit.org/changeset/167700
+
 2014-04-24  Brady Eidson  <beidson@apple.com>
 
         Rename "IMAGE_CONTROLS" feature to "SERVICE_CONTROLS"
index 9ae0091fec415250100ba6c5efa5786bab0c02ac..c9d5d4cfdee764de5f1e48fcc976a5b6337583c1 100644 (file)
@@ -1311,7 +1311,10 @@ OptionalCursor EventHandler::selectCursor(const HitTestResult& result, bool shif
 #endif
 
     Node* node = result.targetNode();
-    auto renderer = node ? node->renderer() : 0;
+    if (!node)
+        return NoCursorChange;
+
+    auto renderer = node->renderer();
     RenderStyle* style = renderer ? &renderer->style() : nullptr;
     bool horizontalText = !style || style->isHorizontalWritingMode();
     const Cursor& iBeam = horizontalText ? iBeamCursor() : verticalTextCursor();
@@ -1383,7 +1386,7 @@ OptionalCursor EventHandler::selectCursor(const HitTestResult& result, bool shif
 
     switch (style ? style->cursor() : CURSOR_AUTO) {
     case CURSOR_AUTO: {
-        bool editable = node && node->hasEditableStyle();
+        bool editable = node->hasEditableStyle();
 
         if (useHandCursor(node, result.isOverLink(), shiftKey))
             return handCursor();
index 9dd6997690677a59e2953f202f4e7258f64d0fd7..39f27a39120a66a545c0a95bb3dad74ba0b86d6f 100644 (file)
@@ -1,3 +1,18 @@
+2014-04-24  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r167700.
+        https://bugs.webkit.org/show_bug.cgi?id=132142
+
+        Incorrectly reverted the change in r167547 for
+        webkit.org/b/131898 (Requested by rniwa on #webkit).
+
+        Reverted changeset:
+
+        "Cursor doesn't change back to pointer when leaving the Safari
+        window"
+        https://bugs.webkit.org/show_bug.cgi?id=132038
+        http://trac.webkit.org/changeset/167700
+
 2014-04-24  Brady Eidson  <beidson@apple.com>
 
         Rename "IMAGE_CONTROLS" feature to "SERVICE_CONTROLS"
index 641249bade0987efb69c10e6af2a5908b41733c2..976b7cf1f504a2b0a99a59ec9106c607da5bdab9 100644 (file)
@@ -773,7 +773,10 @@ void WebChromeClient::setCursor(const WebCore::Cursor& cursor)
         return;
 
     NSWindow *window = [m_webView window];
-    if (!window || ![window isKeyWindow])
+    if (!window)
+        return;
+
+    if ([window windowNumber] != [NSWindow windowNumberAtPoint:[NSEvent mouseLocation] belowWindowWithWindowNumber:0])
         return;
 
     NSCursor *platformCursor = cursor.platformCursor();
index 93263ff471b9a943b25c7131cb9eb624f21cff78..d82a79131b7030d006f87b04c0a4c4002bceab6f 100644 (file)
@@ -1,3 +1,18 @@
+2014-04-24  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r167700.
+        https://bugs.webkit.org/show_bug.cgi?id=132142
+
+        Incorrectly reverted the change in r167547 for
+        webkit.org/b/131898 (Requested by rniwa on #webkit).
+
+        Reverted changeset:
+
+        "Cursor doesn't change back to pointer when leaving the Safari
+        window"
+        https://bugs.webkit.org/show_bug.cgi?id=132038
+        http://trac.webkit.org/changeset/167700
+
 2014-04-24  Brady Eidson  <beidson@apple.com>
 
         Rename "IMAGE_CONTROLS" feature to "SERVICE_CONTROLS"
index 9a1e75e1f8318b6861eddec09d28d87d4c38e47d..81c19c7f4c62687a54973b5e27865b12316177d1 100644 (file)
@@ -324,7 +324,10 @@ void PageClientImpl::setCursor(const WebCore::Cursor& cursor)
         return;
 
     NSWindow *window = [m_wkView window];
-    if (!window || ![window isKeyWindow])
+    if (!window)
+        return;
+
+    if ([window windowNumber] != [NSWindow windowNumberAtPoint:[NSEvent mouseLocation] belowWindowWithWindowNumber:0])
         return;
 
     NSCursor *platformCursor = cursor.platformCursor();