From 0fb288c575d834f7a2b7b505b6901e6d81394d0d Mon Sep 17 00:00:00 2001
From: Devon Carew <devoncarew@gmail.com>
Date: Wed, 2 Mar 2016 21:14:11 -0800
Subject: [PATCH] refactor to move an ios specific class out of globals.dart

---
 packages/flutter_tools/lib/executable.dart           |  2 --
 packages/flutter_tools/lib/src/globals.dart          |  4 ----
 packages/flutter_tools/lib/src/ios/ios_workflow.dart | 11 +++++------
 packages/flutter_tools/lib/src/ios/mac.dart          |  7 ++-----
 packages/flutter_tools/lib/src/ios/simulators.dart   |  4 ++--
 5 files changed, 9 insertions(+), 19 deletions(-)

diff --git a/packages/flutter_tools/lib/executable.dart b/packages/flutter_tools/lib/executable.dart
index 4f36412b9e..7ed6a6d266 100644
--- a/packages/flutter_tools/lib/executable.dart
+++ b/packages/flutter_tools/lib/executable.dart
@@ -32,7 +32,6 @@ import 'src/commands/update_packages.dart';
 import 'src/commands/upgrade.dart';
 import 'src/device.dart';
 import 'src/doctor.dart';
-import 'src/ios/mac.dart';
 import 'src/runner/flutter_command_runner.dart';
 
 /// Main entry point for commands.
@@ -69,7 +68,6 @@ Future main(List<String> args) async {
     context[Logger] = new StdoutLogger();
     context[DeviceManager] = new DeviceManager();
     Doctor.initGlobal();
-    XCode.initGlobal();
 
     dynamic result = await runner.run(args);
 
diff --git a/packages/flutter_tools/lib/src/globals.dart b/packages/flutter_tools/lib/src/globals.dart
index efd072ef86..3ce1c1b3a5 100644
--- a/packages/flutter_tools/lib/src/globals.dart
+++ b/packages/flutter_tools/lib/src/globals.dart
@@ -7,16 +7,12 @@ import 'base/context.dart';
 import 'base/logger.dart';
 import 'device.dart';
 import 'doctor.dart';
-import 'ios/mac.dart';
 
 DeviceManager get deviceManager => context[DeviceManager];
 Logger get logger => context[Logger];
 AndroidSdk get androidSdk => context[AndroidSdk];
 Doctor get doctor => context[Doctor];
 
-// Mac specific globals - will be null on other platforms.
-XCode get xcode => context[XCode];
-
 /// Display an error level message to the user. Commands should use this if they
 /// fail in some way.
 void printError(String message, [StackTrace stackTrace]) => logger.printError(message, stackTrace);
diff --git a/packages/flutter_tools/lib/src/ios/ios_workflow.dart b/packages/flutter_tools/lib/src/ios/ios_workflow.dart
index 4a1d047f1e..67eda8daf1 100644
--- a/packages/flutter_tools/lib/src/ios/ios_workflow.dart
+++ b/packages/flutter_tools/lib/src/ios/ios_workflow.dart
@@ -6,7 +6,6 @@ import 'dart:io';
 
 import '../base/process.dart';
 import '../doctor.dart';
-import '../globals.dart';
 import 'mac.dart';
 
 class IOSWorkflow extends Workflow {
@@ -15,11 +14,11 @@ class IOSWorkflow extends Workflow {
   bool get appliesToHostPlatform => Platform.isMacOS;
 
   // We need xcode (+simctl) to list simulator devices, and idevice_id to list real devices.
-  bool get canListDevices => xcode.isInstalledAndMeetsVersionCheck;
+  bool get canListDevices => XCode.instance.isInstalledAndMeetsVersionCheck;
 
   // We need xcode to launch simulator devices, and ideviceinstaller and ios-deploy
   // for real devices.
-  bool get canLaunchDevices => xcode.isInstalledAndMeetsVersionCheck;
+  bool get canLaunchDevices => XCode.instance.isInstalledAndMeetsVersionCheck;
 
   ValidationResult validate() {
     Validator iosValidator = new Validator(
@@ -28,15 +27,15 @@ class IOSWorkflow extends Workflow {
     );
 
     ValidationType xcodeExists() {
-      return xcode.isInstalled ? ValidationType.installed : ValidationType.missing;
+      return XCode.instance.isInstalled ? ValidationType.installed : ValidationType.missing;
     };
 
     ValidationType xcodeVersionSatisfactory() {
-      return xcode.isInstalledAndMeetsVersionCheck ? ValidationType.installed : ValidationType.missing;
+      return XCode.instance.isInstalledAndMeetsVersionCheck ? ValidationType.installed : ValidationType.missing;
     };
 
     ValidationType xcodeEulaSigned() {
-      return xcode.eulaSigned ? ValidationType.installed : ValidationType.missing;
+      return XCode.instance.eulaSigned ? ValidationType.installed : ValidationType.missing;
     };
 
     ValidationType brewExists() {
diff --git a/packages/flutter_tools/lib/src/ios/mac.dart b/packages/flutter_tools/lib/src/ios/mac.dart
index 163f6f4825..df2ef7d8a9 100644
--- a/packages/flutter_tools/lib/src/ios/mac.dart
+++ b/packages/flutter_tools/lib/src/ios/mac.dart
@@ -18,15 +18,12 @@ import 'setup_xcodeproj.dart';
 
 String get homeDirectory => path.absolute(Platform.environment['HOME']);
 
-// TODO(devoncarew): Refactor functionality into XCode.
-
 const int kXcodeRequiredVersionMajor = 7;
 const int kXcodeRequiredVersionMinor = 2;
 
 class XCode {
-  static void initGlobal() {
-    context[XCode] = new XCode();
-  }
+  /// Returns [XCode] active in the current app context.
+  static XCode get instance => context[XCode] ?? (context[XCode] = new XCode());
 
   bool get isInstalledAndMeetsVersionCheck => isInstalled && xcodeVersionSatisfactory;
 
diff --git a/packages/flutter_tools/lib/src/ios/simulators.dart b/packages/flutter_tools/lib/src/ios/simulators.dart
index d8ea7b7143..832320126a 100644
--- a/packages/flutter_tools/lib/src/ios/simulators.dart
+++ b/packages/flutter_tools/lib/src/ios/simulators.dart
@@ -38,7 +38,7 @@ class IOSSimulatorUtils {
   }
 
   List<IOSSimulator> getAttachedDevices() {
-    if (!xcode.isInstalledAndMeetsVersionCheck)
+    if (!XCode.instance.isInstalledAndMeetsVersionCheck)
       return <IOSSimulator>[];
 
     return SimControl.instance.getConnectedDevices().map((SimDevice device) {
@@ -320,7 +320,7 @@ class IOSSimulator extends Device {
     if (clearLogs)
       this.clearLogs();
 
-    if(!(await _setupUpdatedApplicationBundle(app, toolchain)))
+    if (!(await _setupUpdatedApplicationBundle(app, toolchain)))
       return false;
 
     // Prepare launch arguments.