Mode Indicator using BubbleDelegateView.

chrome/browser/chromeos/input_method/mode_indicator_controller.cc

 // Copyright 2013 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/chromeos/input_method/mode_indicator_controller.h"
+
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/chromeos/input_method/input_method_util.h"
-#include "chrome/browser/chromeos/input_method/mode_indicator_controller.h"
-#include "chrome/browser/chromeos/input_method/mode_indicator_widget.h"
+#include "chrome/browser/chromeos/input_method/mode_indicator_delegate_view.h"
 #include "chromeos/chromeos_switches.h"
 
 namespace chromeos {
 namespace input_method {
 
-ModeIndicatorController::ModeIndicatorController(
-    ModeIndicatorWidget* mi_widget)
-    : is_focused_(false) {
-  mi_widget_.reset(mi_widget);
+class ModeIndicatorObserver : public views::WidgetObserver {

[Jun Mukai, 2013/12/02 21:08:41]

What is this class for?  It just observes the widget but only the observing
method is OnWidgetDestroying which just cancels active_widget_.

[Hiro Komatsu, 2013/12/03 02:39:36]

This is to close the previous widget immediately without fade-out animation. 
Added comment.

+ public:
+  ModeIndicatorObserver()
+    : active_widget_(NULL) {}
 
-  InputMethodManager* imm = InputMethodManager::Get();
-  DCHECK(imm);
-  imm->AddObserver(this);
+  virtual ~ModeIndicatorObserver() {}
+
+  // views::WidgetObserver override:
+  virtual void OnWidgetDestroying(views::Widget* widget) OVERRIDE {

[Jun Mukai, 2013/12/02 21:08:41]

Please move below. Normal order is: ctor/dtor -> normal methods -> inherited.

[Hiro Komatsu, 2013/12/03 02:39:36]

Done.

+    if (widget == active_widget_)
+      active_widget_ = NULL;
+  }
+
+  // If other active mode indicator widget is shown, close it immedicately.
+  // Then store this widget as the active widget.
+  void UpdateActiveModeIndicator(views::Widget* widget) {
+    if (active_widget_)
+      active_widget_->Close();
+    active_widget_ = widget;
+    widget->AddObserver(this);
+  }
+
+ private:
+  views::Widget* active_widget_;
+};
+
+
+ModeIndicatorController::ModeIndicatorController(InputMethodManager* imm)
+    : imm_(imm),
+      is_focused_(false),
+      mi_observer_(new ModeIndicatorObserver) {
+  DCHECK(imm_);
+  imm_->AddObserver(this);
 }
 
 ModeIndicatorController::~ModeIndicatorController() {
-  InputMethodManager* imm = InputMethodManager::Get();
-  DCHECK(imm);
-  imm->RemoveObserver(this);
+  imm_->RemoveObserver(this);
 }
 
 void ModeIndicatorController::SetCursorBounds(
     const gfx::Rect& cursor_bounds) {
-  mi_widget_->SetCursorBounds(cursor_bounds);
+  cursor_bounds_ = cursor_bounds;
 }
 
 void ModeIndicatorController::FocusStateChanged(bool is_focused) {
   is_focused_ = is_focused;
 }
 
 void ModeIndicatorController::InputMethodChanged(InputMethodManager* manager,
                                                  bool show_message) {
   if (!show_message)
     return;
 
-  ShowModeIndicator(manager);
+  ShowModeIndicator();
 }
 
 void ModeIndicatorController::InputMethodPropertyChanged(
     InputMethodManager* manager) {
   // Do nothing.
 }
 
-void ModeIndicatorController::ShowModeIndicator(InputMethodManager* manager) {
+void ModeIndicatorController::ShowModeIndicator() {
   // TODO(komatsu): When this is permanently enabled by defalut,
   // delete command_line.h and chromeos_switches.h from the header
   // files.
   if (CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kDisableIMEModeIndicator))
     return;
 
   // TODO(komatsu): Show the mode indicator in the right bottom of the
   // display when the launch bar is hidden and the focus is out.  To
   // implement it, we should consider to use message center or system
   // notification.  Note, launch bar can be vertical and can be placed
   // right/left side of display.
   if (!is_focused_)
     return;
 
-  DCHECK(manager);
-  DCHECK(mi_widget_.get());
+  // Get the short name of the changed input method (e.g. US, JA, etc.)
+  DCHECK(imm_);

[Jun Mukai, 2013/12/02 21:08:41]

You don't have to DCHECK, because imm_ is set in ctor and you already checked.

[Hiro Komatsu, 2013/12/03 02:39:36]

Done.

+  const InputMethodDescriptor descriptor = imm_->GetCurrentInputMethod();
+  const string16 short_name =
+      imm_->GetInputMethodUtil()->GetInputMethodShortName(descriptor);
 
-  // Get the short name of the changed input method (e.g. US, JA, etc.)
-  const InputMethodDescriptor descriptor = manager->GetCurrentInputMethod();
-  const std::string short_name = UTF16ToUTF8(
-      manager->GetInputMethodUtil()->GetInputMethodShortName(descriptor));
-  mi_widget_->SetLabelTextUtf8(short_name);
-
-  // Show the widget and hide it after 750msec.
-  mi_widget_->Show();
-  const int kDelayMSec = 750;
-  mi_widget_->DelayHide(kDelayMSec);
+  ModeIndicatorDelegateView* mi_delegate_view =
+      new ModeIndicatorDelegateView(cursor_bounds_, short_name);
+  views::BubbleDelegateView::CreateBubble(mi_delegate_view);
+  mi_observer_->UpdateActiveModeIndicator(mi_delegate_view->GetWidget());
+  mi_delegate_view->ShowAndFadeOut();
 }
 
 }  // namespace input_method
 }  // namespace chromeos