Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(530)

Unified Diff: views/controls/native/native_view_host.h

Issue 231022: Reverting 27113. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 11 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/views/location_bar_view.cc ('k') | views/controls/native/native_view_host.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: views/controls/native/native_view_host.h
===================================================================
--- views/controls/native/native_view_host.h (revision 27114)
+++ views/controls/native/native_view_host.h (working copy)
@@ -42,27 +42,15 @@
// Sets a preferred size for the native view attached to this View.
void SetPreferredSize(const gfx::Size& size);
- // A NativeViewHost must keep the focused view in the focus manager and the
- // native focus with in sync . There are 2 aspects to this:
- // - when the native view receives focus, the focus manager must be notified
- // that the associated view is now the focused view.
- // In simple cases where the NativeViewHost directly wraps a native window
- // as is, the associated view is this NativeViewHost. In other cases where
- // the NativeViewHost is part of another view (such as for the TextField
- // class), the actual View is not the NativeViewHost and set_focus_view()
- // must be called to set the associated view before Attach() is called.
- // - when the view is focused (by calling View::RequestFocus()), it must focus
- // the appropriate native view. In simple cases where the native view does
- // not have children or is the native view that should really get the focus,
- // this works without doing anything. In case where the native view that
- // should get the focus is not the native view passed to Attach(), then
- // the appropriate native view should be specified to
- // set_focus_native_view() before Attach() is called.
+ // A NativeViewHost has an associated focus View so that the focus of the
+ // native control and of the View are kept in sync. In simple cases where the
+ // NativeViewHost directly wraps a native window as is, the associated view
+ // is this View. In other cases where the NativeViewHost is part of another
+ // view (such as TextField), the actual View is not the NativeViewHost and
+ // this method must be called to set that.
+ // This method must be called before Attach().
void set_focus_view(View* view) { focus_view_ = view; }
- void set_focus_native_view(gfx::NativeView native_view) {
- focus_native_view_ = native_view;
- }
- gfx::NativeView focus_native_view() const { return focus_native_view_; }
+ View* focus_view() { return focus_view_; }
// Fast resizing will move the native view and clip its visible region, this
// will result in white areas and will not resize the content (so scrollbars
@@ -76,10 +64,6 @@
// Accessor for |native_view_|.
gfx::NativeView native_view() const { return native_view_; }
- // Called by the NativeViewHostWrapper to notify that the |focus_native_view_|
- // got focus.
- void GotNativeFocus();
-
void NativeViewDestroyed();
// Overridden from View:
@@ -112,9 +96,6 @@
// The view that should be given focus when this NativeViewHost is focused.
View* focus_view_;
- // The native view that should get the focus when this View gets focused.
- gfx::NativeView focus_native_view_;
-
DISALLOW_COPY_AND_ASSIGN(NativeViewHost);
};
« no previous file with comments | « chrome/browser/views/location_bar_view.cc ('k') | views/controls/native/native_view_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698