Use mayNotBeMainThread() for wrapper optimization

third_party/WebKit/Source/core/dom/RangeTest.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/dom/Range.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "core/dom/Element.h"
 #include "core/dom/NodeList.h"
 #include "core/dom/Text.h"
@@ skipping 45 matching lines @@
   Element* const result = document().createElement("div");
   result->appendChild(range->extractContents(ASSERT_NO_EXCEPTION));
 
   EXPECT_EQ("<b>abc</b>", result->innerHTML())
       << "DOM mutation event handler should not affect result.";
   EXPECT_EQ("<span>DEF</span>", spanElement->outerHTML())
       << "DOM mutation event handler should be executed.";
 }
 
 TEST_F(RangeTest, SplitTextNodeRangeWithinText) {
+  ScriptState* scriptState = ScriptState::forMainWorld(&frame());
+  ScriptState::Scope scope(scriptState);

[haraken, 2017/03/08 09:25:25]

Can we use V8TestingScope instead?

The same comment for other test files.

[adithyas, 2017/03/08 18:20:10]

Yup, fixed.

+
   document().body()->setInnerHTML("1234");
   Text* oldText = toText(document().body()->firstChild());
 
   Range* range04 = Range::create(document(), oldText, 0, oldText, 4);
   Range* range02 = Range::create(document(), oldText, 0, oldText, 2);
   Range* range22 = Range::create(document(), oldText, 2, oldText, 2);
   Range* range24 = Range::create(document(), oldText, 2, oldText, 4);
 
   oldText->splitText(2, ASSERT_NO_EXCEPTION);
   Text* newText = toText(oldText->nextSibling());
@@ skipping 19 matching lines @@
   EXPECT_EQ(2u, range22->endOffset());
 
   EXPECT_TRUE(range24->boundaryPointsValid());
   EXPECT_EQ(oldText, range24->startContainer());
   EXPECT_EQ(2u, range24->startOffset());
   EXPECT_EQ(newText, range24->endContainer());
   EXPECT_EQ(2u, range24->endOffset());
 }
 
 TEST_F(RangeTest, SplitTextNodeRangeOutsideText) {
+  ScriptState* scriptState = ScriptState::forMainWorld(&frame());
+  ScriptState::Scope scope(scriptState);
+
   document().body()->setInnerHTML(
       "<span id=\"outer\">0<span id=\"inner-left\">1</span>SPLITME<span "
       "id=\"inner-right\">2</span>3</span>");
 
   Element* outer = document().getElementById(AtomicString::fromUTF8("outer"));
   Element* innerLeft =
       document().getElementById(AtomicString::fromUTF8("inner-left"));
   Element* innerRight =
       document().getElementById(AtomicString::fromUTF8("inner-right"));
   Text* oldText = toText(outer->childNodes()->item(2));
@@ skipping 104 matching lines @@
   text->deleteData(0, 3, ASSERT_NO_EXCEPTION);
 
   EXPECT_TRUE(range->boundaryPointsValid());
   EXPECT_EQ(span2, range->startContainer());
   EXPECT_EQ(0u, range->startOffset());
   EXPECT_EQ(div, range->endContainer());
   EXPECT_EQ(2u, range->endOffset());
 }
 
 }  // namespace blink