Added ScopedNSAutoreleasePool to RenderViewTest.

content/public/test/render_view_test.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_TEST_RENDER_VIEW_TEST_H_
 #define CONTENT_PUBLIC_TEST_RENDER_VIEW_TEST_H_
 
 #include <string>
 
 #include "base/command_line.h"
@@ skipping 121 matching lines @@
   RenderView* view_;
   RendererWebKitPlatformSupportImplNoSandbox webkit_platform_support_;
   ContentRendererClient content_renderer_client_;
   scoped_ptr<MockRenderThread> render_thread_;
 
   // Used to setup the process so renderers can run.
   scoped_ptr<RendererMainPlatformDelegate> platform_;
   scoped_ptr<MainFunctionParams> params_;
   scoped_ptr<CommandLine> command_line_;
 
+#if defined(OS_MACOSX)
+  scoped_ptr<base::mac::ScopedNSAutoreleasePool> autorelease_pool_;

[Nico, 2014/02/04 16:12:51]

Does this have to be a scoped_ptr? A regular member variable might be good
enough.

[tasak, 2014/02/05 04:23:02]

I would like to make ScopedNSAutoreleasePool's lifetime the same as test's
lifetime, i.e. from SetUp to TearDown.

Looking at ScopedNSAutoreleasePool's behavior, if we use the variable as a
regular member, it will start working at  RenderViewTest's constructor time.

If we create some MacOSX thing after creating RenderViewTest before invoking its
SetUp, we might have some trouble?

+#endif
+
  private:
   void GoToOffset(int offset, const blink::WebHistoryItem& history_item);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_TEST_RENDER_VIEW_TEST_H_