Browser should get notified if a views textfield changes focus. In addition,

chrome/browser/ui/touch/frame/touch_browser_frame_view.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/touch/frame/touch_browser_frame_view.h"
 
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/renderer_host/render_view_host.h"
 #include "chrome/browser/tab_contents/navigation_controller.h"
 #include "chrome/browser/tab_contents/tab_contents.h"
@@ skipping 20 matching lines @@
       keyboard_(NULL) {
   registrar_.Add(this,
                  NotificationType::NAV_ENTRY_COMMITTED,
                  NotificationService::AllSources());
   registrar_.Add(this,
                  NotificationType::FOCUS_CHANGED_IN_PAGE,
                  NotificationService::AllSources());
 }
 
 TouchBrowserFrameView::~TouchBrowserFrameView() {
+  if (focus_change_listener_.get()) {
+    GetFocusManager()->RemoveFocusChangeListener(focus_change_listener_.get());
+  }
 }
 
 void TouchBrowserFrameView::Layout() {
   OpaqueBrowserFrameView::Layout();
 
   if (!keyboard_)
     return;
 
   keyboard_->SetVisible(keyboard_showing_);
   keyboard_->SetBounds(GetBoundsForReservedArea());
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 // TouchBrowserFrameView, protected:
 int TouchBrowserFrameView::GetReservedHeight() const {
   if (keyboard_showing_)
     return kKeyboardHeight;
 
   return 0;
 }
 
+void TouchBrowserFrameView::ViewHierarchyChanged(bool is_add, View* parent,

[bryeung, 2011/01/27 20:50:34]

Why are we doing this in ViewHierarchyChanged?  Is that the standard idiom? 
(Didn't seem like it from a tiny bit of code searching I did.)

[oshima, 2011/01/27 21:35:41]

Yes, this is a standard item to initialize things that require Widget.

+    View* child) {

[oshima, 2011/01/27 19:58:33]

fix indentation. see my comment in header.

[varunjain, 2011/01/27 23:01:38]

Done.

+  OpaqueBrowserFrameView::ViewHierarchyChanged(is_add, parent, child);
+  if (!focus_change_listener_.get()) {

[oshima, 2011/01/27 19:58:33]

You need to check GetWidget(). This assumes that this view is always added to a
parent view that is already added to widget.

[varunjain, 2011/01/27 23:01:38]

GetWidget is checked inside GetFocusManager. I will check GetFocusManager here.

+    focus_change_listener_.reset(new TouchBrowserFocusChangeListener(this));
+    GetFocusManager()->AddFocusChangeListener(focus_change_listener_.get());

[bryeung, 2011/01/27 20:50:34]

Related to oshima's point: GetFocusManager can return NULL.

[varunjain, 2011/01/27 23:01:38]

Done.

+  }
+}
+
 ///////////////////////////////////////////////////////////////////////////////
 // TouchBrowserFrameView, private:
 
 void TouchBrowserFrameView::InitVirtualKeyboard() {
   if (keyboard_)
     return;
 
   Profile* keyboard_profile = browser_view()->browser()->profile();
   DCHECK(keyboard_profile) << "Profile required for virtual keyboard.";
 
@@ skipping 39 matching lines @@
             Source<RenderViewHost>(source).ptr())
       UpdateKeyboardAndLayout(*Details<const bool>(details).ptr());
   } else if (type == NotificationType::NAV_ENTRY_COMMITTED) {
     Browser* source_browser = Browser::GetBrowserForController(
         Source<NavigationController>(source).ptr(), NULL);
     // If the Browser for the keyboard has navigated, hide the keyboard.
     if (source_browser == browser)
       UpdateKeyboardAndLayout(false);
   }
 }
+
+///////////////////////////////////////////////////////////////////////////////
+//
+// TouchBrowserFocusChangeListener
+//
+///////////////////////////////////////////////////////////////////////////////
+
+TouchBrowserFrameView::TouchBrowserFocusChangeListener
+    ::TouchBrowserFocusChangeListener(TouchBrowserFrameView* browser_frame)
+    : touch_browser_frame_(browser_frame) {
+}
+
+void TouchBrowserFrameView::TouchBrowserFocusChangeListener::FocusWillChange(
+    views::View* focused_before, views::View* focused_now) {
+  if (!focused_before || (focused_now && focused_now->GetVirtualKeyboardType()

[oshima, 2011/01/27 19:58:33]

moving the "(forcusd_now && .." to next line would make it more readable.

[varunjain, 2011/01/27 23:01:38]

Done.

+      != focused_before->GetVirtualKeyboardType())) {
+    // TODO: support other types of keyboard.
+    touch_browser_frame_->UpdateKeyboardAndLayout(
+        focused_now->GetVirtualKeyboardType() == views::View::REGULAR_KEYBOARD);
+  }
+}