In the item view constructors, call setFocus() only after

m_extensionsFactory is initialised. This fixes a possible crash in
Konqueror when changing the settings or manipulating tabs.

CCBUG: 240374

svn path=/trunk/KDE/kdebase/apps/; revision=1199276

1 files changed, 3 insertions, 2 deletions

diff --git a/src/views/dolphiniconsview.cpp b/src/views/dolphiniconsview.cpp
index 636bdd66c..ce7265af7 100644
--- a/src/views/dolphiniconsview.cpp
+++ b/src/views/dolphiniconsview.cpp
@@ -121,13 +121,14 @@ DolphinIconsView::DolphinIconsView(QWidget* parent,
     connect(m_categoryDrawer, SIGNAL(actionRequested(int,QModelIndex)), this, SLOT(categoryDrawerActionRequested(int,QModelIndex)));
     setCategoryDrawer(m_categoryDrawer);
 
-    setFocus();
-
     connect(KGlobalSettings::self(), SIGNAL(settingsChanged(int)),
             this, SLOT(slotGlobalSettingsChanged(int)));
 
     updateGridSize(view->showPreview(), 0);
     m_extensionsFactory = new ViewExtensionsFactory(this, dolphinViewController, viewModeController);
+
+    // setFocus() must be called after m_extensionsFactory is initialised (see bug 240374).
+    setFocus();
 }
 
 DolphinIconsView::~DolphinIconsView()