Mandate unique_ptr for base::IDMap in IDMapOwnPointer mode.

components/drive/job_scheduler.cc

 // 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.
 
 #include "components/drive/job_scheduler.h"
 
 #include <stddef.h>
 #include <algorithm>
 #include <utility>
 
@@ skipping 703 matching lines @@
                              role,
                              base::Bind(&JobScheduler::OnEntryActionJobDone,
                                         weak_ptr_factory_.GetWeakPtr(),
                                         new_job->job_info.job_id,
                                         callback));
   new_job->abort_callback = callback;
   StartJob(new_job);
 }
 
 JobScheduler::JobEntry* JobScheduler::CreateNewJob(JobType type) {
-  JobEntry* job = new JobEntry(type);
-  job->job_info.job_id = job_map_.Add(job);  // Takes the ownership of |job|.
+  int32_t job_key = job_map_.Add(
+      base::MakeUnique<JobEntry>(type));
+  JobEntry* job = job_map_.Lookup(job_key);

[danakj, 2016/11/18 00:15:32]

You don't need to do Lookup, you can just get a pointer from the unique_ptr
before u hand it to Add()

+  job->job_info.job_id = job_key;
   return job;
 }
 
 void JobScheduler::StartJob(JobEntry* job) {
   DCHECK(!job->task.is_null());
 
   QueueJob(job->job_info.job_id);
   NotifyJobAdded(job->job_info);
   DoJobLoop(GetJobQueueType(job->job_info.job_type));
 }
@@ skipping 453 matching lines @@
     case FILE_QUEUE:
       return "FILE_QUEUE";
     case NUM_QUEUES:
       break;  // This value is just a sentinel. Should never be used.
   }
   NOTREACHED();
   return "";
 }
 
 }  // namespace drive