Simplify handling of BiDi cursor movement

ui/views/touchui/touch_selection_controller_impl_unittest.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 "base/utf_string_conversions.h"
 #include "ui/gfx/point.h"
 #include "ui/gfx/rect.h"
 #include "ui/gfx/render_text.h"
 #include "ui/views/controls/textfield/native_textfield_views.h"
 #include "ui/views/controls/textfield/textfield.h"
@@ skipping 131 matching lines @@
   VerifySelectionHandlePositions(false);
 }
 
 // Tests that the selection handles are placed appropriately in bidi text.
 TEST_F(TouchSelectionControllerImplTest, SelectionInBidiTextfieldTest) {
   CreateTextfield();
   textfield_->SetText(WideToUTF16(L"abc\x05d0\x05d1\x05d2"));
 
   // Test cursor at run boundary and with empty selection.
   textfield_->SelectSelectionModel(
-      gfx::SelectionModel(3, 2, gfx::SelectionModel::TRAILING));
+      gfx::SelectionModel(3, gfx::CURSOR_PREVIOUS));

[xji, 2012/02/16 02:38:29]

s/CURSOR_PREVIOUS/CURSOR_BACKWARD/

[benrg, 2012/02/16 09:20:55]

Done. I think this entire file may be dead. It doesn't seem to be mentioned in
any .gyp file.

[xji, 2012/02/16 19:52:55]

it is disabled now.

   VerifySelectionHandlePositions(false);
 
   // Test selection range inside one run and starts or ends at run boundary.
   textfield_->SelectRange(ui::Range(2, 3));
   VerifySelectionHandlePositions(false);
 
   textfield_->SelectRange(ui::Range(3, 2));
   VerifySelectionHandlePositions(false);
 
   textfield_->SelectRange(ui::Range(3, 4));
@@ skipping 169 matching lines @@
   VerifySelectionHandlePositions(true);
 
   // Drag selection handle 2 to right by 1 char.
   x = textfield_->font().GetStringWidth(WideToUTF16(L"\x05e2"));
   SimulateSelectionHandleDrag(gfx::Point(x, 0), 2);
   EXPECT_EQ(WideToUTF16(L"c\x05e1"), textfield_->GetSelectedText());
   VerifySelectionHandlePositions(false);
 }
 
 }  // namespace views