Summary: | Cursor doesn't change back to pointer when leaving Mail | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> | ||||||||||||||
Component: | UI Events | Assignee: | Ryosuke Niwa <rniwa> | ||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||
Severity: | Normal | CC: | ap, commit-queue, darin, enrica, mitz, sam | ||||||||||||||
Priority: | P2 | Keywords: | InRadar, Regression | ||||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||||
Hardware: | Unspecified | ||||||||||||||||
OS: | Unspecified | ||||||||||||||||
Bug Depends on: | 132142 | ||||||||||||||||
Bug Blocks: | |||||||||||||||||
Attachments: |
|
Description
Ryosuke Niwa
2014-04-22 21:57:05 PDT
This bug is partially caused by http://trac.webkit.org/changeset/147739. Created attachment 229955 [details]
Fixes the bug
Comment on attachment 229955 [details] Fixes the bug View in context: https://bugs.webkit.org/attachment.cgi?id=229955&action=review > Source/WebCore/page/EventHandler.cpp:1386 > + bool editable = node ? node->hasEditableStyle() : false; I think that "node && node->hasEditableStyle()" would be more readable than a ternary. (In reply to comment #3) > (From update of attachment 229955 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=229955&action=review > > > Source/WebCore/page/EventHandler.cpp:1386 > > + bool editable = node ? node->hasEditableStyle() : false; > > I think that "node && node->hasEditableStyle()" would be more readable than a ternary. Will fix. Thanks for the review! Created attachment 229957 [details]
Patch for landing
Comment on attachment 229957 [details] Patch for landing Clearing flags on attachment: 229957 Committed r167700: <http://trac.webkit.org/changeset/167700> All reviewed patches have been landed. Closing bug. Comment on attachment 229955 [details] Fixes the bug View in context: https://bugs.webkit.org/attachment.cgi?id=229955&action=review > Source/WebKit/mac/ChangeLog:14 > + Instead, check whether the NSWindow of the WebView is the key window or not since > + key window appears to control the cursor style in Cocoa as far as I've tested: > + https://developer.apple.com/library/mac/documentation/Cocoa/Conceptual/WinPanel/Concepts/ChangingMainKeyWindow.html Did you retest the problems described in <rdar://problem/14619911> to see if it’s still fixed? Re-opened since this is blocked by bug 132142 Created attachment 249459 [details]
Fixes the bug
Created attachment 249508 [details]
Patch
Created attachment 249509 [details]
Fixed change logs
Created attachment 249513 [details]
Reverted the refactroing per Andres' request
Committed r182026: <http://trac.webkit.org/changeset/182026> |