[MERGE #5934 @MikeHolman] reenable ObjectCopyPhase

Merge pull request #5934 from MikeHolman:reenableobjcopy ObjectCopyPhase was inadvertently disabled during a merge a few months ago
chakra-core · Feb 12, 2019 · c95f834 · c95f834
2 parents c642e9f + ab4ef72
commit c95f834
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/Runtime/Types/DynamicObject.cpp b/lib/Runtime/Types/DynamicObject.cpp
@@ -914,7 +914,7 @@ namespace Js
         }
 #endif
 
-        if (!PHASE_ON1(ObjectCopyPhase))
+        if (PHASE_OFF1(ObjectCopyPhase))
         {
             return false;
         }

diff --git a/test/Object/rlexe.xml b/test/Object/rlexe.xml
@@ -452,7 +452,7 @@
   <test>
     <default>
       <files>assign.js</files>
-      <compile-flags>-args summary -endargs -on:ObjectCopy -trace:ObjectCopy</compile-flags>
+      <compile-flags>-args summary -endargs -trace:ObjectCopy</compile-flags>
       <baseline>assign.baseline</baseline>
     </default>
   </test>
-Original file line number
+Diff line change
@@ Expand Up / @@ -914,7 +914,7 @@ namespace Js @@
             }
     #endif
-            if (!PHASE_ON1(ObjectCopyPhase))
+            if (PHASE_OFF1(ObjectCopyPhase))
             {
                 return false;
             }
@@ Expand Down @@