Redesign: Remove TsfEventRouter interface.

chrome/browser/ui/views/omnibox/omnibox_view_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 "chrome/browser/ui/views/omnibox/omnibox_view_win.h"
 
 #include <algorithm>
 #include <locale>
 #include <string>
 
@@ skipping 462 matching lines @@
       font_(parent_view->font()),
       possible_drag_(false),
       in_drag_(false),
       initiated_drag_(false),
       drop_highlight_position_(-1),
       ime_candidate_window_open_(false),
       background_color_(skia::SkColorToCOLORREF(LocationBarView::GetColor(
           chrome::search::IsInstantExtendedAPIEnabled(parent_view_->profile()),
           ToolbarModel::NONE, LocationBarView::BACKGROUND))),
       security_level_(ToolbarModel::NONE),
-      text_object_model_(NULL) {
+      text_object_model_(NULL),
+      tsf_event_router_(base::win::IsTsfAwareRequired()
+          ? new ui::TsfEventRouter() : NULL) {

[Peter Kasting, 2012/10/24 20:48:11]

Nit: '?' goes on previous line

[Seigo Nonaka, 2012/10/25 14:37:16]

Done.

   if (!loaded_library_module_)
     loaded_library_module_ = LoadLibrary(kRichEditDLLName);
 
   saved_selection_for_focus_change_.cpMin = -1;
 
   g_paint_patcher.Pointer()->RefPatch();
 
   Create(location_bar->GetWidget()->GetNativeView(), 0, 0, 0,
          l10n_util::GetExtendedStyles());
   SetReadOnly(popup_window_mode_);
@@ skipping 40 matching lines @@
     // Non-read-only edit controls have a drop target.  Revoke it so that we can
     // install our own.  Revoking automatically deletes the existing one.
     HRESULT hr = RevokeDragDrop(m_hWnd);
     DCHECK_EQ(S_OK, hr);
 
     // Register our drop target.  The scoped_refptr here will delete the drop
     // target if it fails to register itself correctly on |m_hWnd|.  Otherwise,
     // the edit control will invoke RevokeDragDrop when it's being destroyed, so
     // we don't have to do so.
     scoped_refptr<EditDropTarget> drop_target(new EditDropTarget(this));
-
-    if (base::win::IsTsfAwareRequired())
-      tsf_event_router_ = ui::TsfEventRouter::Create();
   }
 }
 
 OmniboxViewWin::~OmniboxViewWin() {
   // Explicitly release the text object model now that we're done with it, and
   // before we free the library. If the library gets unloaded before this
   // released, it becomes garbage. Note that since text_object_model_ is lazy
   // initialized, it may still be null.
   if (text_object_model_)
     text_object_model_->Release();
@@ skipping 2200 matching lines @@
   return (rect.left - client_rect.left) + (client_rect.right - rect.right);
 }
 
 int OmniboxViewWin::WidthNeededToDisplay(const string16& text) const {
   // Use font_.GetStringWidth() instead of
   // PosFromChar(location_entry_->GetTextLength()) because PosFromChar() is
   // apparently buggy. In both LTR UI and RTL UI with left-to-right layout,
   // PosFromChar(i) might return 0 when i is greater than 1.
   return font_.GetStringWidth(text) + GetHorizontalMargin();
 }