[Textfield] Consider select all status for select all context menu

ui/views/controls/textfield/textfield_unittest.cc

Index: ui/views/controls/textfield/textfield_unittest.cc
diff --git a/ui/views/controls/textfield/textfield_unittest.cc b/ui/views/controls/textfield/textfield_unittest.cc
index 71d3ad8ca556465e47b116b7506401dc885914d7..915ad06138f2426b7b52b042e727a42f63edfad6 100644
--- a/ui/views/controls/textfield/textfield_unittest.cc
+++ b/ui/views/controls/textfield/textfield_unittest.cc
@@ -948,6 +948,21 @@ TEST_F(TextfieldTest, LineSelection) {
   EXPECT_FALSE(textfield_->GetSelectedRange().is_reversed());
 }
 
+TEST_F(TextfieldTest, IsAllSelected) {
+  InitTextfield();
+  EXPECT_FALSE(textfield_->IsAllSelected());
+
+  textfield_->SetText(ASCIIToUTF16("12 34567 89"));
+  textfield_->SelectAll(false);
+  EXPECT_TRUE(textfield_->IsAllSelected());
+
+  textfield_->SetSelectionRange(gfx::Range(0, 2));
+  EXPECT_FALSE(textfield_->IsAllSelected());
+
+  textfield_->SetText(base::string16());

[Peter Kasting, 2017/04/20 00:08:30]

Maybe after this we should SelectAll(), before checking IsAllSelected().

[simonhong, 2017/04/20 13:11:51]

We don't need to that because when different text is set, previous selection is
cleared.
Anyway, this test case was removed because Textfield::IsAllSelected() doesn't
exist anymore.

+  EXPECT_FALSE(textfield_->IsAllSelected());
+}
+
 TEST_F(TextfieldTest, MoveUpDownAndModifySelection) {
   InitTextfield();
   textfield_->SetText(ASCIIToUTF16("12 34567 89"));