diff options
| author | ananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-26 22:09:46 +0000 |
|---|---|---|
| committer | ananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-26 22:09:46 +0000 |
| commit | 6fd1a7a0bbac30baf3e8b3dae4b6ca147ae6b0d0 (patch) | |
| tree | 29f4a818624baffec4b18f2c914018a3dfaa25f8 /chrome_frame/chrome_active_document.h | |
| parent | 44693a7a71fa420a541aaa8a9e79c2280fc61bc7 (diff) | |
| download | chromium_src-6fd1a7a0bbac30baf3e8b3dae4b6ca147ae6b0d0.zip chromium_src-6fd1a7a0bbac30baf3e8b3dae4b6ca147ae6b0d0.tar.gz chromium_src-6fd1a7a0bbac30baf3e8b3dae4b6ca147ae6b0d0.tar.bz2 | |
ChromeFrame host browser commands need to account for the command group guid and the command id in all cases. We were
detecting MSHTML command ids based on the command id only while ignoring the group which was wrong.
Part of the fix for http://code.google.com/p/chromium/issues/detail?id=24034
Bug=24034
Review URL: http://codereview.chromium.org/2873070
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53699 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome_frame/chrome_active_document.h')
| -rw-r--r-- | chrome_frame/chrome_active_document.h | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/chrome_frame/chrome_active_document.h b/chrome_frame/chrome_active_document.h index 39b0e74..636d195 100644 --- a/chrome_frame/chrome_active_document.h +++ b/chrome_frame/chrome_active_document.h @@ -433,7 +433,7 @@ END_EXEC_COMMAND_MAP() static bool ShouldFireDocumentComplete(); protected: - typedef std::map<int, bool> EnabledCommandsMap; + typedef std::map<int, OLECMDF> CommandStatusMap; IPC::NavigationInfo navigation_info_; bool is_doc_object_; @@ -446,8 +446,11 @@ END_EXEC_COMMAND_MAP() // Our find dialog CFFindDialog find_dialog_; - // Contains the list of enabled commands ids. - EnabledCommandsMap enabled_commands_map_; + // These members contain the status of the commands we support. + CommandStatusMap null_group_commands_map_; + CommandStatusMap mshtml_group_commands_map_; + CommandStatusMap explorer_group_commands_map_; + CommandStatusMap shdoc_view_group_commands_map_; // Set to true if the automation_client_ member is initialized from // an existing ChromeActiveDocument instance which is going away and |