Add safari cache clearing browser functionality to the safari browser.

tools/testing/dart/browser_controller.dart

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 library browser;
 
 import "dart:async";
 import "dart:core";
 import "dart:io";
 
 import 'android.dart';
@@ skipping 165 matching lines @@
    * The binary used to run safari - changing this can be nececcary for
    * testing or using non standard safari installation.
    */
   const String binary = "/Applications/Safari.app/Contents/MacOS/Safari";
 
   /**
    * We get the safari version by parsing a version file
    */
   const String versionFile = "/Applications/Safari.app/Contents/version.plist";
 
-
   Future<bool> allowPopUps() {
     var command = "defaults";
     var args = ["write", "com.apple.safari",
                 "com.apple.Safari.ContentPageGroupIdentifier."
                 "WebKit2JavaScriptCanOpenWindowsAutomatically",
                 "1"];
     return Process.run(command, args).then((result) {
-        if (result.exitCode != 0) {
-          _logEvent("Could not disable pop-up blocking for safari");
-          return false;
-        }
-        return true;
+      if (result.exitCode != 0) {
+        _logEvent("Could not disable pop-up blocking for safari");
+        return false;
+      }
+      return true;
+    });
+  }
+
+  // Clears the cache if the static deleteCache flag is set.
+  // Returns false if the command to actually clear the cache did not complete.
+  Future<bool> clearCache() {
+    if (!deleteCache) return new Future.immediate(true);
+    var command = "rm";
+    var args = ["-rf", "\$HOME/Library/Caches/com.apple.Safari"];

[kustermann, 2013/06/20 09:21:04]

As discussed offline, this doesn't work.
Please read the environment using 'dart:io' and delete using
Directory.delete(recursive: true) [or something like that].

[ricow1, 2013/06/20 10:24:05]

Done.

+    _logEvent("Clearing safari cache: $command ${args.join(' ')}");
+    return Process.run(command, args).then((result) {
+      if (result.exitCode != 0) {
+        _logEvent("Could not clear safari cache");
+        return false;
+      }
+      return true;
     });
   }
 
   Future<String> getVersion() {
     /**
      * Example of the file:
      * <?xml version="1.0" encoding="UTF-8"?>
      * <!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
      * <plist version="1.0">
      * <dict>
@@ skipping 26 matching lines @@
   void _createLaunchHTML(var path, var url) {
     var file = new File("${path}/launch.html");
     var randomFile = file.openSync(FileMode.WRITE);
     var content = '<script language="JavaScript">location = "$url"</script>';
     randomFile.writeStringSync(content);
     randomFile.close();
   }
 
   Future<bool> start(String url) {
     _logEvent("Starting Safari browser on: $url");
-    // Get the version and log that.
     return allowPopUps().then((success) {
       if (!success) {
         return new Future.immediate(false);
       }
-      return getVersion().then((version) {
-        _logEvent("Got version: $version");
-        var args = ["'$url'"];
-        return new Directory('').createTemp().then((userDir) {
-          _cleanup = () { userDir.deleteSync(recursive: true); };
-          _createLaunchHTML(userDir.path, url);
-          var args = ["${userDir.path}/launch.html"];
-          return startBrowser(binary, args);
+      return clearCache().then((cleared) {
+        if (!cleared) {
+          return new Future.immediate(false);
+        }
+        // Get the version and log that.
+        return getVersion().then((version) {
+          _logEvent("Got version: $version");
+          return new Directory('').createTemp().then((userDir) {
+            _cleanup = () { userDir.deleteSync(recursive: true); };
+            _createLaunchHTML(userDir.path, url);
+            var args = ["${userDir.path}/launch.html"];
+            return startBrowser(binary, args);
+          });
+        }).catchError((e) {

[kustermann, 2013/06/20 09:21:04]

e -> error.

[ricow1, 2013/06/20 10:24:05]

Done.

+          _logEvent("Running $binary --version failed with $e");
+          return false;
         });
-      }).catchError((e) {
-        _logEvent("Running $binary --version failed with $e");
-        return false;
       });
     });
   }
 
   String toString() => "Safari";
+
+  static bool deleteCache = false;

[kustermann, 2013/06/20 09:21:04]

Add a comment here.

[ricow1, 2013/06/20 10:24:05]

Done.

+
 }
 
 
 class Chrome extends Browser {
   /**
    * The binary used to run chrome - changing this can be nececcary for
    * testing or using non standard chrome installation.
    */
   const String binary = "google-chrome";
 
@@ skipping 730 matching lines @@
 </head>
   <body onload="startTesting()">
     Dart test driver, number of tests: <div id="number"></div>
     <iframe id="embedded_iframe"></iframe>
   </body>
 </html>
 """;
     return driverContent;
   }
 }