Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent invisible windows from being activated #8676

Merged
merged 7 commits into from Feb 15, 2017
Merged

Conversation

kevinsawicki
Copy link
Contributor

@kevinsawicki kevinsawicki commented Feb 14, 2017

This pull request adds a Activate override to the native aura widget used on Windows and only activates it when the window is visible so the currently focused window doesn't get incorrectly blurred when an invisible window is created.

The focus manager appears to have changed in Chrome 56 and focusing a web contents in an invisible window causes the focused window to lose focus even though the invisible window cannot gain focus.

https://chromium.googlesource.com/chromium/src/+/5c16e0e849526c1c627e08a05351ab38a9cfbcf1%5E%21/#F7

  • Add tests

Closes #8649

@kevinsawicki kevinsawicki force-pushed the focus-delegate branch 3 times, most recently from 79430b7 to 44443f5 Compare February 14, 2017 00:52
@kevinsawicki kevinsawicki changed the title Implement CanFocus window delegate and map to window state [WIP] Implement CanFocus delegate and map to window state Feb 14, 2017
@kevinsawicki kevinsawicki changed the title [WIP] Implement CanFocus delegate and map to window state Implement CanFocus delegate and map to window state Feb 14, 2017
@kevinsawicki kevinsawicki force-pushed the focus-delegate branch 2 times, most recently from 2eeb87f to 7aebf96 Compare February 14, 2017 20:41
@kevinsawicki kevinsawicki changed the title Implement CanFocus delegate and map to window state Prevent invisible windows from being activated Feb 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants