Commit f7471231 authored by Wei Li's avatar Wei Li Committed by Commit Bot

Minor updates on focus manager's comments

Bug: none
Change-Id: I5f98be89e8a0735ae5eaeb1295430b3d23b4bbb6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2508875Reviewed-by: default avatarPeter Kasting <pkasting@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#822757}
parent 3717028f
...@@ -22,12 +22,6 @@ ...@@ -22,12 +22,6 @@
// - The native focus, which is the focus that a gfx::NativeView has. // - The native focus, which is the focus that a gfx::NativeView has.
// - The view focus, which is the focus that a views::View has. // - The view focus, which is the focus that a views::View has.
// //
// When registering a view with the FocusManager, the caller can provide a view
// whose focus will be kept in sync with the view the FocusManager is managing.
//
// (Focus registration is already done for you if you subclass the NativeControl
// class or if you use the NativeViewHost class.)
//
// When a top window (derived from views::Widget) that is not a child window is // When a top window (derived from views::Widget) that is not a child window is
// created, it creates and owns a FocusManager to manage the focus for itself // created, it creates and owns a FocusManager to manage the focus for itself
// and all its child windows. // and all its child windows.
...@@ -43,8 +37,8 @@ ...@@ -43,8 +37,8 @@
// order. // order.
// //
// If you are embedding a native view containing a nested RootView (for example // If you are embedding a native view containing a nested RootView (for example
// by adding a NativeControl that contains a NativeWidgetWin as its native // by adding a view that contains a native widget as its native component),
// component), then you need to: // then you need to:
// //
// - Override the View::GetFocusTraversable method in your outer component. // - Override the View::GetFocusTraversable method in your outer component.
// It should return the RootView of the inner component. This is used when // It should return the RootView of the inner component. This is used when
......
...@@ -14,8 +14,8 @@ namespace views { ...@@ -14,8 +14,8 @@ namespace views {
class FocusManager; class FocusManager;
class Widget; class Widget;
// A factory to create FocusManager. This is used in unit tests // A factory to create FocusManager. This can be used to inject a custom
// to inject a custom factory. // factory.
class VIEWS_EXPORT FocusManagerFactory { class VIEWS_EXPORT FocusManagerFactory {
public: public:
// Create a FocusManager for the given |widget| using the installed Factory. // Create a FocusManager for the given |widget| using the installed Factory.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment