[Cleanup] Used scoped pointers in KeyedServiceFactory's SetTestingFactory functions.

chrome/browser/extensions/api/image_writer_private/operation_manager_unittest.cc

Index: chrome/browser/extensions/api/image_writer_private/operation_manager_unittest.cc
diff --git a/chrome/browser/extensions/api/image_writer_private/operation_manager_unittest.cc b/chrome/browser/extensions/api/image_writer_private/operation_manager_unittest.cc
index bec819037df5107eaf134e31b30ee4efdb18406e..da5fa74237238d1aa257741834cd52a805e4123c 100644
--- a/chrome/browser/extensions/api/image_writer_private/operation_manager_unittest.cc
+++ b/chrome/browser/extensions/api/image_writer_private/operation_manager_unittest.cc
@@ -3,6 +3,7 @@
 // found in the LICENSE file.
 
 #include "base/command_line.h"
+#include "base/memory/scoped_ptr.h"
 #include "chrome/browser/extensions/api/image_writer_private/error_messages.h"
 #include "chrome/browser/extensions/api/image_writer_private/operation_manager.h"
 #include "chrome/browser/extensions/api/image_writer_private/test_utils.h"
@@ -35,8 +36,9 @@ class FakeEventRouter : public extensions::EventRouter {
 };
 
 // FakeEventRouter factory function
-KeyedService* FakeEventRouterFactoryFunction(content::BrowserContext* context) {
-  return new FakeEventRouter(static_cast<Profile*>(context));
+scoped_ptr<KeyedService> FakeEventRouterFactoryFunction(
+    content::BrowserContext* context) {
+  return make_scoped_ptr(new FakeEventRouter(static_cast<Profile*>(context)));

[Devlin, 2015/06/08 19:29:43]

nit: I liked that you updated some of these to be Profile::FromBrowserContext()
(even though it doesn't really matter).  If you don't mind, it'd be good to
update this one to.  If it's a hassle, don't bother (clearly it was already like
this).

[Ilya Sherman, 2015/06/08 22:11:43]

I was thinking of making a separate pass to update these across the codebase. 
I'm going to punt on this for now.

 }
 
 namespace {