[cleanup] Fix uses deprecated v8 Object::Get in gin

Bug: v8:7283, v8:8562
Change-Id: I037bfb3818fdb9f92d5d9e0a8a71152ccdddade3
Reviewed-on: https://chromium-review.googlesource.com/c/1450255
Auto-Submit: Dan Elphick <delphick@chromium.org>
Commit-Queue: Ross McIlroy <rmcilroy@chromium.org>
Reviewed-by: Ross McIlroy <rmcilroy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#628380}

gin/converter_unittest.cc

@@ -139,9 +139,12 @@ TEST_F(ConverterTest, Vector) {
       Converter<std::vector<int>>::ToV8(instance_->isolate(), expected)
           .As<Array>();
   EXPECT_EQ(3u, js_array->Length());
+  v8::Local<v8::Context> context = instance_->isolate()->GetCurrentContext();
   for (size_t i = 0; i < expected.size(); ++i) {
-    EXPECT_TRUE(Integer::New(instance_->isolate(), expected[i])
-                    ->StrictEquals(js_array->Get(static_cast<int>(i))));
+    EXPECT_TRUE(
+        Integer::New(instance_->isolate(), expected[i])
+            ->StrictEquals(
+                js_array->Get(context, static_cast<int>(i)).ToLocalChecked()));
   }
 }
 

gin/shell_runner_unittest.cc

@@ -45,8 +45,11 @@ TEST(RunnerTest, Run) {
   runner.Run(source, "test_data.js");
 
   std::string result;
-  EXPECT_TRUE(Converter<std::string>::FromV8(isolate,
-      runner.global()->Get(StringToV8(isolate, "result")),
+  EXPECT_TRUE(Converter<std::string>::FromV8(
+      isolate,
+      runner.global()
+          ->Get(isolate->GetCurrentContext(), StringToV8(isolate, "result"))
+          .ToLocalChecked(),
       &result));
   EXPECT_EQ("PASS", result);
 }