diff options
author | Caolán McNamara <caolanm@redhat.com> | 2013-07-18 14:27:15 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2013-07-18 14:36:17 +0100 |
commit | 4a578dc9c7b9c5697f624807790565c99ccb919d (patch) | |
tree | b795b4a2e0e8fbecc02568bf7422eeb9187928e0 /sd/source/ui/view/drviews1.cxx | |
parent | a2e599f16ba253c706aa84d0a17f2936f78c3892 (diff) |
Resolves: fdo#66924 switching to master view is broken
It seems 0143805a565418d2a114c16b7eeba3b784176d9e wanted to not call
SfxShell::Activate from ViewShell::Activate and removed that call and added an
explicit SfxShell::BroadcastContextForActivation(true) (which is the body of
SfxShell::Activate) to OutlineViewShell::Activate for the case where it was
apparently wanted.
Then DrawViewShell::Deactivate had the call to ViewShell::Deactivate removed
persumably to avoid calling SfxShell::Deactivate via ViewShell::Deactivate
But with ViewShell::Deactivate omitted switching to master view is broken, it
does quite a lot more than call just SfxShell::Deactivate
Restore the call of ViewShell::Deactivate from DrawViewShell::Deactivate and
remove the call of SfxShell::Deactivate from ViewShell::Deactivate and put its
equivalent of SfxShell::BroadcastContextForActivation(false) into
OutlineViewShell::Deactivate to balance OutlineViewShell::Activate
regression since 0143805a565418d2a114c16b7eeba3b784176d9e
Change-Id: I5175378af6a6527a8be4d5011f13029d028b4304
Diffstat (limited to 'sd/source/ui/view/drviews1.cxx')
-rw-r--r-- | sd/source/ui/view/drviews1.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sd/source/ui/view/drviews1.cxx b/sd/source/ui/view/drviews1.cxx index fe0f1dd598d8..f6eb20f545d5 100644 --- a/sd/source/ui/view/drviews1.cxx +++ b/sd/source/ui/view/drviews1.cxx @@ -114,9 +114,9 @@ void DrawViewShell::UIDeactivated( SfxInPlaceClient* pCli ) } -void DrawViewShell::Deactivate(sal_Bool /*bIsMDIActivate*/) +void DrawViewShell::Deactivate(sal_Bool bIsMDIActivate) { - // Do not forward to ViewShell::Deactivate() to prevent a context change. + ViewShell::Deactivate(bIsMDIActivate); } namespace |