Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(26)

Unified Diff: chrome/browser/ui/views/autofill/autofill_popup_view_views.h

Issue 2670003002: Accessibility for Autofill Popup View in native code (Closed)
Patch Set: Switches to CHECK to ensure crash is caught early. Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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.
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,

Powered by Google App Engine
This is Rietveld 408576698