winaccessibility: remove GetXAccByAccObj()

It is pointlessly slow and the AccObject has a method for that.

Change-Id: I7ba1cc853255ac3b3b5008657d1cc79efc3a3f4b

2 files changed, 2 insertions, 21 deletions

diff --git a/winaccessibility/source/service/AccObject.cxx b/winaccessibility/source/service/AccObject.cxx
index ac51a0527348..39a95348da83 100644
--- a/winaccessibility/source/service/AccObject.cxx
+++ b/winaccessibility/source/service/AccObject.cxx
@@ -1152,7 +1152,7 @@ IMAccessible*  AccObject::GetIMAccessible()
     return m_pIMAcc;
 }
 
-Reference < XAccessible > AccObject::GetXAccessible()
+Reference<XAccessible> const& AccObject::GetXAccessible()
 {
     return m_xAccRef;
 }
diff --git a/winaccessibility/source/service/AccObjectWinManager.cxx b/winaccessibility/source/service/AccObjectWinManager.cxx
index a058683c355d..cd51b7e37ebe 100644
--- a/winaccessibility/source/service/AccObjectWinManager.cxx
+++ b/winaccessibility/source/service/AccObjectWinManager.cxx
@@ -173,24 +173,6 @@ AccObject* AccObjectWinManager::GetAccObjByXAcc( XAccessible* pXAcc)
 }
 
 /**
-   * Search XAccessible by AccObject pointer from our container.
-   * @param pAccObj AccObject pointer.
-   * @return Pointer of XAccessible Interface.
-   */
-XAccessible* AccObjectWinManager::GetXAccByAccObj(AccObject* pAccObj)
-{
-    XIdToAccObjHash::iterator iter = XIdAccList.begin();
-    while(iter!=XIdAccList.end())
-    {
-        AccObject* tmp = &(iter->second);
-        if(tmp== pAccObj)
-            return (XAccessible*)(iter->first);
-        ++iter;
-    }
-    return NULL;
-}
-
-/**
    * get acc object of top window by its handle
    * @param hWnd, top window handle
    * @return pointer to AccObject
@@ -563,7 +545,6 @@ void AccObjectWinManager::DeleteChildrenAccObj(XAccessible* pXAcc)
 {
     AccObject* currentObj=NULL;
     AccObject* childObj=NULL;
-    XAccessible* pTmpXAcc=NULL;
 
     currentObj =  GetAccObjByXAcc( pXAcc);
     if(currentObj)
@@ -571,7 +552,7 @@ void AccObjectWinManager::DeleteChildrenAccObj(XAccessible* pXAcc)
         childObj = currentObj->NextChild();
         while(childObj)
         {
-            pTmpXAcc = GetXAccByAccObj(childObj);
+            XAccessible *const pTmpXAcc = childObj->GetXAccessible().get();
             if(pTmpXAcc)
             {
                 DeleteChildrenAccObj(pTmpXAcc);