Removes the redundant IMB::OnFocus call in DNWA::HandleNativeFocus().

ui/views/widget/desktop_aura/desktop_window_tree_host_win.cc

 // Copyright (c) 2012 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 "ui/views/widget/desktop_aura/desktop_window_tree_host_win.h"
 
 #include "base/win/metro.h"
 #include "third_party/skia/include/core/SkPath.h"
 #include "third_party/skia/include/core/SkRegion.h"
 #include "ui/aura/client/aura_constants.h"
@@ skipping 801 matching lines @@
     OnHostResized(new_size);
 }
 
 void DesktopWindowTreeHostWin::HandleFrameChanged() {
   SetWindowTransparency();
   // Replace the frame and layout the contents.
   GetWidget()->non_client_view()->UpdateFrame();
 }
 
 void DesktopWindowTreeHostWin::HandleNativeFocus(HWND last_focused_window) {
   // TODO(beng): inform the native_widget_delegate_.

[sky, 2015/05/22 17:29:02]

I think it's worth a comment here as to why you don't need to call to
InputMethod.

[Shu Chen, 2015/05/25 01:14:25]

Thanks for your comment. My later cl soon to refactor the IMF (ownership change
for InputMethod) will make window tree host to call OnFocus/OnBlur of
InputMethod.

-  InputMethod* input_method = GetInputMethod();
-  if (input_method)
-    input_method->OnFocus();
 }
 
 void DesktopWindowTreeHostWin::HandleNativeBlur(HWND focused_window) {
   // TODO(beng): inform the native_widget_delegate_.
-  InputMethod* input_method = GetInputMethod();
-  if (input_method)
-    input_method->OnBlur();
 }

[sadrul, 2015/05/16 02:24:25]

The X11 version seems to still do this (e.g.
DesktopWindowTreeHostX11::OnNativeWidgetFocus(), OnNativeWidgetBlur()). Should
those be removed too?

[Shu Chen, 2015/05/20 06:30:52]

Let's limit the scope of this change in Windows platform. Could change for X11
separately if we confirm this fix is correct and DWTHX11::OnNativeWidgetFocus()
can be removed.

 
 bool DesktopWindowTreeHostWin::HandleMouseEvent(const ui::MouseEvent& event) {
   SendEventToProcessor(const_cast<ui::MouseEvent*>(&event));
   return event.handled();
 }
 
 bool DesktopWindowTreeHostWin::HandleKeyEvent(const ui::KeyEvent& event) {
   return false;
 }
 
@@ skipping 148 matching lines @@
 
 // static
 DesktopWindowTreeHost* DesktopWindowTreeHost::Create(
     internal::NativeWidgetDelegate* native_widget_delegate,
     DesktopNativeWidgetAura* desktop_native_widget_aura) {
   return new DesktopWindowTreeHostWin(native_widget_delegate,
                                       desktop_native_widget_aura);
 }
 
 }  // namespace views