[Extensions UI Mac] Move popup close logic to ExtensionPopupController
commit75c54672467984e2f096e2fa7d41b40849bbe2c8
authorrdevlin.cronin <rdevlin.cronin@chromium.org>
Wed, 6 May 2015 00:26:04 +0000 (5 17:26 -0700)
committerCommit bot <commit-bot@chromium.org>
Wed, 6 May 2015 00:27:32 +0000 (6 00:27 +0000)
tree356c3917384ccb05d54b69b1c35f3ffaa525ed80
parent2104ce61878e278ae5e13fb5fc5bb2cb4c9e528f
[Extensions UI Mac] Move popup close logic to ExtensionPopupController

Move the logic to close an extension popup to the ExtensionPopupController.
In addition to being a little cleaner design, this allows us to remove some of
the accessors on the ExtensionPopupController.
This also should fix crbug.com/484538, which I suspect is happening when we
try and close the popup after the extension is removed.

BUG=484538

Review URL: https://codereview.chromium.org/1126823002

Cr-Commit-Position: refs/heads/master@{#328449}
chrome/browser/ui/cocoa/extensions/extension_action_platform_delegate_cocoa.mm
chrome/browser/ui/cocoa/extensions/extension_popup_controller.h
chrome/browser/ui/cocoa/extensions/extension_popup_controller.mm
chrome/browser/ui/extensions/extension_action_view_controller.cc
chrome/browser/ui/extensions/extension_action_view_controller.h