Removed code that checks that the workspace is headless in upgrade functions.

Author: lizlooney

src/blocks/mrc_class_method_def.ts

@@ -623,10 +623,6 @@ export function getMethodNamesAlreadyOverriddenInWorkspace(
  * This function should only be called when upgrading old projects.
  */
 export function upgrade_002_to_003(workspace: Blockly.Workspace): void {
-  // Make sure the workspace is headless.
-  if (workspace.rendered) {
-    throw new Error('upgrade_002_to_003 should never be called with a rendered workspace.');
-  }
   workspace.getBlocksByType(BLOCK_NAME).forEach(block => {
     (block as ClassMethodDefBlock).upgrade_002_to_003();
   });
@@ -637,10 +633,6 @@ export function upgrade_002_to_003(workspace: Blockly.Workspace): void {
  * This function should only be called when upgrading old projects.
  */
 export function upgrade_004_to_005(workspace: Blockly.Workspace): void {
-  // Make sure the workspace is headless.
-  if (workspace.rendered) {
-    throw new Error('upgrade_004_to_005 should never be called with a rendered workspace.');
-  }
   workspace.getBlocksByType(BLOCK_NAME).forEach(block => {
     (block as ClassMethodDefBlock).upgrade_004_to_005();
   });

src/blocks/mrc_mechanism_component_holder.ts

@@ -526,10 +526,6 @@ export function mrcDescendantsMayHaveChanged(workspace: Blockly.Workspace): void
  * This function should only be called when upgrading old projects.
  */
 export function upgrade_001_to_002(workspace: Blockly.Workspace) {
-  // Make sure the workspace is headless.
-  if (workspace.rendered) {
-    throw new Error('upgrade_001_to_002 should never be called with a rendered workspace.');
-  }
   // Make sure the module type is ROBOT.
   if (getModuleTypeForWorkspace(workspace) !== storageModule.ModuleType.ROBOT) {
     throw new Error('upgrade_001_to_002 should only be called for a robot module.');