Currently NativeWidgetMus supports activation via an application of focus on mus::Window. We need to implement the remainder of activation and focus methods. - Deactivate - IsActive - ClearNativeFocus We plan to have activation map to focus for mus.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/fe2dda84ac99039a639e6be859ce087474cad166 commit fe2dda84ac99039a639e6be859ce087474cad166 Author: sky <sky@chromium.org> Date: Wed Apr 06 20:37:40 2016 Makes WindowTree::SetFocus(null) work We need this so clients can clear focus as necessary. BUG= 599552 TEST=covered by tests R=ben@chromium.org Review URL: https://codereview.chromium.org/1862853002 Cr-Commit-Position: refs/heads/master@{#385550} [modify] https://crrev.com/fe2dda84ac99039a639e6be859ce087474cad166/components/mus/public/cpp/lib/window_tree_client_impl.cc [modify] https://crrev.com/fe2dda84ac99039a639e6be859ce087474cad166/components/mus/public/cpp/lib/window_tree_client_impl.h [modify] https://crrev.com/fe2dda84ac99039a639e6be859ce087474cad166/components/mus/public/cpp/window.h [modify] https://crrev.com/fe2dda84ac99039a639e6be859ce087474cad166/components/mus/public/cpp/window_tree_connection.h [modify] https://crrev.com/fe2dda84ac99039a639e6be859ce087474cad166/components/mus/public/interfaces/window_tree.mojom [modify] https://crrev.com/fe2dda84ac99039a639e6be859ce087474cad166/components/mus/ws/default_access_policy.cc [modify] https://crrev.com/fe2dda84ac99039a639e6be859ce087474cad166/components/mus/ws/window_manager_client_unittest.cc [modify] https://crrev.com/fe2dda84ac99039a639e6be859ce087474cad166/components/mus/ws/window_server.cc [modify] https://crrev.com/fe2dda84ac99039a639e6be859ce087474cad166/components/mus/ws/window_server.h [modify] https://crrev.com/fe2dda84ac99039a639e6be859ce087474cad166/components/mus/ws/window_tree.cc [modify] https://crrev.com/fe2dda84ac99039a639e6be859ce087474cad166/ui/views/mus/native_widget_mus.cc
It looks like between #2, https://codereview.chromium.org/1968133002, & https://codereview.chromium.org/2049723002 that this issue is resolved. Were we planning any more work here?
Yes, this is done now.
Comment 1 by sky@chromium.org
, Apr 5 2016Status: Started (was: Available)