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

google chrome drop down windows are missing attributes #519

Closed
totaam opened this issue Feb 14, 2014 · 9 comments
Closed

google chrome drop down windows are missing attributes #519

totaam opened this issue Feb 14, 2014 · 9 comments

Comments

@totaam
Copy link
Collaborator

totaam commented Feb 14, 2014

Issue migrated from trac ticket # 519

component: external | priority: major | resolution: wontfix

2014-02-14 07:32:42: totaam created the issue


split from #139: we now handle grabs correctly and simulate pressing the Escape key when the application loses focus (since there are no grabs on win32).

The problem with the drop down windows as used by chrome is that they are:

  • not using the correct window-type for the drop down, sending it as NORMAL instead
  • not grabbing the keyboard or mouse input when the drop down is shown
  • not setting the window's transient-for attribute on the temporary window either

I'm not sure there's much we can do about it. Any amount of guessing is bound to break well behaved applications. It would be much easier to get chrome fixed instead.

FWIW: I've tried google chrome 32, and now google chrome beta 33 (which is meant to be using the "aura" toolkit instead of gtk), as well as chromium. No luck with any of them.

@totaam
Copy link
Collaborator Author

totaam commented Feb 15, 2014

2014-02-15 11:59:16: totaam changed status from new to assigned

@totaam
Copy link
Collaborator Author

totaam commented Feb 15, 2014

2014-02-15 11:59:16: totaam changed owner from antoine to totaam

@totaam
Copy link
Collaborator Author

totaam commented Feb 15, 2014

2014-02-15 11:59:16: totaam edited the issue description

@totaam
Copy link
Collaborator Author

totaam commented Mar 5, 2014

2014-03-05 04:32:06: totaam commented


Here's a diff of the xwininfo showing the new drop down window as a child of the root window, it should really be a child of the browser window it came from:

6c6,10
<      110 children:
---
>      111 children:
>      0x2a50015 "Google Chrome": ("google-chrome-beta" "Google-chrome-beta")  252x302+513+579  +513+579
>         2 children:
>         0x2a50018 (has no name): ()  252x302+0+0  +513+579
>         0x2a50016 (has no name): ()  1x1+-1+-1  +512+578
164a169,172
>      0x2a4a2d3 "Test Page - Google Chrome": ("Google-chrome-beta (/home/antoine/.config/google-chrome-beta)" "Google-chrome-beta")  1515x898+23+45  +23+45
>         2 children:
>         0x2a42419 (has no name): ()  1507x789+4+105  +27+150
>         0x2a4a2d4 (has no name): ()  1x1+-1+-1  +22+44
170,173d177
<      0x2a4a2d3 "Test Page - Google Chrome": ("Google-chrome-beta (/home/antoine/.config/google-chrome-beta)" "Google-chrome-beta")  1515x898+23+45  +23+45
<         2 children:
<         0x2a42419 (has no name): ()  1507x789+4+105  +27+150
<         0x2a4a2d4 (has no name): ()  1x1+-1+-1  +22+44

The drop down window is 252x302 pixels in dimension.

@totaam
Copy link
Collaborator Author

totaam commented Mar 5, 2014

2014-03-05 05:02:18: totaam commented


I have created an upstream bug with chrome: chromium #349322

@totaam
Copy link
Collaborator Author

totaam commented Mar 16, 2014

2014-03-16 08:52:13: totaam changed status from assigned to closed

@totaam
Copy link
Collaborator Author

totaam commented Mar 16, 2014

2014-03-16 08:52:13: totaam changed resolution from ** to wontfix

@totaam
Copy link
Collaborator Author

totaam commented Mar 16, 2014

2014-03-16 08:52:13: totaam commented


Nothing we can do about this but wait for upstream to fix their code.

Closing.

@totaam totaam closed this as completed Mar 16, 2014
@totaam
Copy link
Collaborator Author

totaam commented Sep 25, 2015

2015-09-25 00:08:24: Dweller commented


Upstream bug has been archived due to inactivity.. might need a prod, else it'll never be looked at.

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

No branches or pull requests

1 participant