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

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

 // 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.
 
 #ifndef CHROME_BROWSER_UI_TOUCH_FRAME_TOUCH_BROWSER_FRAME_VIEW_H_
 #define CHROME_BROWSER_UI_TOUCH_FRAME_TOUCH_BROWSER_FRAME_VIEW_H_
 #pragma once
 
 #include "chrome/browser/ui/views/frame/opaque_browser_frame_view.h"
 #include "chrome/common/notification_observer.h"
 #include "chrome/common/notification_registrar.h"
+#include "views/focus/focus_manager.h"
 
 class BrowserFrame;
 class BrowserView;
 class KeyboardContainerView;
 class NotificationDetails;
 class NotificationSource;
 
 class TouchBrowserFrameView : public OpaqueBrowserFrameView,
                               public NotificationObserver {
  public:
+
+  // A focus listener that performs touch specific tasks on focus change such
+  // as showing/hiding virtual keyboard.
+  class TouchBrowserFocusChangeListener : public views::FocusChangeListener {

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

It seems a more common idiom might be to make TouchBrowserFrameView a
views::FocusChangeListener.

That would also allow you to keep UpdateKeyboardAndLayout private, which I would
prefer.

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

Done.

+   public:
+    TouchBrowserFocusChangeListener(TouchBrowserFrameView* browser_frame);

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

explicit

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

not needed anymore.

+
+    // views::FocusChangeListener implementation
+    virtual void FocusWillChange(views::View* focused_before,
+        views::View* focused_now);

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

indent should be either
function(a,
         b)
function(
  a,
  b);
 or
function(
  a , b);
(your pick)

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

Done.

+
+   private:
+    TouchBrowserFrameView* touch_browser_frame_;

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

DISALLOW_COPY_AND_ASSIGN

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

not needed anymore

+  };

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

Does this needs to be here? If not, I'd prefer to make this
private to implementation.

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

not needed anymore 

+
   // Constructs a non-client view for an BrowserFrame.
   TouchBrowserFrameView(BrowserFrame* frame, BrowserView* browser_view);
   virtual ~TouchBrowserFrameView();
 
   // Overridden from OpaqueBrowserFrameView
   virtual void Layout();
+  virtual void UpdateKeyboardAndLayout(bool should_show_keyboard);
 
  protected:
   // Overridden from OpaqueBrowserFrameView
   virtual int GetReservedHeight() const;
+  virtual void ViewHierarchyChanged(bool is_add, View* parent, View* child);
 
  private:
   virtual void InitVirtualKeyboard();
-  virtual void UpdateKeyboardAndLayout(bool should_show_keyboard);
 
   // Overridden from NotificationObserver.
   virtual void Observe(NotificationType type,
                        const NotificationSource& source,
                        const NotificationDetails& details);
 
   bool keyboard_showing_;
   KeyboardContainerView* keyboard_;
   NotificationRegistrar registrar_;
+  scoped_ptr<TouchBrowserFocusChangeListener> focus_change_listener_;

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

can this be just a FocusChangeListener?

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

not needed anymore 

 
   DISALLOW_COPY_AND_ASSIGN(TouchBrowserFrameView);
 };
 
 #endif  // CHROME_BROWSER_UI_TOUCH_FRAME_TOUCH_BROWSER_FRAME_VIEW_H_