Chromium Code Reviews| Index: chrome/browser/ui/cocoa/autofill/password_generation_popup_view_bridge.mm |
| diff --git a/chrome/browser/ui/cocoa/autofill/password_generation_popup_view_bridge.mm b/chrome/browser/ui/cocoa/autofill/password_generation_popup_view_bridge.mm |
| index 6c22cb8bb63cad3504e9ea8fe2c4f0915c237b54..9ab0ec5b8b92031425cf7589988d4c8d5622a025 100644 |
| --- a/chrome/browser/ui/cocoa/autofill/password_generation_popup_view_bridge.mm |
| +++ b/chrome/browser/ui/cocoa/autofill/password_generation_popup_view_bridge.mm |
| @@ -8,9 +8,10 @@ |
| #include "base/logging.h" |
| #include "chrome/browser/ui/autofill/autofill_popup_controller.h" |
| +#include "chrome/browser/ui/autofill/password_generation_popup_controller.h" |
| +#include "chrome/browser/ui/autofill/popup_constants.h" |
|
groby-ooo-7-16
2014/08/21 04:40:05
I believe both of these are unnecessary?
dconnelly
2014/08/21 10:44:25
Done.
|
| #import "chrome/browser/ui/cocoa/autofill/password_generation_popup_view_cocoa.h" |
| -#include "ui/base/cocoa/window_size_constants.h" |
| -#include "ui/gfx/rect.h" |
| +#include "ui/gfx/size.h" |
| namespace autofill { |
| @@ -36,8 +37,7 @@ void PasswordGenerationPopupViewBridge::Show() { |
| } |
| gfx::Size PasswordGenerationPopupViewBridge::GetPreferredSizeOfPasswordView() { |
| - // TODO(gcasto): Implement this function. |
| - return gfx::Size(); |
| + return gfx::Size(NSSizeToCGSize([view_ preferredSize])); |
| } |
| void PasswordGenerationPopupViewBridge::UpdateBoundsAndRedrawPopup() { |
| @@ -50,8 +50,7 @@ void PasswordGenerationPopupViewBridge::PasswordSelectionUpdated() { |
| bool PasswordGenerationPopupViewBridge::IsPointInPasswordBounds( |
| const gfx::Point& point) { |
| - // TODO(gcasto): Implement this function. |
| - return true; |
| + return [view_ isPointInPasswordBounds:NSPointFromCGPoint(point.ToCGPoint())]; |
| } |
| PasswordGenerationPopupView* PasswordGenerationPopupView::Create( |