Cosmetic changes to tests to make it easier to concatenate them.

test/cctest/compiler/test-graph-visualizer.cc

 // Copyright 2014 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/v8.h"
 #include "test/cctest/cctest.h"
 
 #include "src/compiler/common-operator.h"
 #include "src/compiler/graph.h"
 #include "src/compiler/graph-visualizer.h"
 #include "src/compiler/js-operator.h"
 #include "src/compiler/machine-operator.h"
 #include "src/compiler/node.h"
 #include "src/compiler/operator.h"
 #include "src/compiler/schedule.h"
 #include "src/compiler/scheduler.h"
 #include "src/compiler/source-position.h"
 #include "src/compiler/verifier.h"
 
+namespace test_graph_visualizer {

[ulan, 2015/06/01 13:08:24]

Now some tests are namespaced and others are not.

I think this should be done in concatenation script: insert "namespace
${test-file} {" before "using namespace v8::internal;".

[Erik Corry, 2015/06/01 15:15:47]

test namespaces removed

+
 using namespace v8::internal;
 using namespace v8::internal::compiler;
 
 static Operator dummy_operator(IrOpcode::kParameter, Operator::kNoWrite,
                                "dummy", 0, 0, 0, 1, 0, 0);
 
 
 TEST(NodeWithNullInputReachableFromEnd) {
   HandleAndZoneScope scope;
   Graph graph(scope.main_zone());
@@ skipping 88 matching lines @@
   Node* n11 = graph.NewNode(&dummy_operator, n9);
   Node* end_dependencies[6] = {n4, n8, n10, n11, n6, n7};
   Node* end = graph.NewNode(&dummy_operator, 6, end_dependencies);
   graph.SetEnd(end);
 
   OFStream os(stdout);
   os << AsDOT(graph);
   SourcePositionTable table(&graph);
   os << AsJSON(graph, &table);
 }
+
+}  // namespace test_graph_visualizer