[Android] Switch to DeviceUtils versions of Ls, SetJavaAsserts, GetProp, and SetProp.

build/android/pylib/device/device_utils.py

Index: build/android/pylib/device/device_utils.py
diff --git a/build/android/pylib/device/device_utils.py b/build/android/pylib/device/device_utils.py
index 28d258f4d12a080d604d16a4924315dfa47da505..ee8493676489e68b19cf137e70d29ca5524dea52 100644
--- a/build/android/pylib/device/device_utils.py
+++ b/build/android/pylib/device/device_utils.py
@@ -546,6 +546,62 @@ class DeviceUtils(object):
     else:
       self.old_interface.SetFileContents(device_path, contents)
 
+  @decorators.WithTimeoutAndRetriesFromInstance()
+  def Ls(self, device_path, as_root=False, timeout=None, retries=None):
+    """Lists the contents of a directory on the device.
+
+    Args:
+      device_path: A string containing the path of the directory on the device
+                   to list.
+      as_root: A boolean inidcating whether the ls should be executed with root

[frankf, 2014/07/07 17:29:03]

where's this used?

[jbudorick, 2014/07/07 17:43:15]

It currently isn't used by any caller, and obviously it isn't currently used in
the implementation.

Given that there are some directories on devices that require root permissions
to list, I think it's an option worth providing, and I'm planning to use it in
the final implementation.

[frankf, 2014/07/07 18:00:52]

Just leave it out for now then. You can always add it back.

On 2014/07/07 17:43:15, jbudorick wrote:

[jbudorick, 2014/07/07 18:13:49]

Done.

+               priveleges.
+      timeout: Same as for |IsOnline|.
+      retries: Same as for |IsOnline|.
+    Returns:
+      The contents of the directory specified by |device_path|.
+    """
+    return self.old_interface.ListPathContents(device_path)
+
+  @decorators.WithTimeoutAndRetriesFromInstance()
+  def SetJavaAsserts(self, enabled, timeout=None, retries=None):
+    """Enables or disables Java asserts.
+
+    Args:
+      enabled: A boolean indicating whether Java asserts should be enabled
+               or disabled.
+      timeout: Same as for |IsOnline|.
+      retries: Same as for |IsOnline|.
+    """
+    self.old_interface.SetJavaAssertsEnabled(enabled)
+
+  @decorators.WithTimeoutAndRetriesFromInstance()
+  def GetProp(self, property_name, timeout=None, retries=None):
+    """Gets a property from the device.
+
+    Args:
+      property_name: A string containing the name of the property to get from
+                     the device.
+      timeout: Same as for |IsOnline|.
+      retries: Same as for |IsOnline|.
+    Returns:
+      The value of the device's |property_name| property.
+    """
+    return self.old_interface.system_properties[property_name]
+
+  @decorators.WithTimeoutAndRetriesFromInstance()
+  def SetProp(self, property_name, value, timeout=None, retries=None):
+    """Sets a property on the device.
+
+    Args:
+      property_name: A string containing the name of the property to set on
+                     the device.
+      value: A string containing the value to set to the property on the
+             device.
+      timeout: Same as for |IsOnline|.
+      retries: Same as for |IsOnline|.
+    """
+    self.old_interface.system_properties[property_name] = value
+
   def __str__(self):
     """Returns the device serial."""
     return self.old_interface.GetDevice()