Bugfix: Crash when clear the scene tree view and a non-top-level object was selected

Source/EditorCore/GUI/BsGUIResourceTreeView.cpp

@@ -54,6 +54,11 @@ namespace bs
 	{
 		clearDropTarget();
 
+		for(auto& child : mRootElement.mChildren)
+			deleteTreeElement((ResourceTreeElement*)&child);
+
+		mRootElement.mChildren.clear();
+
 		ResourceTreeViewLocator::_provide(nullptr);
 	}
 
@@ -164,6 +169,11 @@ namespace bs
 	{
 		closeTemporarilyExpandedElements(); // In case this element is one of them
 
+		for(auto& child : element->mChildren)
+			deleteTreeElement((ResourceTreeElement*)child);
+
+		element->mChildren.clear();
+
 		if (element->mIsHighlighted)
 			clearPing();
 

Source/EditorCore/GUI/BsGUISceneTreeView.cpp

@@ -54,6 +54,11 @@ namespace bs
 
 	GUISceneTreeView::~GUISceneTreeView()
 	{
+		for(auto& child : mRootElement.mChildren)
+			deleteTreeElementInternal(child);
+
+		mRootElement.mChildren.clear();
+
 		SceneTreeViewLocator::_remove(this);
 	}
 
@@ -286,6 +291,11 @@ namespace bs
 	{
 		closeTemporarilyExpandedElements(); // In case this element is one of them
 
+		for(auto& child : element->mChildren)
+			deleteTreeElementInternal(child);
+
+		element->mChildren.clear();
+
 		if (element->mIsHighlighted)
 			clearPing();
 

Source/EditorCore/GUI/BsGUISceneTreeView.h

@@ -124,8 +124,6 @@ namespace bs
 		virtual ~GUISceneTreeView();
 
 	protected:
-		SceneTreeElement mRootElement;
-
 		GUISceneTreeView(const String& backgroundStyle, const String& elementBtnStyle, 
 			const String& foldoutBtnStyle, const String& highlightBackgroundStyle, const String& selectionBackgroundStyle, 
 			const String& editBoxStyle, const String& dragHighlightStyle, const String& dragSepHighlightStyle, const GUIDimensions& dimensions);
@@ -190,6 +188,8 @@ namespace bs
 		 */
 		static void cleanDuplicates(Vector<HSceneObject>& objects);
 
+		SceneTreeElement mRootElement;
+
 		Vector<HSceneObject> mCopyList;
 		bool mCutFlag;
 

Source/EditorCore/GUI/BsGUITreeView.cpp

@@ -38,16 +38,13 @@ namespace bs
 
 	GUITreeView::TreeElement::~TreeElement()
 	{
-		for(auto& child : mChildren)
-			bs_delete(child);
+		assert(mChildren.empty());
 
 		if(mFoldoutBtn != nullptr)
 			GUIElement::destroy(mFoldoutBtn);
 
 		if(mElement != nullptr)
 			GUIElement::destroy(mElement);
-
-		mChildren.clear();
 	}
 
 	bool GUITreeView::TreeElement::isParentRec(TreeElement* element) const