am a8048b0e: Merge "Fix TextViewTests#testMarquee for Landscape Mode" into eclair

Merge commit 'a8048b0ee20e0fce5bff081da6579548acf29c47' into eclair

* commit 'a8048b0ee20e0fce5bff081da6579548acf29c47':
  Fix TextViewTests#testMarquee for Landscape Mode
diff --git a/tests/core/runner/src/android/test/InstrumentationCtsTestRunner.java b/tests/core/runner/src/android/test/InstrumentationCtsTestRunner.java
index 4ca1cc2..5ac7251 100644
--- a/tests/core/runner/src/android/test/InstrumentationCtsTestRunner.java
+++ b/tests/core/runner/src/android/test/InstrumentationCtsTestRunner.java
@@ -16,30 +16,26 @@
 
 package android.test;
 
-import com.android.internal.util.Predicate;
-import com.android.internal.util.Predicates;
-
-import dalvik.annotation.BrokenTest;
-import dalvik.annotation.SideEffect;
-
-import android.app.KeyguardManager;
-import android.content.Context;
-import android.content.pm.PackageManager;
-import android.os.Bundle;
-import android.test.suitebuilder.TestMethod;
-import android.test.suitebuilder.annotation.HasAnnotation;
-import android.util.Log;
-
 import java.io.File;
 import java.lang.reflect.Field;
 import java.lang.reflect.Modifier;
 import java.util.List;
 import java.util.TimeZone;
 
+import com.android.internal.util.Predicate;
+import com.android.internal.util.Predicates;
+
+import dalvik.annotation.BrokenTest;
+import dalvik.annotation.SideEffect;
+
 import junit.framework.AssertionFailedError;
 import junit.framework.Test;
 import junit.framework.TestCase;
 import junit.framework.TestListener;
+import android.os.Bundle;
+import android.test.suitebuilder.TestMethod;
+import android.test.suitebuilder.annotation.HasAnnotation;
+import android.util.Log;
 
 /**
  * This test runner extends the default InstrumentationTestRunner. It overrides
@@ -56,50 +52,37 @@
 
     /**
      * Convenience definition of our log tag.
-     */
+     */ 
     private static final String TAG = "InstrumentationCtsTestRunner";
-
+    
     /**
      * True if (and only if) we are running in single-test mode (as opposed to
      * batch mode).
      */
     private boolean singleTest = false;
-
+    
     @Override
     public void onCreate(Bundle arguments) {
         // We might want to move this to /sdcard, if is is mounted/writable.
         File cacheDir = getTargetContext().getCacheDir();
 
-        // Set some properties that the core tests absolutely need.
+        // Set some properties that the core tests absolutely need. 
         System.setProperty("user.language", "en");
         System.setProperty("user.region", "US");
-
+        
         System.setProperty("java.home", cacheDir.getAbsolutePath());
         System.setProperty("user.home", cacheDir.getAbsolutePath());
         System.setProperty("java.io.tmpdir", cacheDir.getAbsolutePath());
         System.setProperty("javax.net.ssl.trustStore",
                 "/etc/security/cacerts.bks");
-
+        
         TimeZone.setDefault(TimeZone.getTimeZone("GMT"));
-
+        
         if (arguments != null) {
             String classArg = arguments.getString(ARGUMENT_TEST_CLASS);
-            singleTest = classArg != null && classArg.contains("#");
+            singleTest = classArg != null && classArg.contains("#"); 
         }
-
-        // attempt to disable keyguard,  if current test has permission to do so
-        // TODO: move this to a better place, such as InstrumentationTestRunner ?
-        if (getContext().checkCallingOrSelfPermission(android.Manifest.permission.DISABLE_KEYGUARD)
-                == PackageManager.PERMISSION_GRANTED) {
-            Log.i(TAG, "Disabling keyguard");
-            KeyguardManager keyguardManager =
-                (KeyguardManager) getContext().getSystemService(Context.KEYGUARD_SERVICE);
-            keyguardManager.newKeyguardLock("cts").disableKeyguard();
-        } else {
-            Log.i(TAG, "Test lacks permission to disable keyguard. " +
-                    "UI based tests may fail if keyguard is up");
-        }
-
+        
         super.onCreate(arguments);
     }
 
@@ -109,36 +92,36 @@
 
         runner.addTestListener(new TestListener() {
             /**
-             * The last test class we executed code from.
+             * The last test class we executed code from.  
              */
             private Class<?> lastClass;
-
+            
             /**
              * The minimum time we expect a test to take.
              */
             private static final int MINIMUM_TIME = 100;
-
+            
             /**
              * The start time of our current test in System.currentTimeMillis().
              */
             private long startTime;
-
+            
             public void startTest(Test test) {
                 if (test.getClass() != lastClass) {
                     lastClass = test.getClass();
                     printMemory(test.getClass());
                 }
-
+                
                 Thread.currentThread().setContextClassLoader(
                         test.getClass().getClassLoader());
-
+                
                 startTime = System.currentTimeMillis();
             }
-
+            
             public void endTest(Test test) {
                 if (test instanceof TestCase) {
                     cleanup((TestCase)test);
-
+                    
                     /*
                      * Make sure all tests take at least MINIMUM_TIME to
                      * complete. If they don't, we wait a bit. The Cupcake
@@ -146,7 +129,7 @@
                      * short time, which causes headache for the CTS.
                      */
                     long timeTaken = System.currentTimeMillis() - startTime;
-
+                    
                     if (timeTaken < MINIMUM_TIME) {
                         try {
                             Thread.sleep(MINIMUM_TIME - timeTaken);
@@ -156,15 +139,15 @@
                     }
                 }
             }
-
+            
             public void addError(Test test, Throwable t) {
                 // This space intentionally left blank.
             }
-
+            
             public void addFailure(Test test, AssertionFailedError t) {
                 // This space intentionally left blank.
             }
-
+            
             /**
              * Dumps some memory info.
              */
@@ -174,7 +157,7 @@
                 long total = runtime.totalMemory();
                 long free = runtime.freeMemory();
                 long used = total - free;
-
+                
                 Log.d(TAG, "Total memory  : " + total);
                 Log.d(TAG, "Used memory   : " + used);
                 Log.d(TAG, "Free memory   : " + free);
@@ -190,7 +173,7 @@
              */
             private void cleanup(TestCase test) {
                 Class<?> clazz = test.getClass();
-
+                
                 while (clazz != TestCase.class) {
                     Field[] fields = clazz.getDeclaredFields();
                     for (int i = 0; i < fields.length; i++) {
@@ -205,15 +188,15 @@
                             }
                         }
                     }
-
+                    
                     clazz = clazz.getSuperclass();
                 }
             }
-
+            
         });
-
+        
         return runner;
-    }
+    }    
 
     @Override
     List<Predicate<TestMethod>> getBuilderRequirements() {
diff --git a/tests/tests/graphics/src/android/graphics/drawable/cts/NinePatchDrawableTest.java b/tests/tests/graphics/src/android/graphics/drawable/cts/NinePatchDrawableTest.java
index f62fea8..3b376fd 100644
--- a/tests/tests/graphics/src/android/graphics/drawable/cts/NinePatchDrawableTest.java
+++ b/tests/tests/graphics/src/android/graphics/drawable/cts/NinePatchDrawableTest.java
@@ -87,8 +87,7 @@
 
         new NinePatchDrawable(bmp, chunk, r, name);
 
-        // Omit for now - known failure, fixed in froyo. Bug 2219785
-        //new NinePatchDrawable(new NinePatch(bmp, chunk, name));
+        new NinePatchDrawable(new NinePatch(bmp, chunk, name));
 
         chunk = new byte[MIN_CHUNK_SIZE - 1];
         chunk[MIN_CHUNK_SIZE - 2] = 1;