Add handle scopes around executeScriptAndReturnValue callsites

content/renderer/browser_plugin/browser_plugin_browsertest.cc

 // Copyright (c) 2012 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 "content/renderer/browser_plugin/browser_plugin_browsertest.h"
 
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/memory/singleton.h"
 #include "base/path_service.h"
@@ skipping 81 matching lines @@
 }
 
 void BrowserPluginTest::TearDown() {
   BrowserPluginManager::set_factory_for_testing(
       TestBrowserPluginManagerFactory::GetInstance());
   content::RenderViewTest::TearDown();
 }
 
 std::string BrowserPluginTest::ExecuteScriptAndReturnString(
     const std::string& script) {
+  v8::HandleScope handle_scope;
   v8::Handle<v8::Value> value = GetMainFrame()->executeScriptAndReturnValue(
       WebKit::WebScriptSource(WebKit::WebString::fromUTF8(script.c_str())));
   if (value.IsEmpty() || !value->IsString())
     return std::string();
 
   v8::Local<v8::String> v8_str = value->ToString();
   int length = v8_str->Utf8Length() + 1;
   scoped_array<char> str(new char[length]);
   v8_str->WriteUtf8(str.get(), length);
   return str.get();
 }
 
 int BrowserPluginTest::ExecuteScriptAndReturnInt(
     const std::string& script) {
+  v8::HandleScope handle_scope;
   v8::Handle<v8::Value> value = GetMainFrame()->executeScriptAndReturnValue(
       WebKit::WebScriptSource(WebKit::WebString::fromUTF8(script.c_str())));
   if (value.IsEmpty() || !value->IsInt32())
     return 0;
 
   return value->Int32Value();
 }
 
 // A return value of false means that a value was not present. The return value
 // of the script is stored in |result|
 bool BrowserPluginTest::ExecuteScriptAndReturnBool(
     const std::string& script, bool* result) {
+  v8::HandleScope handle_scope;
   v8::Handle<v8::Value> value = GetMainFrame()->executeScriptAndReturnValue(
       WebKit::WebScriptSource(WebKit::WebString::fromUTF8(script.c_str())));
   if (value.IsEmpty() || !value->IsBoolean())
     return false;
 
   *result = value->BooleanValue();
   return true;
 }
 
 // This test verifies that an initial resize occurs when we instantiate the
@@ skipping 755 matching lines @@
     // These value are not populated (as an optimization) if autosize is
     // disabled.
     EXPECT_EQ(0, auto_size_params.min_size.width());
     EXPECT_EQ(0, auto_size_params.min_size.height());
     EXPECT_EQ(0, auto_size_params.max_size.width());
     EXPECT_EQ(0, auto_size_params.max_size.height());
   }
 }
 
 }  // namespace content