Part 2 of base::IDMap refactor to eliminate IDMapOwnPointer/IDMapExternalPointer modes

components/drive/job_scheduler.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 COMPONENTS_DRIVE_JOB_SCHEDULER_H_
 #define COMPONENTS_DRIVE_JOB_SCHEDULER_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 364 matching lines @@
   // Jobs should not start running until this time. Used for throttling.
   base::Time wait_until_;
 
   // Disables throttling for testing.
   bool disable_throttling_;
 
   // The queues of jobs.
   std::unique_ptr<JobQueue> queue_[NUM_QUEUES];
 
   // The list of queued job info indexed by job IDs.
-  typedef IDMap<JobEntry, IDMapOwnPointer> JobIDMap;
+  typedef IDMap<std::unique_ptr<JobEntry>> JobIDMap;

[danakj, 2016/11/30 23:02:21]

s/typedef/using/ while you're here?

   JobIDMap job_map_;
 
   // The list of observers for the scheduler.
   base::ObserverList<JobListObserver> observer_list_;
 
   EventLogger* logger_;
   DriveServiceInterface* drive_service_;
   base::SequencedTaskRunner* blocking_task_runner_;
   std::unique_ptr<DriveUploaderInterface> uploader_;
 
   PrefService* pref_service_;
 
   base::ThreadChecker thread_checker_;
 
   // Note: This should remain the last member so it'll be destroyed and
   // invalidate its weak pointers before any other members are destroyed.
   base::WeakPtrFactory<JobScheduler> weak_ptr_factory_;
   DISALLOW_COPY_AND_ASSIGN(JobScheduler);
 };
 
 }  // namespace drive
 
 #endif  // COMPONENTS_DRIVE_JOB_SCHEDULER_H_