Reverse OmniboxViewViews::HandleMouseReleaseEvent selection.

chrome/browser/ui/views/omnibox/omnibox_view_views.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_views.h"
 
 #include "base/logging.h"
 #include "base/property_bag.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
@@ skipping 379 matching lines @@
       !textfield_->HasFocus();
 }
 
 void OmniboxViewViews::HandleMouseDragEvent(const views::MouseEvent& event) {
   select_all_on_mouse_release_ = false;
 }
 
 void OmniboxViewViews::HandleMouseReleaseEvent(const views::MouseEvent& event) {
   if ((event.IsOnlyLeftMouseButton() || event.IsOnlyRightMouseButton()) &&
       select_all_on_mouse_release_) {
-    textfield_->SelectAll();
+    // Select-all in reverse to show the leading omnibox text on mouse release.

[Peter Kasting, 2012/07/11 20:49:17]

Nit: The Windows comment is:

    // Select all in the reverse direction so as not to scroll the caret
    // into view and shift the contents jarringly.

[msw, 2012/07/11 21:46:29]

Done.

+    SelectAll(true);
   }
   select_all_on_mouse_release_ = false;
 }
 
 void OmniboxViewViews::HandleFocusIn() {
   // TODO(oshima): Get control key state.
   model_->OnSetFocus(false);
   // Don't call controller_->OnSetFocus as this view has already
   // acquired the focus.
 }
@@ skipping 532 matching lines @@
                                                const ui::Range& range) {
   if (text != GetText())
     textfield_->SetText(text);
   textfield_->SelectRange(range);
 }
 
 string16 OmniboxViewViews::GetSelectedText() const {
   // TODO(oshima): Support instant, IME.
   return textfield_->GetSelectedText();
 }