Index: content/public/test/browser_test_utils.cc |
diff --git a/content/public/test/browser_test_utils.cc b/content/public/test/browser_test_utils.cc |
index 8021ad4283fe63eb1d47a7a10adb5a0ca17dccac..721e0d1a19c24eb6e8a79a03c3ea99a9b287bca9 100644 |
--- a/content/public/test/browser_test_utils.cc |
+++ b/content/public/test/browser_test_utils.cc |
@@ -137,17 +137,17 @@ class InterstitialObserver : public content::WebContentsObserver { |
}; |
// Specifying a prototype so that we can add the WARN_UNUSED_RESULT attribute. |
-bool ExecuteScriptHelper( |
- RenderFrameHost* render_frame_host, |
- const std::string& original_script, |
- scoped_ptr<base::Value>* result) WARN_UNUSED_RESULT; |
+bool ExecuteScriptHelper(RenderFrameHost* render_frame_host, |
+ const std::string& original_script, |
+ std::unique_ptr<base::Value>* result) |
+ WARN_UNUSED_RESULT; |
// Executes the passed |original_script| in the frame specified by |
// |render_frame_host|. If |result| is not NULL, stores the value that the |
// evaluation of the script in |result|. Returns true on success. |
bool ExecuteScriptHelper(RenderFrameHost* render_frame_host, |
const std::string& original_script, |
- scoped_ptr<base::Value>* result) { |
+ std::unique_ptr<base::Value>* result) { |
// TODO(jcampan): we should make the domAutomationController not require an |
// automation id. |
std::string script = |
@@ -179,13 +179,13 @@ bool ExecuteScriptHelper(RenderFrameHost* render_frame_host, |
bool ExecuteScriptInIsolatedWorldHelper(RenderFrameHost* render_frame_host, |
const int world_id, |
const std::string& original_script, |
- scoped_ptr<base::Value>* result) |
+ std::unique_ptr<base::Value>* result) |
WARN_UNUSED_RESULT; |
bool ExecuteScriptInIsolatedWorldHelper(RenderFrameHost* render_frame_host, |
const int world_id, |
const std::string& original_script, |
- scoped_ptr<base::Value>* result) { |
+ std::unique_ptr<base::Value>* result) { |
std::string script = |
"window.domAutomationController.setAutomationId(0);" + original_script; |
DOMOperationObserver dom_op_observer(render_frame_host); |
@@ -285,13 +285,13 @@ void SetCookieOnIOThread(const GURL& url, |
base::Bind(&SetCookieCallback, result, event)); |
} |
-scoped_ptr<net::test_server::HttpResponse> CrossSiteRedirectResponseHandler( |
- const GURL& server_base_url, |
- const net::test_server::HttpRequest& request) { |
+std::unique_ptr<net::test_server::HttpResponse> |
+CrossSiteRedirectResponseHandler(const GURL& server_base_url, |
+ const net::test_server::HttpRequest& request) { |
std::string prefix("/cross-site/"); |
if (!base::StartsWith(request.relative_url, prefix, |
base::CompareCase::SENSITIVE)) |
- return scoped_ptr<net::test_server::HttpResponse>(); |
+ return std::unique_ptr<net::test_server::HttpResponse>(); |
std::string params = request.relative_url.substr(prefix.length()); |
@@ -299,7 +299,7 @@ scoped_ptr<net::test_server::HttpResponse> CrossSiteRedirectResponseHandler( |
// one '/' character is expected. |
size_t slash = params.find('/'); |
if (slash == std::string::npos) |
- return scoped_ptr<net::test_server::HttpResponse>(); |
+ return std::unique_ptr<net::test_server::HttpResponse>(); |
// Replace the host of the URL with the one passed in the URL. |
GURL::Replacements replace_host; |
@@ -311,7 +311,7 @@ scoped_ptr<net::test_server::HttpResponse> CrossSiteRedirectResponseHandler( |
GURL redirect_target(redirect_server.Resolve(path)); |
DCHECK(redirect_target.is_valid()); |
- scoped_ptr<net::test_server::BasicHttpResponse> http_response( |
+ std::unique_ptr<net::test_server::BasicHttpResponse> http_response( |
new net::test_server::BasicHttpResponse); |
http_response->set_code(net::HTTP_MOVED_PERMANENTLY); |
http_response->AddCustomHeader("Location", redirect_target.spec()); |
@@ -663,7 +663,7 @@ bool ExecuteScript(const ToRenderFrameHost& adapter, |
bool ExecuteScriptAndExtractInt(const ToRenderFrameHost& adapter, |
const std::string& script, int* result) { |
DCHECK(result); |
- scoped_ptr<base::Value> value; |
+ std::unique_ptr<base::Value> value; |
if (!ExecuteScriptHelper(adapter.render_frame_host(), script, &value) || |
!value.get()) { |
return false; |
@@ -675,7 +675,7 @@ bool ExecuteScriptAndExtractInt(const ToRenderFrameHost& adapter, |
bool ExecuteScriptAndExtractBool(const ToRenderFrameHost& adapter, |
const std::string& script, bool* result) { |
DCHECK(result); |
- scoped_ptr<base::Value> value; |
+ std::unique_ptr<base::Value> value; |
if (!ExecuteScriptHelper(adapter.render_frame_host(), script, &value) || |
!value.get()) { |
return false; |
@@ -690,7 +690,7 @@ bool ExecuteScriptInIsolatedWorldAndExtractBool( |
const std::string& script, |
bool* result) { |
DCHECK(result); |
- scoped_ptr<base::Value> value; |
+ std::unique_ptr<base::Value> value; |
if (!ExecuteScriptInIsolatedWorldHelper(adapter.render_frame_host(), world_id, |
script, &value) || |
!value.get()) { |
@@ -704,7 +704,7 @@ bool ExecuteScriptAndExtractString(const ToRenderFrameHost& adapter, |
const std::string& script, |
std::string* result) { |
DCHECK(result); |
- scoped_ptr<base::Value> value; |
+ std::unique_ptr<base::Value> value; |
if (!ExecuteScriptHelper(adapter.render_frame_host(), script, &value) || |
!value.get()) { |
return false; |
@@ -1113,7 +1113,7 @@ bool FrameWatcher::OnMessageReceived(const IPC::Message& message) { |
ViewHostMsg_SwapCompositorFrame::Param param; |
if (!ViewHostMsg_SwapCompositorFrame::Read(&message, ¶m)) |
return false; |
- scoped_ptr<cc::CompositorFrame> frame(new cc::CompositorFrame); |
+ std::unique_ptr<cc::CompositorFrame> frame(new cc::CompositorFrame); |
base::get<1>(param).AssignTo(frame.get()); |
BrowserThread::PostTask( |