Remove dangerous constructor from CompilationInfoWithZone.

test/cctest/compiler/test-instruction.cc

Index: test/cctest/compiler/test-instruction.cc
diff --git a/test/cctest/compiler/test-instruction.cc b/test/cctest/compiler/test-instruction.cc
index 03d8660deb593541f9ae8693a219333e0ce2e607..92db31e2eb0874e14a75c0b885c0b5277ae9d0f3 100644
--- a/test/cctest/compiler/test-instruction.cc
+++ b/test/cctest/compiler/test-instruction.cc
@@ -31,7 +31,7 @@ class InstructionTester : public HandleAndZoneScope {
         graph(zone()),
         schedule(zone()),
         fake_stub(main_isolate()),
-        info(&fake_stub, main_isolate()),
+        info(&fake_stub, main_isolate(), zone()),
         common(zone()),
         machine(zone()),
         code(NULL) {}
@@ -40,7 +40,7 @@ class InstructionTester : public HandleAndZoneScope {
   Graph graph;
   Schedule schedule;
   FakeStubForTesting fake_stub;
-  CompilationInfoWithZone info;
+  CompilationInfo info;

[titzer, 2015/03/23 17:04:34]

Do we even need this field anymore?

[Michael Starzinger, 2015/03/23 17:10:54]

Done. Same for some other fields.

   CommonOperatorBuilder common;
   MachineOperatorBuilder machine;
   TestInstrSeq* code;