Revert of ApkHelper: Add GetViewActivityName(). Make GetActivityName() more strict

devil/devil/android/apk_helper.py

 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Module containing utilities for apk packages."""
 
 import itertools
 import re
 
 from devil import base_error
@@ skipping 59 matching lines @@
       if manifest_key in node:
         raise base_error.BaseError(
             "A single attribute should have one key and one value")
       else:
         node[manifest_key] = m.group(2) or m.group(3)
       continue
 
   return parsed_manifest
 
 
-def _ParseNumericKey(obj, key, default=0):
-  val = obj.get(key)
-  if val is None:
-    return default
-  return int(val, 0)
-
-
-class _ExportedActivity(object):
-  def __init__(self, name):
-    self.name = name
-    self.actions = set()
-    self.categories = set()
-    self.schemes = set()
-
-
-def _IterateExportedActivities(manifest_info):
-  app_node = manifest_info['manifest'][0]['application'][0]
-  activities = app_node.get('activity', []) + app_node.get('activity-alias', [])
-  for activity_node in activities:
-    if not _ParseNumericKey(activity_node, 'android:exported'):
-      continue
-    activity = _ExportedActivity(activity_node.get('android:name'))
-    # Merge all intent-filters into a single set because there is not
-    # currently a need to keep them separate.
-    for intent_filter in activity_node.get('intent-filter', []):
-      for action in intent_filter.get('action', []):
-        activity.actions.add(action.get('android:name'))
-      for category in intent_filter.get('category', []):
-        activity.categories.add(category.get('android:name'))
-      for data in intent_filter.get('data', []):
-        activity.schemes.add(data.get('android:scheme'))
-    yield activity
-
-
 class ApkHelper(object):
 
   def __init__(self, path):
     self._apk_path = path
     self._manifest = None
 
   @property
   def path(self):
     return self._apk_path
 
   def GetActivityName(self):
-    """Returns the name of the first launcher Activity in the apk."""
+    """Returns the name of the Activity in the apk."""
     manifest_info = self._GetManifest()
-    for activity in _IterateExportedActivities(manifest_info):
-      if ('android.intent.action.MAIN' in activity.actions and
-          'android.intent.category.LAUNCHER' in activity.categories):
-        return self._ResolveName(activity.name)
-    return None
-
-  def GetViewActivityName(self):
-    """Returns name of the first action=View Activity that can handle http."""
-    manifest_info = self._GetManifest()
-    for activity in _IterateExportedActivities(manifest_info):
-      if ('android.intent.action.VIEW' in activity.actions and
-          'http' in activity.schemes):
-        return self._ResolveName(activity.name)
-    return None
+    try:
+      activity = (
+          manifest_info['manifest'][0]['application'][0]['activity'][0]
+              ['android:name'])
+    except KeyError:
+      return None
+    if '.' not in activity:
+      activity = '%s.%s' % (self.GetPackageName(), activity)
+    elif activity.startswith('.'):
+      activity = '%s%s' % (self.GetPackageName(), activity)
+    return activity
 
   def GetInstrumentationName(
       self, default='android.test.InstrumentationTestRunner'):
     """Returns the name of the Instrumentation in the apk."""
     all_instrumentations = self.GetAllInstrumentations(default=default)
     if len(all_instrumentations) != 1:
       raise base_error.BaseError(
           'There is more than one instrumentation. Expected one.')
     else:
-      return self._ResolveName(all_instrumentations[0]['android:name'])
+      return all_instrumentations[0]['android:name']
 
   def GetAllInstrumentations(
       self, default='android.test.InstrumentationTestRunner'):
     """Returns a list of all Instrumentations in the apk."""
     try:
       return self._GetManifest()['manifest'][0]['instrumentation']
     except KeyError:
       return [{'android:name': default}]
 
   def GetPackageName(self):
@@ skipping 21 matching lines @@
       return None
 
   def HasIsolatedProcesses(self):
     """Returns whether any services exist that use isolatedProcess=true."""
     manifest_info = self._GetManifest()
     try:
       applications = manifest_info['manifest'][0].get('application', [])
       services = itertools.chain(
           *(application.get('service', []) for application in applications))
       return any(
-          _ParseNumericKey(s, 'android:isolatedProcess') for s in services)
+          int(s.get('android:isolatedProcess', '0'), 0)
+          for s in services)
     except KeyError:
       return False
 
   def _GetManifest(self):
     if not self._manifest:
       self._manifest = _ParseManifestFromApk(self._apk_path)
     return self._manifest
 
-  def _ResolveName(self, name):
-    name = name.lstrip('.')
-    if '.' not in name:
-      return '%s.%s' % (self.GetPackageName(), name)
-    return name