UNI-40132 code review fixes

- made the changes on this branch to avoid merge conflicts

Author: vkovec

Assets/FbxExporters/Editor/ConvertToModel.cs

@@ -77,9 +77,8 @@ public static GameObject[] CreateInstantiatedModelPrefab (
                 GameObject [] unityGameObjectsToConvert)
             {
                 var toExport = ModelExporter.RemoveRedundantObjects (unityGameObjectsToConvert);
-                var wasExported = Enumerable.ToArray (toExport);
-                ConvertToPrefabEditorWindow.Init (wasExported);
-                return wasExported;
+                ConvertToPrefabEditorWindow.Init (toExport);
+                return toExport.ToArray();
             }
 
             /// <summary>

Assets/FbxExporters/Editor/ConvertToPrefabEditorWindow.cs

@@ -13,7 +13,7 @@ namespace Editor
         public class ConvertToPrefabEditorWindow : ExportOptionsEditorWindow
         {
             protected override GUIContent WindowTitle { get { return new GUIContent ("Convert Options"); }}
-            protected override float MinWindowHeight { get { return 280; } }
+            protected override float MinWindowHeight { get { return 280; } } // determined by trial and error
             protected override string ExportButtonName { get { return "Convert"; } }
             private GameObject[] m_toConvert;
             private string m_prefabFileName = "";
@@ -27,7 +27,7 @@ public static void Init (IEnumerable<GameObject> toConvert)
             }
 
             protected void SetGameObjectsToConvert(IEnumerable<GameObject> toConvert){
-                m_toConvert = Enumerable.ToArray (toConvert);
+                m_toConvert = toConvert.OrderBy (go => go.name).ToArray ();
 
                 if (m_toConvert.Length == 1) {
                     m_prefabFileName = m_toConvert [0].name;