diff options
author | twiz@google.com <twiz@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-16 23:28:26 +0000 |
---|---|---|
committer | twiz@google.com <twiz@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-16 23:28:26 +0000 |
commit | b2725756d6f40456048f052427746f347dcd8219 (patch) | |
tree | cf02ff046b428d9418291f0f7c6376ac71bbb1ea /chrome/chrome.gyp | |
parent | ba1d6b0893d6d07cf1fb924f4212446d4e6136d4 (diff) | |
download | chromium_src-b2725756d6f40456048f052427746f347dcd8219.zip chromium_src-b2725756d6f40456048f052427746f347dcd8219.tar.gz chromium_src-b2725756d6f40456048f052427746f347dcd8219.tar.bz2 |
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
Diffstat (limited to 'chrome/chrome.gyp')
-rwxr-xr-x | chrome/chrome.gyp | 2 |
1 files changed, 2 insertions, 0 deletions
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', |