Convert prototype traversal to an iterative loop instead of recursive.

dilijev · dilijev · commit 50e9328e4e3e · 2016-04-29T15:32:16.000-07:00
diff --git a/lib/Runtime/Language/JavascriptOperators.cpp b/lib/Runtime/Language/JavascriptOperators.cpp
@@ -10032,22 +10032,23 @@ namespace Js
         return true;
     }
 
+    // Checks to see if the specified object (which should be a prototype object)
+    // contains a proxy anywhere in the prototype chain.
     bool JavascriptOperators::CheckIfPrototypeChainContainsProxyObject(RecyclableObject* prototype)
     {
         Assert(JavascriptOperators::IsObjectOrNull(prototype));
 
-        if (prototype->GetTypeId() == TypeIds_Null)
+        while (prototype->GetTypeId() != TypeIds_Null)
         {
-            return false;
-        }
-        else if (prototype->GetTypeId() == TypeIds_Proxy)
-        {
-            return true;
-        }
-        else
-        {
-            return CheckIfPrototypeChainContainsProxyObject(prototype->GetPrototype());
+            if (prototype->GetTypeId() == TypeIds_Proxy)
+            {
+                return true;
+            }
+
+            prototype = prototype->GetPrototype();
         }
+
+        return false;
     }
 
     BOOL JavascriptOperators::Equal(Var aLeft, Var aRight, ScriptContext* scriptContext)
