[iwyu] api.h iwyu

This reduces the build steps from touching api.h: 269 -> 156

BUG=v8:7754,v8:7490

Change-Id: I75abaeea4cc78027a47304ff9b9f6b12bdb2b75e
Reviewed-on: https://chromium-review.googlesource.com/1144929
Reviewed-by: Clemens Hammacher <clemensh@chromium.org>
Reviewed-by: Yang Guo <yangguo@chromium.org>
Commit-Queue: Marja Hölttä <marja@chromium.org>
Cr-Commit-Position: refs/heads/master@{#54583}

src/ast/ast-value-factory.cc

@@ -27,7 +27,6 @@
 
 #include "src/ast/ast-value-factory.h"
 
-#include "src/api.h"
 #include "src/char-predicates-inl.h"
 #include "src/objects-inl.h"
 #include "src/objects.h"

src/elements-kind.cc

@@ -4,7 +4,6 @@
 
 #include "src/elements-kind.h"
 
-#include "src/api.h"
 #include "src/base/lazy-instance.h"
 #include "src/elements.h"
 #include "src/objects-inl.h"

src/objects/js-locale.h

@@ -9,7 +9,6 @@
 #ifndef V8_OBJECTS_JS_LOCALE_H_
 #define V8_OBJECTS_JS_LOCALE_H_
 
-#include "src/api.h"
 #include "src/global-handles.h"
 #include "src/heap/factory.h"
 #include "src/isolate.h"

src/optimized-compilation-info.cc

@@ -5,12 +5,10 @@
 #include "src/optimized-compilation-info.h"
 
 #include "src/api.h"
-#include "src/ast/ast.h"
-#include "src/ast/scopes.h"
 #include "src/debug/debug.h"
 #include "src/isolate.h"
 #include "src/objects-inl.h"
-#include "src/parsing/parse-info.h"
+#include "src/objects/shared-function-info.h"
 #include "src/source-position.h"
 
 namespace v8 {

src/parsing/parse-info.cc

@@ -4,7 +4,6 @@
 
 #include "src/parsing/parse-info.h"
 
-#include "src/api.h"
 #include "src/ast/ast-source-ranges.h"
 #include "src/ast/ast-value-factory.h"
 #include "src/ast/ast.h"

src/snapshot/snapshot-common.cc

@@ -6,7 +6,6 @@
 
 #include "src/snapshot/snapshot.h"
 
-#include "src/api.h"
 #include "src/assembler-inl.h"
 #include "src/base/platform/platform.h"
 #include "src/callable.h"

src/unoptimized-compilation-info.cc

@@ -4,7 +4,6 @@
 
 #include "src/unoptimized-compilation-info.h"
 
-#include "src/api.h"
 #include "src/ast/ast.h"
 #include "src/ast/scopes.h"
 #include "src/debug/debug.h"

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

@@ -2,7 +2,6 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "src/api.h"
 #include "src/contexts.h"
 #include "src/flags.h"
 #include "src/objects-inl.h"

test/cctest/test-dictionary.cc

@@ -28,7 +28,6 @@
 #include "src/v8.h"
 #include "test/cctest/cctest.h"
 
-#include "src/api.h"
 #include "src/builtins/builtins-constructor.h"
 #include "src/debug/debug.h"
 #include "src/execution.h"

test/cctest/test-global-object.cc

@@ -25,7 +25,6 @@
 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
-#include "src/api.h"
 #include "src/objects-inl.h"
 #include "src/v8.h"
 #include "test/cctest/cctest.h"

test/cctest/test-hashcode.cc

@@ -6,7 +6,6 @@
 #include <sstream>
 #include <utility>
 
-#include "src/api.h"
 #include "src/objects-inl.h"
 #include "src/objects.h"
 #include "src/v8.h"

test/cctest/test-lockers.cc

@@ -31,7 +31,6 @@
 
 #include "src/v8.h"
 
-#include "src/api.h"
 #include "src/base/platform/platform.h"
 #include "src/compilation-cache.h"
 #include "src/execution.h"

test/cctest/test-typedarrays.cc

@@ -7,7 +7,6 @@
 #include "src/v8.h"
 #include "test/cctest/cctest.h"
 
-#include "src/api.h"
 #include "src/heap/heap.h"
 #include "src/objects-inl.h"
 #include "src/objects.h"

test/cctest/torque/test-torque.cc

@@ -4,7 +4,6 @@
 
 #include <cmath>
 
-#include "src/api.h"
 #include "src/base/utils/random-number-generator.h"
 #include "src/builtins/builtins-promise-gen.h"
 #include "src/builtins/builtins-string-gen.h"

test/unittests/test-utils.cc

@@ -89,6 +89,10 @@ TestWithIsolateAndZone::~TestWithIsolateAndZone() {}
 
 Factory* TestWithIsolate::factory() const { return isolate()->factory(); }
 
+Handle<Object> TestWithIsolate::RunJSInternal(const char* source) {
+  return Utils::OpenHandle(*::v8::TestWithIsolate::RunJS(source));
+}
+
 base::RandomNumberGenerator* TestWithIsolate::random_number_generator() const {
   return isolate()->random_number_generator();
 }

test/unittests/test-utils.h

@@ -8,7 +8,6 @@
 #include <vector>
 
 #include "include/v8.h"
-#include "src/api.h"
 #include "src/base/macros.h"
 #include "src/base/utils/random-number-generator.h"
 #include "src/handles.h"
@@ -85,10 +84,9 @@ class TestWithIsolate : public virtual ::v8::TestWithIsolate {
   Isolate* isolate() const { return i_isolate(); }
   template <typename T = Object>
   Handle<T> RunJS(const char* source) {
-    Handle<Object> result =
+    return Handle<T>::cast(RunJSInternal(source));
-        Utils::OpenHandle(*::v8::TestWithIsolate::RunJS(source));
-    return Handle<T>::cast(result);
   }
+  Handle<Object> RunJSInternal(const char* source);
   base::RandomNumberGenerator* random_number_generator() const;
 
  private: