diff options
| author | Tamás Zolnai <tamas.zolnai@collabora.com> | 2019-02-19 10:24:46 +0100 |
|---|---|---|
| committer | Tamás Zolnai <tamas.zolnai@collabora.com> | 2019-06-19 15:58:30 +0200 |
| commit | 29c1084e155d045d7a8709ad5bfffdaf5740a3bc (patch) | |
| tree | cfa63753fe02819bb8ea77387b70f6457351470c /framework | |
| parent | 4e1549ba5b55ab566b3693c8c772515794f95efd (diff) | |
MSForms: Fix a regression in inplace menu handling
We should not call implts_resetInplaceMenuBar() unconditionally.
The problem which was fixed by that call can be solved by setting
m_bInplaceMenuSet to false, which is the proper solution. So after
the menu cleared up (inplace menu too), this indicator flag also
contains the right value.
Regression from:
dc3a2546bd0ad0afe20cba9940934405174fd593
Change-Id: I85243417eb273629cc57b762b98fdd32dc9b1dd5
Reviewed-on: https://gerrit.libreoffice.org/68010
Reviewed-by: Andras Timar <andras.timar@collabora.com>
Tested-by: Andras Timar <andras.timar@collabora.com>
Diffstat (limited to 'framework')
| -rw-r--r-- | framework/source/layoutmanager/layoutmanager.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/framework/source/layoutmanager/layoutmanager.cxx b/framework/source/layoutmanager/layoutmanager.cxx index 86182188c708..af370d042bd9 100644 --- a/framework/source/layoutmanager/layoutmanager.cxx +++ b/framework/source/layoutmanager/layoutmanager.cxx @@ -217,8 +217,6 @@ void LayoutManager::impl_clearUpMenuBar() { implts_lock(); - implts_resetInplaceMenuBar(); - // Clear up VCL menu bar to prepare shutdown if ( m_xContainerWindow.is() ) { @@ -269,6 +267,7 @@ void LayoutManager::impl_clearUpMenuBar() m_xInplaceMenuBar.clear(); } pMenuBar.disposeAndClear(); + m_bInplaceMenuSet = false; Reference< XComponent > xComp( m_xMenuBar, UNO_QUERY ); if ( xComp.is() ) |