Accessibility for Autofill Popup View in native code

chrome/browser/ui/views/autofill/autofill_popup_view_views.h

Index: chrome/browser/ui/views/autofill/autofill_popup_view_views.h
diff --git a/chrome/browser/ui/views/autofill/autofill_popup_view_views.h b/chrome/browser/ui/views/autofill/autofill_popup_view_views.h
index 01f214b2e1f05b4604a48e686c7252c46741ea07..1714d83b335f19aa329cc3f9524aaedc7dea718b 100644
--- a/chrome/browser/ui/views/autofill/autofill_popup_view_views.h
+++ b/chrome/browser/ui/views/autofill/autofill_popup_view_views.h
@@ -19,6 +19,7 @@ class AutofillPopupController;
 class AutofillPopupViewViews : public AutofillPopupBaseView,
                                public AutofillPopupView {
  public:
+  // |controller| should not be null.

[sky, 2017/02/27 21:05:57]

should->must
But a DCHECK in the constructor makes this more obvious.

   AutofillPopupViewViews(AutofillPopupController* controller,
                          views::Widget* parent_widget);
 
@@ -30,9 +31,12 @@ class AutofillPopupViewViews : public AutofillPopupBaseView,
   void Hide() override;
   void InvalidateRow(size_t row) override;
   void UpdateBoundsAndRedrawPopup() override;
+  void NotifyAccessibilityEventForRow(ui::AXEvent event_type,
+                                      size_t row) override;
 
   // views::Views implementation
   void OnPaint(gfx::Canvas* canvas) override;
+  void GetAccessibleNodeData(ui::AXNodeData* node_data) override;
 
   // Draw the given autofill entry in |entry_rect|.
   void DrawAutofillEntry(gfx::Canvas* canvas,