[Interpreter] Updates InlineTwice in test-run-inlining to match other…

… tests.

Updates InlineTwice to declare a function and then return a function
instead of using function expressions by wrapping a function with '('
and ')'. The earlier implementation would cause the function to
compile immediately instead of lazy compile. Also updates cctest.status

BUG=v8:4280,v8:4837,v8:4680
LOG=N

Review URL: https://codereview.chromium.org/1800073002

Cr-Commit-Position: refs/heads/master@{#34790}

test/cctest/cctest.status

@@ -549,10 +549,6 @@
 
   # TODO(mythria,4680): Incorrect column number on eval.
   'test-api/PromiseRejectCallback': [FAIL],
-
-  # TODO(rmcilroy,4680): Test assert errors.
-  'test-run-inlining/InlineTwice': [FAIL],
-
 }],  # ignition == True
 
 ['ignition == True and msan', {

test/cctest/compiler/test-run-inlining.cc

@@ -268,7 +268,8 @@ TEST(InlineTwice) {
       "(function () {"
       "  var x = 42;"
       "  function bar(s) { AssertInlineCount(2); return x + s; };"
-      "  return (function (s,t) { return bar(s) + bar(t); });"
+      "  function foo(s, t) { return bar(s) + bar(t); };"
+      "  return foo;"
       "})();",
       kInlineFlags);