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

Side by Side Diff: chrome/browser/ui/views/autofill/autofill_popup_view_views.h

Issue 2727233003: Uses child views in Autofill Popup so we can trigger (Closed)
Patch Set: Removes NotifyAccessibilityEventForRow and InvalidateRow wrapper methods. Created 3 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_VIEWS_AUTOFILL_AUTOFILL_POPUP_VIEW_VIEWS_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_AUTOFILL_AUTOFILL_POPUP_VIEW_VIEWS_H_
6 #define CHROME_BROWSER_UI_VIEWS_AUTOFILL_AUTOFILL_POPUP_VIEW_VIEWS_H_ 6 #define CHROME_BROWSER_UI_VIEWS_AUTOFILL_AUTOFILL_POPUP_VIEW_VIEWS_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
11 #include "chrome/browser/ui/autofill/autofill_popup_view.h" 11 #include "chrome/browser/ui/autofill/autofill_popup_view.h"
12 #include "chrome/browser/ui/views/autofill/autofill_popup_base_view.h" 12 #include "chrome/browser/ui/views/autofill/autofill_popup_base_view.h"
13 13
14 namespace autofill { 14 namespace autofill {
15 15
16 class AutofillPopupController; 16 class AutofillPopupController;
17 17
18 // Views toolkit implementation for AutofillPopupView. 18 // Views toolkit implementation for AutofillPopupView.
19 class AutofillPopupViewViews : public AutofillPopupBaseView, 19 class AutofillPopupViewViews : public AutofillPopupBaseView,
20 public AutofillPopupView { 20 public AutofillPopupView {
21 public: 21 public:
22 // |controller| should not be null.
22 AutofillPopupViewViews(AutofillPopupController* controller, 23 AutofillPopupViewViews(AutofillPopupController* controller,
23 views::Widget* parent_widget); 24 views::Widget* parent_widget);
25 ~AutofillPopupViewViews() override;
24 26
25 private: 27 private:
26 ~AutofillPopupViewViews() override; 28 FRIEND_TEST_ALL_PREFIXES(AutofillPopupViewViewsTest, OnSelectedRowChanged);
27 29
28 // AutofillPopupView implementation. 30 // AutofillPopupView implementation.
29 void Show() override; 31 void Show() override;
30 void Hide() override; 32 void Hide() override;
31 void InvalidateRow(size_t row) override; 33 void OnSelectedRowChanged(size_t previous_row_selection,
32 void UpdateBoundsAndRedrawPopup() override; 34 size_t current_row_selection) override;
35 void OnSuggestionsChanged() override;
33 36
34 // views::Views implementation 37 // views::Views implementation
35 void OnPaint(gfx::Canvas* canvas) override; 38 void OnPaint(gfx::Canvas* canvas) override;
39 void GetAccessibleNodeData(ui::AXNodeData* node_data) override;
36 40
37 // Draw the given autofill entry in |entry_rect|. 41 // Draw the given autofill entry in |entry_rect|.
38 void DrawAutofillEntry(gfx::Canvas* canvas, 42 void DrawAutofillEntry(gfx::Canvas* canvas,
39 int index, 43 int index,
40 const gfx::Rect& entry_rect); 44 const gfx::Rect& entry_rect);
41 45
46 // Creates child views based on the suggestions given by |controller_|. These
47 // child views are used for accessibility events only. We need child views to
48 // populate the correct |AXNodeData| when user selects a suggestion.
49 void CreateChildViews();
50
42 AutofillPopupController* controller_; // Weak reference. 51 AutofillPopupController* controller_; // Weak reference.
43 52
44 DISALLOW_COPY_AND_ASSIGN(AutofillPopupViewViews); 53 DISALLOW_COPY_AND_ASSIGN(AutofillPopupViewViews);
45 }; 54 };
46 55
47 } // namespace autofill 56 } // namespace autofill
48 57
49 #endif // CHROME_BROWSER_UI_VIEWS_AUTOFILL_AUTOFILL_POPUP_VIEW_VIEWS_H_ 58 #endif // CHROME_BROWSER_UI_VIEWS_AUTOFILL_AUTOFILL_POPUP_VIEW_VIEWS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698