[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-6.4' - sd/qa sfx2/source

[Tamás Zolnai (via logerrit)]

 sd/qa/unit/tiledrendering/tiledrendering.cxx |    4 ----
 sfx2/source/control/bindings.cxx             |   23 -----------------------
 2 files changed, 27 deletions(-)

New commits:
commit c5f8b8b58bdc04211b490303c3b9230f7e5ea368
Author:     Tamás Zolnai <tamas.zolnai at collabora.com>
AuthorDate: Tue Mar 23 14:53:58 2021 +0100
Commit:     Aron Budea <aron.budea at collabora.com>
CommitDate: Fri Mar 26 22:08:04 2021 +0100

    Revert "lok: switch to the correct view before any UI update is done
    
    via SfxBindings."
    
    In online Impress the status bar content didn't get filled in a
    2nd view.
    
    This reverts commit 35b81a05c7c58d0db2276627a9387a738c409974.
    
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112997
    Tested-by: Jenkins
    Reviewed-by: Tamás Zolnai <tamas.zolnai at collabora.com>
    (cherry picked from commit bdbb5d0389642c0d445b5779fe2a18fda3e4a4d4)
    
    Change-Id: I29adf767d2bf3fa0f19b13f8eae3d2464cd90601
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/113186
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice at gmail.com>
    Reviewed-by: Aron Budea <aron.budea at collabora.com>

diff --git a/sd/qa/unit/tiledrendering/tiledrendering.cxx b/sd/qa/unit/tiledrendering/tiledrendering.cxx
index b25c53c31d4b..64e6196ae605 100644
--- a/sd/qa/unit/tiledrendering/tiledrendering.cxx
+++ b/sd/qa/unit/tiledrendering/tiledrendering.cxx
@@ -1002,7 +1002,6 @@ void SdTiledRenderingTest::testViewCursorParts()
     SfxLokHelper::createView();
     pXImpressDocument->initializeForTiledRendering(uno::Sequence<beans::PropertyValue>());
     ViewCallback aView2;
-    int nView2 = SfxLokHelper::getView();
 
     // Select the shape in the second view.
     sd::ViewShell* pViewShell = pXImpressDocument->GetDocShell()->GetViewShell();
@@ -1016,7 +1015,6 @@ void SdTiledRenderingTest::testViewCursorParts()
     pView->UnmarkAllObj(pView->GetSdrPageView());
 
     // Now switch to the second part in the second view.
-    SfxLokHelper::setView(nView2);
     pXImpressDocument->setPart(1);
     aView2.m_nPart = 1;
     aView1.m_bGraphicViewSelectionInvalidated = false;
@@ -1184,8 +1182,6 @@ void SdTiledRenderingTest::testCursorVisibility_MultiView()
     pXImpressDocument->postMouseEvent(LOK_MOUSEEVENT_MOUSEBUTTONDOWN,
                                       centerX, centerY,
                                       2, MOUSE_LEFT, 0);
-
-    SfxLokHelper::setView(nView1);
     pXImpressDocument->postMouseEvent(LOK_MOUSEEVENT_MOUSEBUTTONUP,
                                       centerX, centerY,
                                       2, MOUSE_LEFT, 0);
diff --git a/sfx2/source/control/bindings.cxx b/sfx2/source/control/bindings.cxx
index 6be545d382bf..1d6fd0d62441 100644
--- a/sfx2/source/control/bindings.cxx
+++ b/sfx2/source/control/bindings.cxx
@@ -55,9 +55,6 @@
 #include <sfx2/viewfrm.hxx>
 #include <sfx2/objsh.hxx>
 #include <sfx2/msgpool.hxx>
-#include <sfx2/viewsh.hxx>
-#include <sfx2/lokhelper.hxx>
-#include <comphelper/lok.hxx>
 
 #include <cstddef>
 #include <memory>
@@ -1243,26 +1240,6 @@ bool SfxBindings::NextJob_Impl(Timer const * pTimer)
 
     SfxApplication *pSfxApp = SfxGetpApp();
 
-    // Switch to the right view for state update.
-    if (comphelper::LibreOfficeKit::isActive())
-    {
-        if( pDispatcher )
-        {
-            SfxViewFrame* pFrame = pDispatcher->GetFrame();
-            if (pFrame)
-            {
-                vcl::Window* pFrameWin = pFrame->GetWindow().GetFrameWindow();
-                if (pFrameWin && pFrameWin->GetLOKNotifier()) {
-                    const SfxViewShell* pView = dynamic_cast<const SfxViewShell*>(pFrameWin->GetLOKNotifier());
-                    if (pView)
-                    {
-                        SfxLokHelper::setView(pView->GetViewShellId().get());
-                    }
-                }
-            }
-        }
-    }
-
     if( pDispatcher )
         pDispatcher->Update_Impl();