MacViews: Implement Full Keyboard Access.

ui/views/controls/button/image_button.h

Index: ui/views/controls/button/image_button.h
diff --git a/ui/views/controls/button/image_button.h b/ui/views/controls/button/image_button.h
index 79e84d7d3e1a16682433ef83c118035ecbb10be1..9131ae25aaa7a77ccd41f24ba014913af266cd35 100644
--- a/ui/views/controls/button/image_button.h
+++ b/ui/views/controls/button/image_button.h
@@ -20,9 +20,8 @@ class Painter;
 // An image button.
 

[tapted, 2016/04/20 06:05:58]

this line should either be removed or be just `//` by itself

[karandeepb, 2016/05/03 02:54:12]

Sorry it isn't clear to me which line you are talking about.

[tapted, 2016/05/03 08:08:25]

The comment "// An image button." is currently 'detached' from its class because
line 21 is blank. It should be formatted like


// An image button.
//
// Note that....
class .. ImageButton

or

// An image button.
// Note that....
class .. ImageButton

[karandeepb, 2016/05/04 01:56:38]

Done.

 // Note that this type of button is not focusable by default and will not be
-// part of the focus chain.  Call SetFocusBehavior(FocusBehavior::ALWAYS) to
-// make it part of the focus chain.
-
+// part of the focus chain, unless in accessibility mode. Call
+// Button::ConfigureDefaultFocus() to make it part of the focus chain.
 class VIEWS_EXPORT ImageButton : public CustomButton {
  public:
   static const char kViewClassName[];