summaryrefslogtreecommitdiffstats
path: root/views/controls/native/native_view_host.cc
diff options
context:
space:
mode:
Diffstat (limited to 'views/controls/native/native_view_host.cc')
-rw-r--r--views/controls/native/native_view_host.cc40
1 files changed, 6 insertions, 34 deletions
diff --git a/views/controls/native/native_view_host.cc b/views/controls/native/native_view_host.cc
index b5eb509..9cbeaf7 100644
--- a/views/controls/native/native_view_host.cc
+++ b/views/controls/native/native_view_host.cc
@@ -7,7 +7,6 @@
#include "base/logging.h"
#include "app/gfx/canvas.h"
#include "views/controls/native/native_view_host_wrapper.h"
-#include "views/focus/focus_manager.h"
#include "views/widget/widget.h"
namespace views {
@@ -21,10 +20,7 @@ const char NativeViewHost::kViewClassName[] = "views/NativeViewHost";
NativeViewHost::NativeViewHost()
: native_view_(NULL),
fast_resize_(false),
- focus_native_view_(NULL) {
- // By default we make this view the one that should be set as the focused view
- // when the native view gets the focus.
- focus_view_ = this;
+ focus_view_(NULL) {
// The native widget is placed relative to the root. As such, we need to
// know when the position of any ancestor changes, or our visibility relative
// to other views changed as it'll effect our position relative to the root.
@@ -37,12 +33,10 @@ NativeViewHost::~NativeViewHost() {
void NativeViewHost::Attach(gfx::NativeView native_view) {
DCHECK(!native_view_);
native_view_ = native_view;
- // If this NativeViewHost is the focus view, than it should obviously be
- // focusable. If it is not, then it acts as a container and we don't want it
- // to get focus through tab-traversal, so we make it not focusable.
- SetFocusable(focus_view_ == this);
- if (!focus_native_view_)
- focus_native_view_ = native_view;
+ // If set_focus_view() has not been invoked, this view is the one that should
+ // be seen as focused when the native view receives focus.
+ if (!focus_view_)
+ focus_view_ = this;
native_wrapper_->NativeViewAttached();
}
@@ -50,7 +44,6 @@ void NativeViewHost::Detach() {
DCHECK(native_view_);
native_wrapper_->NativeViewDetaching();
native_view_ = NULL;
- focus_native_view_ = NULL;
}
void NativeViewHost::SetPreferredSize(const gfx::Size& size) {
@@ -64,22 +57,6 @@ void NativeViewHost::NativeViewDestroyed() {
native_view_ = NULL;
}
-void NativeViewHost::GotNativeFocus() {
- // Some NativeViewHost may not have an associated focus view. This is the
- // case for containers for example. They might still get the native focus,
- // if one non native view they contain gets focused. In that case, we should
- // not change the focused view.
- if (!focus_view_ || !focus_view_->IsFocusable())
- return;
-
- FocusManager* focus_manager = GetFocusManager();
- if (!focus_manager) {
- NOTREACHED();
- return;
- }
- focus_manager->SetFocusedView(focus_view_);
-}
-
////////////////////////////////////////////////////////////////////////////////
// NativeViewHost, View overrides:
@@ -157,12 +134,7 @@ std::string NativeViewHost::GetClassName() const {
}
void NativeViewHost::Focus() {
- FocusManager* focus_manager = GetFocusManager();
- if (!focus_manager) {
- NOTREACHED();
- return;
- }
- focus_manager->FocusNativeView(focus_native_view_);
+ native_wrapper_->SetFocus();
}
} // namespace views