Chromium Code Reviews| Index: chrome/browser/ui/cocoa/autofill/autofill_popup_view_bridge.h |
| diff --git a/chrome/browser/ui/cocoa/autofill/autofill_popup_view_bridge.h b/chrome/browser/ui/cocoa/autofill/autofill_popup_view_bridge.h |
| index 5867e2be8d1f0bf960e7d2bcc91321fd425b2ebc..247314b3fa72b7cc3ed534a2e7cceae57dafa9a1 100644 |
| --- a/chrome/browser/ui/cocoa/autofill/autofill_popup_view_bridge.h |
| +++ b/chrome/browser/ui/cocoa/autofill/autofill_popup_view_bridge.h |
| @@ -12,8 +12,10 @@ |
| #include "base/compiler_specific.h" |
| #include "base/mac/scoped_nsobject.h" |
| #include "base/macros.h" |
| +#include "base/optional.h" |
| #include "chrome/browser/ui/autofill/autofill_popup_view.h" |
| #include "chrome/browser/ui/cocoa/autofill/autofill_popup_view_cocoa.h" |
| +#include "ui/accessibility/ax_enums.h" |
|
groby-ooo-7-16
2017/03/20 22:53:10
Probably not needed.
csashi
2017/03/20 23:13:30
Done.
|
| @class AutofillPopupViewCocoa; |
| @class NSWindow; |
| @@ -24,7 +26,7 @@ class AutofillPopupViewCocoaDelegate { |
| public: |
| // Returns the bounds of the item at |index| in the popup, relative to |
| // the top left of the popup. |
| - virtual gfx::Rect GetRowBounds(size_t index) = 0; |
| + virtual gfx::Rect GetRowBounds(int index) = 0; |
| // Gets the resource value for the given resource, returning -1 if the |
| // resource isn't recognized. |
| @@ -40,7 +42,7 @@ class AutofillPopupViewBridge : public AutofillPopupView, |
| explicit AutofillPopupViewBridge(AutofillPopupController* controller); |
| // AutofillPopupViewCocoaDelegate implementation. |
| - gfx::Rect GetRowBounds(size_t index) override; |
| + gfx::Rect GetRowBounds(int index) override; |
| int GetIconResourceID(const base::string16& resource_name) override; |
| private: |
| @@ -49,8 +51,9 @@ class AutofillPopupViewBridge : public AutofillPopupView, |
| // AutofillPopupView implementation. |
| void Hide() override; |
| void Show() override; |
| - void InvalidateRow(size_t row) override; |
| - void UpdateBoundsAndRedrawPopup() override; |
| + void OnSelectedRowChanged(base::Optional<int> previous_row_selection, |
| + base::Optional<int> current_row_selection) override; |
| + void OnSuggestionsChanged() override; |
| // Set the initial bounds of the popup, including its placement. |
| void SetInitialBounds(); |