commit 764d342c053e018eda1607a2be5c4ea1d51b4743
author Dan Elphick <delphick@chromium.org> Tue Apr 23 10:28:55 2019
committer Commit Bot <commit-bot@chromium.org> Tue Apr 23 10:28:55 2019
tree aa251c55f01ef4a6b21c8b22ee4022a1ba4685e7
parent 0cbed54f7a63d80ccc22b5cb331c224f9f4e3e08

[v8] Fix use of deprecated Object::Set in supervised_user_error_page

Uses the context version of Set and checks the return value (which
cannot actually fail in this case but the API requires it).

Bug: v8:7283
Change-Id: Ib22e132cee49da508a00a811f5bd67e4f5903ff2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1576570
Auto-Submit: Dan Elphick <delphick@chromium.org>
Commit-Queue: Marc Treib <treib@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#653120}

components/supervised_user_error_page/gin_wrapper.cc

diff --git a/components/supervised_user_error_page/gin_wrapper.cc b/components/supervised_user_error_page/gin_wrapper.cc
index fe4d773..bb17510 100644
--- a/components/supervised_user_error_page/gin_wrapper.cc
+++ b/components/supervised_user_error_page/gin_wrapper.cc
@@ -47,7 +47,10 @@
   if (controller.IsEmpty())
     return;
   v8::Local<v8::Object> global = context->Global();
-  global->Set(gin::StringToV8(isolate, "webRestrictions"), controller.ToV8());
+  global
+      ->Set(context, gin::StringToV8(isolate, "webRestrictions"),
+            controller.ToV8())
+      .Check();
 }
 
 void GinWrapper::Loader::OnDestruct() {