diff options
| author | Maxim Monastirsky <momonasmon@gmail.com> | 2020-09-06 22:15:16 +0300 |
|---|---|---|
| committer | Maxim Monastirsky <momonasmon@gmail.com> | 2020-09-07 00:19:05 +0200 |
| commit | 94a7a71b070d3911b39d1026ba266768b71ba8a6 (patch) | |
| tree | 3aebfa7309204af744d096dc5b816c5b840ccfef /framework | |
| parent | 34f37ce6b9feef6091e0f27a2618e3f812f42008 (diff) | |
MenuBarManager: Actually use xPopupMenuDispatchProvider
i.e. restore the fix for i#65734/i#75851, which for some
reason was disabled in 3c471fa5480d58e867e05a1ef474a23aef94b07d.
This can be tested by inserting chart into Writer or Calc, and
looking at the File menu. Given that this menu is merged from
the container app, it makes sense to use its dispatch provider
for everything, not just for a few commands hardcoded in
CommandDispatchContainer::getDispatchForURL of chart2.
Change-Id: I03f233a84249284354ea709d8aa7c950390294e6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102137
Tested-by: Jenkins
Reviewed-by: Maxim Monastirsky <momonasmon@gmail.com>
Diffstat (limited to 'framework')
| -rw-r--r-- | framework/source/uielement/menubarmanager.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/framework/source/uielement/menubarmanager.cxx b/framework/source/uielement/menubarmanager.cxx index f72e5471d36a..21817bd36b93 100644 --- a/framework/source/uielement/menubarmanager.cxx +++ b/framework/source/uielement/menubarmanager.cxx @@ -1043,7 +1043,7 @@ void MenuBarManager::FillMenuManager( Menu* pMenu, const Reference< XFrame >& rF } MenuBarManager* pSubMenuManager = new MenuBarManager( m_xContext, rFrame, m_xURLTransformer, - rDispatchProvider, aModuleIdentifier, pPopup, false, m_bHasMenuBar ); + xPopupMenuDispatchProvider, aModuleIdentifier, pPopup, false, m_bHasMenuBar ); AddMenu(pSubMenuManager, aItemCommand, nItemId); } |