Remove some duplication in code that extracts the source for a function.

runtime/vm/compiler_test.cc

Index: runtime/vm/compiler_test.cc
diff --git a/runtime/vm/compiler_test.cc b/runtime/vm/compiler_test.cc
index 43649e33c26ce1e1b90f53df98e2f8997d83514c..1351fbc41a94434f977871637f05a8ccc51adea5 100644
--- a/runtime/vm/compiler_test.cc
+++ b/runtime/vm/compiler_test.cc
@@ -54,7 +54,7 @@ TEST_CASE(CompileFunction) {
       Function::Handle(cls.LookupStaticFunction(function_foo_name));
   EXPECT(!function_foo.IsNull());
   String& function_source = String::Handle(function_foo.GetSource());
-  EXPECT_STREQ("static foo() { return 42; }\n  ", function_source.ToCString());
+  EXPECT_STREQ("static foo() { return 42; }", function_source.ToCString());
   EXPECT(CompilerTest::TestCompileFunction(function_foo));
   EXPECT(function_foo.HasCode());
 
@@ -66,7 +66,7 @@ TEST_CASE(CompileFunction) {
   EXPECT(CompilerTest::TestCompileFunction(function_moo));
   EXPECT(function_moo.HasCode());
   function_source = function_moo.GetSource();
-  EXPECT_STREQ("static moo() {\n    // A.foo();\n  }\n",
+  EXPECT_STREQ("static moo() {\n    // A.foo();\n  }",
                function_source.ToCString());

[siva, 2015/04/14 18:00:45]

maybe better to retain the new line as it would look cleaner when an error
happens.

 }