From b2725756d6f40456048f052427746f347dcd8219 Mon Sep 17 00:00:00 2001 From: "twiz@google.com" Date: Mon, 16 Nov 2009 23:28:26 +0000 Subject: Refactoring of the chrome.experimental.popup API implementation to allow display of pop-ups for extensions viewed through a tab-contents view.I added a new class, ExtensionPopupHost. This class implements the necessary environment for managing child popup windows from either an ExtensionHost, or an ExtensionDOMUI. Note that this class is added as a member to ExtensionHost and ExtensionDOMUI. I decided to take this approach to prevent multiple inheritance of the NotificationObserver class: Both ExtensionPopupHost and ExtensionHost must inherit from this class, and I was uncertain of how the system would behave wrt virtual inheritance. Please comment on if I should have used the inheritance approach. I also removed the customHandler tag (in extension_api.json) that I had added in the initial submission. The arguments in the schema are now those that users of the API see. The nodocs tags were also removed. The api experimental.popup.getAnchorWindow() has been renamed to popup.getParentWindow, as per a suggestion from Erik K. BUG=none TEST=extension_popup_apitest.cc Review URL: http://codereview.chromium.org/385061 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32120 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/chrome.gyp | 2 ++ 1 file changed, 2 insertions(+) (limited to 'chrome/chrome.gyp') diff --git a/chrome/chrome.gyp b/chrome/chrome.gyp index 3ecd885..4b3b096 100755 --- a/chrome/chrome.gyp +++ b/chrome/chrome.gyp @@ -1412,6 +1412,8 @@ 'browser/extensions/extension_page_actions_module_constants.h', 'browser/extensions/extension_popup_api.cc', 'browser/extensions/extension_popup_api.h', + 'browser/extensions/extension_popup_host.cc', + 'browser/extensions/extension_popup_host.h', 'browser/extensions/extension_prefs.cc', 'browser/extensions/extension_prefs.h', 'browser/extensions/extension_process_manager.cc', -- cgit v1.1