MacViews: Implement Full Keyboard Access.

ui/views/controls/button/image_button.h

 // 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.
 
 #ifndef UI_VIEWS_CONTROLS_BUTTON_IMAGE_BUTTON_H_
 #define UI_VIEWS_CONTROLS_BUTTON_IMAGE_BUTTON_H_
 
 #include <memory>
 
 #include "base/gtest_prod_util.h"
 #include "base/macros.h"
 #include "ui/base/layout.h"
 #include "ui/gfx/image/image_skia.h"
 #include "ui/views/controls/button/custom_button.h"
 
 namespace views {
 
 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[];
 
   enum HorizontalAlignment {
     ALIGN_LEFT = 0,
     ALIGN_CENTER,
     ALIGN_RIGHT
   };
 
@@ skipping 127 matching lines @@
   // The parent class's tooltip_text_ is displayed when not toggled, and
   // this one is shown when toggled.
   base::string16 toggled_tooltip_text_;
 
   DISALLOW_COPY_AND_ASSIGN(ToggleImageButton);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_CONTROLS_BUTTON_IMAGE_BUTTON_H_