Factory should use unique_ptr

Today, the `Factory` class returns `shared_ptr` on `Allocate(...)` calls - it should instead return `unique_ptr` as we do not continue to own the lifetime of the pointer, nor do we maintain a reference to it.