Prevent reentrancy of WebFrameClient::didClearWindowObject.

Source/web/tests/WebFrameTest.cpp

 /*
  * Copyright (C) 2010 Google Inc. 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
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 393 matching lines @@
     webViewHelper.initializeAndLoad(m_baseURL + fileName, true);
 
     FrameTestHelpers::loadFrame(webViewHelper.webView()->mainFrame(), "javascript:location.port = ''; void 0;");
 
     FrameTestHelpers::loadFrame(webViewHelper.webView()->mainFrame(), "javascript:document.body.textContent = location.href; void 0;");
 
     std::string content = webViewHelper.webView()->mainFrame()->contentAsText(1024).utf8();
     EXPECT_EQ("http://internal.test:0/" + fileName, content);
 }
 
+class EvaluateOnLoadWebFrameClient : public FrameTestHelpers::TestWebFrameClient {
+public:
+    EvaluateOnLoadWebFrameClient() : m_executing(false), m_wasExecuted(false) { }
+
+    void didClearWindowObject(WebLocalFrame* frame) override
+    {
+        EXPECT_FALSE(m_executing);
+        m_wasExecuted = true;
+        m_executing = true;
+        v8::HandleScope handleScope(v8::Isolate::GetCurrent());
+        frame->executeScriptAndReturnValue(WebScriptSource(WebString("window.someProperty = 42;")));
+        m_executing = false;
+    }
+
+    bool m_executing;
+    bool m_wasExecuted;
+};
+
+TEST_P(ParametrizedWebFrameTest, DidClearWindowObjectIsNotRecursive)
+{
+    EvaluateOnLoadWebFrameClient webFrameClient;
+    FrameTestHelpers::WebViewHelper webViewHelper(this);
+    webViewHelper.initializeAndLoad("about:blank", true, &webFrameClient);
+    EXPECT_TRUE(webFrameClient.m_wasExecuted);
+}
+
 class CSSCallbackWebFrameClient : public FrameTestHelpers::TestWebFrameClient {
 public:
     CSSCallbackWebFrameClient() : m_updateCount(0) { }
     virtual void didMatchCSS(WebLocalFrame*, const WebVector<WebString>& newlyMatchingSelectors, const WebVector<WebString>& stoppedMatchingSelectors) override;
 
     std::map<WebLocalFrame*, std::set<std::string>> m_matchedSelectors;
     int m_updateCount;
 };
 
 void CSSCallbackWebFrameClient::didMatchCSS(WebLocalFrame* frame, const WebVector<WebString>& newlyMatchingSelectors, const WebVector<WebString>& stoppedMatchingSelectors)
@@ skipping 6981 matching lines @@
 
     EXPECT_EQ(parent, firstFrame->parent());
     EXPECT_EQ(parent, secondFrame->parent());
     EXPECT_EQ(parent, thirdFrame->parent());
     EXPECT_EQ(parent, fourthFrame->parent());
 
     view->close();
 }
 
 } // namespace blink