Review comments.

Improve a test.
Improve a comment.

Author: Benoit Hudson

Assets/FbxExporters/Editor/UnitTests/FbxPrefabTest.cs

@@ -208,6 +208,7 @@ public void BasicTest() {
                 AssertAreIdentical(m_originalRep, History(m_manualPrefab));
 
                 // Make sure we got the right changes.
+                Assert.AreEqual (1, updateSet.NumUpdates);
                 Assert.That (updateSet.Updated, Is.EquivalentTo (new string [] {
                     // TODO: UNI-24579 - we should only be seeing Parent3 here,
                     // the other two are for transform changes, but

Assets/FbxExporters/FbxPrefab.cs

@@ -1042,9 +1042,11 @@ public void SetSourceModel(GameObject fbxModel) {
         /// by the updatedInstance. Those changes will be applied to the prefab.
         ///
         /// The updatedObjects include all objects in the temporary instance
-        /// that were created, plus all objects which had a component get
-        /// created, destroyed, or updated. You get notification for objects
-        /// that were destroyed.
+        /// that were:
+        /// - created, or
+        /// - changed parent, or
+        /// - had a component that was created, destroyed, or updated.
+        /// There is no notification for entire objects that were destroyed.
         /// </summary>
         public delegate void HandleUpdate(FbxPrefab updatedInstance, IEnumerable<GameObject> updatedObjects);