Fix omnibox mouse click highlight/word select/focus issue in aura build.

views/controls/textfield/native_textfield_views.cc

 // Copyright (c) 2011 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 "views/controls/textfield/native_textfield_views.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 76 matching lines @@
   set_drag_controller(this);
 }
 
 NativeTextfieldViews::~NativeTextfieldViews() {
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NativeTextfieldViews, View overrides:
 
 bool NativeTextfieldViews::OnMousePressed(const MouseEvent& event) {
+  if (textfield_->OnMousePressed(event)) {

[msw, 2011/11/11 01:28:58]

Please add a comment here like "Allow the textfield/omnibox to optionally handle
the mouse pressed event." Also, remove the braces for a single statement.

[jennyz, 2011/11/12 01:50:17]

Done.

+    return true;
+  }

[oshima, 2011/11/11 01:52:32]

This should be after OnBeforeUserAction() and has to
call OnAfteruserAction before it returns true.
I also think you need to update last_click_time_ to get double/triple clicking
work correctly.

Please mention that this should be removed once native textfield impls are
consolidated to textfield.

[jennyz, 2011/11/12 01:50:17]

Done.

+
   OnBeforeUserAction();
   textfield_->RequestFocus();
 
   if (event.IsOnlyLeftMouseButton()) {
     base::TimeDelta time_delta = event.time_stamp() - last_click_time_;
     gfx::Point location_delta = event.location().Subtract(last_click_location_);
     if (time_delta.InMilliseconds() <= GetDoubleClickInterval() &&
         !ExceededDragThreshold(location_delta.x(), location_delta.y())) {
       aggregated_clicks_ = (aggregated_clicks_ + 1) % 3;
     } else {
@@ skipping 971 matching lines @@
 
 #if defined(USE_AURA)
 // static
 NativeTextfieldWrapper* NativeTextfieldWrapper::CreateWrapper(
     Textfield* field) {
   return new NativeTextfieldViews(field);
 }
 #endif
 
 }  // namespace views