Refactor ViewsDelegate singleton

ui/views/controls/webview/webview_unittest.cc

Index: ui/views/controls/webview/webview_unittest.cc
diff --git a/ui/views/controls/webview/webview_unittest.cc b/ui/views/controls/webview/webview_unittest.cc
index 349ad3f732d4a25c87e3c1a99b4aa3e6dd02b4a7..eb7a05f0acfc6f8498c16bfd5a682463963eaada 100644
--- a/ui/views/controls/webview/webview_unittest.cc
+++ b/ui/views/controls/webview/webview_unittest.cc
@@ -130,7 +130,9 @@ class WebViewUnitTest : public views::test::WidgetTest {
 
   void SetUp() override {
     // The ViewsDelegate is deleted when the ViewsTestBase class is torn down.
-    WidgetTest::set_views_delegate(new WebViewTestViewsDelegate);
+    scoped_ptr<views::TestViewsDelegate> views_delegate(
+        new WebViewTestViewsDelegate);
+    set_views_delegate(views_delegate.Pass());

[sadrul, 2015/06/08 03:27:43]

You can also do:
  set_views_delegate(make_scoped_ptr(new WebViewTestViewsDelegate));

(this applies to the rest of the set_views_delegate() calls too).

[mohsen, 2015/06/08 19:27:34]

Done.

     browser_context_.reset(new content::TestBrowserContext);
     WidgetTest::SetUp();
     // Set the test content browser client to avoid pulling in needless
@@ -176,7 +178,6 @@ class WebViewUnitTest : public views::test::WidgetTest {
   content::TestBrowserThread file_blocking_thread_;
   content::TestBrowserThread io_thread_;
   scoped_ptr<content::TestBrowserContext> browser_context_;
-  scoped_ptr<WebViewTestViewsDelegate> views_delegate_;
   content::TestContentBrowserClient test_browser_client_;
 
   Widget* top_level_widget_;