Bug fix when proxy is from different context

kunalspathak · kunalspathak · commit 4d74acc6f18a · 2016-09-09T11:49:08.000-07:00
We set `target` only if proxy is from same context, but while asserting we were not taking cross-context into account.
diff --git a/lib/Runtime/Library/JavascriptArray.cpp b/lib/Runtime/Library/JavascriptArray.cpp
@@ -4252,7 +4252,7 @@ namespace Js
         [&](bool/*hasException*/)
         {
             Var top = scriptContext->PopObject();
-            if (JavascriptProxy::Is(thisArg))
+            if (isProxy)
             {
                 AssertMsg(top == target, "Unmatched operation stack");
             }

Original file line number	Diff line number	Diff line change
`@@ -4252,7 +4252,7 @@ namespace Js`
`4252`	`4252`	`[&](bool/hasException/)`
`4253`	`4253`	`{`
`4254`	`4254`	`Var top = scriptContext->PopObject();`
`4255`		`- if (JavascriptProxy::Is(thisArg))`
	`4255`	`+ if (isProxy)`
`4256`	`4256`	`{`
`4257`	`4257`	`AssertMsg(top == target, "Unmatched operation stack");`
`4258`	`4258`	`}`