Deactivate omnibox touch editing on command execution

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/command_line.h"
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/strings/string_util.h"
@@ skipping 298 matching lines @@
   // Returns the width necessary to display the current text, including any
   // necessary space for the cursor or border/margin.
   return GetRenderText()->GetContentWidth() + GetInsets().width();
 }
 
 bool OmniboxViewViews::IsImeComposing() const {
   return IsIMEComposing();
 }
 
 void OmniboxViewViews::ExecuteCommand(int command_id, int event_flags) {
+  DestroyTouchSelection();

[sky, 2014/05/21 16:20:45]

Add a comment as to why the touch selection needs to be destroyed here.

[mohsen, 2014/05/21 19:54:28]

Done.

   switch (command_id) {
     // These commands don't invoke the popup via OnBefore/AfterPossibleChange().
     case IDS_PASTE_AND_GO:
       model()->PasteAndGo(GetClipboardText());
       break;
     case IDS_SHOW_URL:
       controller()->ShowURL();
       break;
     case IDC_EDIT_SEARCH_ENGINES:
       command_updater()->ExecuteCommand(command_id);
@@ skipping 716 matching lines @@
     menu_contents->InsertItemWithStringIdAt(
         select_all_position + 1, IDS_SHOW_URL, IDS_SHOW_URL);
   }
 
   // Minor note: We use IDC_ for command id here while the underlying textfield
   // is using IDS_ for all its command ids. This is because views cannot depend
   // on IDC_ for now.
   menu_contents->AddItemWithStringId(IDC_EDIT_SEARCH_ENGINES,
       IDS_EDIT_SEARCH_ENGINES);
 }