Views: Add a11y information for IconLabelBubbleView and LocationIconView.

chrome/browser/ui/views/location_bar/icon_label_bubble_view.h

Index: chrome/browser/ui/views/location_bar/icon_label_bubble_view.h
diff --git a/chrome/browser/ui/views/location_bar/icon_label_bubble_view.h b/chrome/browser/ui/views/location_bar/icon_label_bubble_view.h
index 9d1812a681cfa1098835587e146c376161d064fd..627e88862f1bb379ca1c26e4d67ca6ded4f85199 100644
--- a/chrome/browser/ui/views/location_bar/icon_label_bubble_view.h
+++ b/chrome/browser/ui/views/location_bar/icon_label_bubble_view.h
@@ -73,6 +73,9 @@ class IconLabelBubbleView : public views::InkDropHostView {
   bool OnKeyPressed(const ui::KeyEvent& event) override;
   bool OnKeyReleased(const ui::KeyEvent& event) override;
   void OnNativeThemeChanged(const ui::NativeTheme* native_theme) override;
+  void GetAccessibleNodeData(ui::AXNodeData* node_data) override;

[Peter Kasting, 2016/12/06 04:20:28]

Nit: This goes above OnNativeThemeChanged (match superclass declaration order).

[Patti Lor, 2016/12/06 07:07:21]

Thanks for picking that up, fixed!

+
+  // views::InkDropHostView:

[Peter Kasting, 2016/12/06 04:20:28]

Nit: Don't add this separate divider, instead reword the "views::View" comment
above to "views::InkDropHostView", since that's the only direct base class of
this class (it doesn't directly subclass View).

[Patti Lor, 2016/12/06 07:07:21]

Done.

   void AddInkDropLayer(ui::Layer* ink_drop_layer) override;
   void RemoveInkDropLayer(ui::Layer* ink_drop_layer) override;
   std::unique_ptr<views::InkDropHighlight> CreateInkDropHighlight()