Move CompileString to Compiler

src/compiler.h

Index: src/compiler.h
diff --git a/src/compiler.h b/src/compiler.h
index e2f71479407bccbd139518236500017f7ae0f121..71b01143737a949019b4ca0e3000fbd0c2a54e12 100644
--- a/src/compiler.h
+++ b/src/compiler.h
@@ -93,6 +93,10 @@ class Compiler : public AllStatic {
       ScriptCompiler::CompileOptions compile_options,
       NativesFlag is_natives_code, bool is_module);
 
+  MUST_USE_RESULT static MaybeHandle<JSFunction> CompileString(

[Michael Starzinger, 2016/07/20 12:03:35]

Can we call this "GetFunctomFromString" instead. The family of CompileFoo
methods is intended to compile existing functions and mutate them to have
appropriate machine code or byte code. Whereas this family of methods actually
instantiates new functions.

Also let's move this up one function, to after GetFunctionFromEval, as it is
closely related to that.

Also, same comment as for GetFunctionFromEval applies to this.

[Michael Starzinger, 2016/07/20 12:05:12]

s/GetFunctomFromString/GetFunctionFromString/ of course. :)

[jgruber, 2016/07/20 12:42:11]

Done.

+      Handle<Context> context, Handle<String> source,
+      ParseRestriction restriction);
+
   // Create a shared function info object for a Script that has already been
   // parsed while the script was being loaded from a streamed source.
   static Handle<SharedFunctionInfo> GetSharedFunctionInfoForStreamedScript(