Bug 1097577 - CreateFakeTracedTask should return a already_AddRefed. r=thinker

tools/profiler/GeckoTaskTracer.h

@@ -76,7 +76,7 @@ Task* CreateTracedTask(Task* aTask);
 
 already_AddRefed<nsIRunnable> CreateTracedRunnable(nsIRunnable* aRunnable);
 
-FakeTracedTask* CreateFakeTracedTask(int* aVptr);
+already_AddRefed<FakeTracedTask> CreateFakeTracedTask(int* aVptr);
 
 // Free the TraceInfo allocated on a thread's TLS. Currently we are wrapping
 // tasks running on nsThreads and base::thread, so FreeTraceInfo is called at

tools/profiler/TracedTaskCommon.cpp

@@ -162,10 +162,10 @@ CreateTracedTask(Task* aTask)
  * CreateFakeTracedTask() returns a FakeTracedTask tracking the event which is
  * not dispatched from its parent task directly, such as timer events.
  */
-FakeTracedTask*
+already_AddRefed<FakeTracedTask>
 CreateFakeTracedTask(int* aVptr)
 {
-  nsAutoPtr<FakeTracedTask> task(new FakeTracedTask(aVptr));
+  nsRefPtr<FakeTracedTask> task(new FakeTracedTask(aVptr));
   return task.forget();
 }
 

tools/profiler/TracedTaskCommon.h

@@ -81,6 +81,8 @@ public:
   void BeginFakeTracedTask();
   void EndFakeTracedTask();
 private:
+  virtual ~FakeTracedTask() {}
+
   // No copy allowed.
   FakeTracedTask() MOZ_DELETE;
   FakeTracedTask(const FakeTracedTask& aTask) MOZ_DELETE;