[orca] Cache results of shouldInferLabelFor()

[Joanmarie Diggs <joanied src gnome org>]

commit b515173e945ce225115a604f4d5a21fb8f2361fd
Author: Joanmarie Diggs <jdiggs igalia com>
Date:   Wed May 4 18:28:50 2016 -0400

    Cache results of shouldInferLabelFor()

 src/orca/scripts/web/script_utilities.py |   51 ++++++++++++++---------------
 1 files changed, 25 insertions(+), 26 deletions(-)
---
diff --git a/src/orca/scripts/web/script_utilities.py b/src/orca/scripts/web/script_utilities.py
index 788db4e..7dc2381 100644
--- a/src/orca/scripts/web/script_utilities.py
+++ b/src/orca/scripts/web/script_utilities.py
@@ -77,6 +77,7 @@ class Utilities(script_utilities.Utilities):
         self._displayedLabelText = {}
         self._roleDescription = {}
         self._shouldFilter = {}
+        self._shouldInferLabelFor = {}
         self._text = {}
         self._tag = {}
         self._treatAsDiv = {}
@@ -128,6 +129,7 @@ class Utilities(script_utilities.Utilities):
         self._displayedLabelText = {}
         self._roleDescription = {}
         self._shouldFilter = {}
+        self._shouldInferLabelFor = {}
         self._tag = {}
         self._treatAsDiv = {}
         self._posinset = {}
@@ -2340,45 +2342,42 @@ class Utilities(script_utilities.Utilities):
         return rv
 
     def shouldInferLabelFor(self, obj):
+        if not self.inDocumentContent() or self.inTopLevelWebApp():
+            return False
+
+        rv = self._shouldInferLabelFor.get(hash(obj))
+        if rv:
+            return not self._script.inSayAll()
+        if rv == False:
+            return rv
+
         try:
+            role = obj.getRole()
             name = obj.name
         except:
-            msg = "WEB: Exception getting name for %s" % obj
+            msg = "WEB: Exception getting role and name for %s" % obj
             debug.println(debug.LEVEL_INFO, msg, True)
+            rv = False
         else:
             if name:
-                return False
-
-        if self._script.inSayAll():
-            return False
+                rv = False
+            else:
+                roles =  [pyatspi.ROLE_CHECK_BOX,
+                          pyatspi.ROLE_COMBO_BOX,
+                          pyatspi.ROLE_ENTRY,
+                          pyatspi.ROLE_LIST_BOX,
+                          pyatspi.ROLE_PASSWORD_TEXT,
+                          pyatspi.ROLE_RADIO_BUTTON]
+                rv = role in roles and not self.displayedLabel(obj)
 
-        if not self.inDocumentContent() or self.inTopLevelWebApp():
-            return False
-        try:
-            role = obj.getRole()
-        except:
-            msg = "WEB: Exception getting role for %s" % obj
-            debug.println(debug.LEVEL_INFO, msg, True)
-            return False
+        self._shouldInferLabelFor[hash(obj)] = rv
 
         # TODO - JD: This is private.
         if self._script._lastCommandWasCaretNav \
            and role not in [pyatspi.ROLE_RADIO_BUTTON, pyatspi.ROLE_CHECK_BOX]:
             return False
 
-        roles =  [pyatspi.ROLE_CHECK_BOX,
-                  pyatspi.ROLE_COMBO_BOX,
-                  pyatspi.ROLE_ENTRY,
-                  pyatspi.ROLE_LIST_BOX,
-                  pyatspi.ROLE_PASSWORD_TEXT,
-                  pyatspi.ROLE_RADIO_BUTTON]
-        if role not in roles:
-            return False
-
-        if self.displayedLabel(obj):
-            return False
-
-        return True
+        return rv
 
     def displayedLabel(self, obj):
         if not (obj and self.inDocumentContent(obj)):