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

ui/views/controls/textfield/textfield.h

Index: ui/views/controls/textfield/textfield.h
diff --git a/ui/views/controls/textfield/textfield.h b/ui/views/controls/textfield/textfield.h
index 358a248140cdc44189647f8c7eadb290a17c55cf..acb71b46205ad8f2b5484bd96072749602287561 100644
--- a/ui/views/controls/textfield/textfield.h
+++ b/ui/views/controls/textfield/textfield.h
@@ -105,6 +105,9 @@ class VIEWS_EXPORT Textfield : public View,
   // of text that overflows its display area.
   void SelectAll(bool reversed);
 
+  // Return false when text is empty.

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

Nit: "Returns whether all the text is currently selected.  Returns false if
there is no text."

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

Removed.

+  bool IsAllSelected() const;

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

Nit: Does this need to be public?  If we're only using it internally, why not
make it private?  If it's only being called in one place, why not just inline
the implementation there?  Or are you planning on using this in other places?

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

Making it public is mistake.
At first, I added as a method for test purpose.
However, I convinced that we can test without this method.
Inlined.

+
   // A convenience method to select the word closest to |point|.
   void SelectWordAt(const gfx::Point& point);