Add a DevTools command to create an isolated world for a given frame

third_party/WebKit/Source/bindings/core/v8/ScriptController.cpp

 /*
  * Copyright (C) 2008, 2009 Google Inc. All rights reserved.
  * Copyright (C) 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2014 Opera Software ASA. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
@@ skipping 42 matching lines @@
 #include "core/loader/NavigationScheduler.h"
 #include "core/loader/ProgressTracker.h"
 #include "core/plugins/PluginView.h"
 #include "core/probe/CoreProbes.h"
 #include "platform/FrameViewBase.h"
 #include "platform/Histogram.h"
 #include "platform/UserGestureIndicator.h"
 #include "platform/instrumentation/tracing/TraceEvent.h"
 #include "platform/weborigin/SecurityOrigin.h"
 #include "platform/wtf/CurrentTime.h"
+#include "platform/wtf/HashSet.h"
 #include "platform/wtf/StdLibExtras.h"
 #include "platform/wtf/StringExtras.h"
 #include "platform/wtf/text/CString.h"
 #include "platform/wtf/text/StringBuilder.h"
 
 namespace blink {
 
 DEFINE_TRACE(ScriptController) {
   visitor->Trace(frame_);
   visitor->Trace(window_proxy_manager_);
@@ skipping 291 matching lines @@
   if (results) {
     for (size_t i = 0; i < result_array->Length(); ++i) {
       v8::Local<v8::Value> value;
       if (!result_array->Get(context, i).ToLocal(&value))
         return;
       results->push_back(value);
     }
   }
 }
 
+int ScriptController::CreateNewIsolatedWorld(const String& world_name) {
+  // Find an unused id, unfortunately there doesn't seem to be a better way of
+  // doing this because DOMWrapperWorld::Create doesn't support isolated worlds.
+  Vector<RefPtr<DOMWrapperWorld>> worlds;
+  DOMWrapperWorld::AllWorldsInCurrentThread(worlds);
+  WTF::HashSet<int> existingWorlds;
+  for (const auto& world : worlds) {
+    if (!world->IsIsolatedWorld())
+      continue;
+    existingWorlds.insert(world->GetWorldId());
+  }
+  int world_id = 1;

[caseq, 2017/04/28 21:41:30]

I think allocating ids like this is going to bring some unpleasant surprises due
to collisions with other world id users that presume authority over their id
range and would not check if the world already exists. 

[Sami, 2017/05/02 09:47:19]

Is there some registry of world ids? Should we just start at a higher number to
make collisions less likely? In any case, I would prefer to have the
implementation allocate ids instead of assuming users (e.g., different
libraries) can keep their ids straight.

[alex clarke (OOO till 29th), 2017/05/02 10:00:06]

OK do you have any suggestions for what else we might do?

+  while (existingWorlds.Contains(world_id)) {
+    world_id++;
+  }
+  DCHECK_LT(world_id, DOMWrapperWorld::kEmbedderWorldIdLimit);
+  RefPtr<DOMWrapperWorld> world =
+      DOMWrapperWorld::EnsureIsolatedWorld(GetIsolate(), world_id);
+
+  DOMWrapperWorld::SetIsolatedWorldHumanReadableName(world_id, world_name);
+  // Make sure the execution context exists.
+  WindowProxy(*world);
+  return world_id;
+}
+
 }  // namespace blink