Chromium Code Reviews
Help | Chromium Project | Sign in
(2)

Issue 10789018: aura: Add X11 host window management. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 11 months ago by reveman
Modified:
2 years, 5 months ago
CC:
chromium-reviews, sadrul, ben+watch_chromium.org, Elliot Glaysher, tfarina
Visibility:
Public.

Description

aura: Add X11 host window management. BUG=124444 TEST=manual

Patch Set 1 #

Total comments: 1

Patch Set 2 : Keep ash switches alphabetized. #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+1910 lines, -22 lines) Patch
M ash/ash.gyp View 1 chunk +4 lines, -0 lines 0 comments Download
M ash/ash_switches.h View 1 1 chunk +1 line, -0 lines 0 comments Download
M ash/ash_switches.cc View 1 1 chunk +4 lines, -0 lines 0 comments Download
A ash/hostwm/OWNERS View 1 chunk +2 lines, -0 lines 0 comments Download
A ash/hostwm/host_window_container.h View 1 chunk +47 lines, -0 lines 3 comments Download
A ash/hostwm/host_window_container.cc View 1 chunk +108 lines, -0 lines 0 comments Download
A ash/hostwm/host_window_manager.h View 1 chunk +51 lines, -0 lines 0 comments Download
A ash/hostwm/host_window_manager.cc View 1 chunk +119 lines, -0 lines 0 comments Download
M ash/shell.h View 3 chunks +9 lines, -0 lines 0 comments Download
M ash/shell.cc View 5 chunks +26 lines, -0 lines 0 comments Download
M ui/aura/aura.gyp View 2 chunks +7 lines, -0 lines 0 comments Download
A ui/aura/hostwm/OWNERS View 1 chunk +2 lines, -0 lines 0 comments Download
A ui/aura/hostwm/host_window_delegate.h View 1 chunk +36 lines, -0 lines 0 comments Download
A ui/aura/hostwm/host_window_delegate.cc View 1 chunk +25 lines, -0 lines 0 comments Download
A ui/aura/hostwm/host_window_manager_client.h View 1 chunk +36 lines, -0 lines 0 comments Download
A ui/aura/hostwm/host_window_manager_x11.h View 1 chunk +262 lines, -0 lines 0 comments Download
A ui/aura/hostwm/host_window_manager_x11.cc View 1 chunk +1042 lines, -0 lines 0 comments Download
M ui/aura/root_window.h View 3 chunks +9 lines, -0 lines 0 comments Download
M ui/aura/root_window.cc View 3 chunks +16 lines, -0 lines 0 comments Download
M ui/aura/root_window_host.h View 2 chunks +4 lines, -0 lines 0 comments Download
M ui/aura/root_window_host_linux.h View 3 chunks +3 lines, -0 lines 0 comments Download
M ui/aura/root_window_host_linux.cc View 10 chunks +73 lines, -22 lines 0 comments Download
M ui/aura/root_window_host_mac.mm View 2 chunks +6 lines, -0 lines 0 comments Download
M ui/aura/root_window_host_win.h View 2 chunks +3 lines, -0 lines 0 comments Download
M ui/aura/root_window_host_win.cc View 2 chunks +10 lines, -0 lines 0 comments Download
M ui/aura/root_window_observer.h View 2 chunks +5 lines, -0 lines 0 comments Download
Trybot results:
Commit: CQ not working?

Messages

Total messages: 6 (0 generated)
reveman
Hi, this is Dana's and my current implementation of X11 host window management. Basically allowing ...
2 years, 11 months ago (2012-07-17 10:13:42 UTC) #1
Daniel Erat
This is a big, unexpected change. :-) What kind of timeframe were you hoping for ...
2 years, 11 months ago (2012-07-17 18:23:57 UTC) #2
danakj OOO back july 6
To go top to bottom, I'd suggest: 1. ash/hostwm is the user-facing code, and shows ...
2 years, 11 months ago (2012-07-17 19:20:48 UTC) #3
Daniel Erat
+erg since this may have some overlap with the Linux work that he's doing. (This ...
2 years, 11 months ago (2012-07-17 19:48:19 UTC) #4
tfarina
http://codereview.chromium.org/10789018/diff/7001/ash/hostwm/host_window_container.h File ash/hostwm/host_window_container.h (right): http://codereview.chromium.org/10789018/diff/7001/ash/hostwm/host_window_container.h#newcode8 ash/hostwm/host_window_container.h:8: #include "base/compiler_specific.h" include basictypes.h as well. http://codereview.chromium.org/10789018/diff/7001/ash/hostwm/host_window_container.h#newcode24 ash/hostwm/host_window_container.h:24: // ...
2 years, 11 months ago (2012-07-18 14:43:42 UTC) #5
Ben Goodger (Google)
2 years, 11 months ago (2012-07-27 21:40:45 UTC) #6
so, I think you should be able to achieve pretty much all of this from a
separate component that lives downstream of aura/ash and potentially in a
separate repo.

In ash, it looks like you're just hanging a few things off the shell. You can do
that from somewhere else in your process' main function.

In aura, again it seems like a bunch of x window configuration that you could
probably do from a downstream component.

As part of the win-aura work I am going to make it more easily possible to
replace the RootWindowHost, maybe that would work well for you in that you could
write your own that better supports this use case.

Look at how other components are set up. e.g. look at ash.gyp and how it depends
on other components. Look at the src/DEPS files and how it consumes components
from other repos.

At any rate i would like to keep this kind of thing out of aura/ash since it is
very platform/application specific.

-Ben
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1f9106d