chromeos: Stop returning scoped_ptr from GDataCache::GetCacheEntry

chrome/browser/chromeos/gdata/gdata_cache_metadata.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 "chrome/browser/chromeos/gdata/gdata_cache_metadata.h"
 
 #include "base/file_util.h"
 #include "chrome/browser/chromeos/gdata/gdata_util.h"
 
 namespace gdata {
@@ skipping 183 matching lines @@
 void GDataCacheMetadataMap::RemoveCacheEntry(const std::string& resource_id) {
   AssertOnSequencedWorkerPool();
 
   CacheMap::iterator iter = cache_map_.find(resource_id);
   if (iter != cache_map_.end()) {
     // Delete the CacheEntry and remove it from the map.
     cache_map_.erase(iter);
   }
 }
 
-scoped_ptr<GDataCacheEntry> GDataCacheMetadataMap::GetCacheEntry(
-    const std::string& resource_id,
-    const std::string& md5) {
+bool GDataCacheMetadataMap::GetCacheEntry(const std::string& resource_id,
+                                          const std::string& md5,
+                                          GDataCacheEntry* entry) {
+  DCHECK(entry);
   AssertOnSequencedWorkerPool();
 
   CacheMap::iterator iter = cache_map_.find(resource_id);
   if (iter == cache_map_.end()) {
     DVLOG(1) << "Can't find " << resource_id << " in cache map";
-    return scoped_ptr<GDataCacheEntry>();
+    return false;
   }
 
-  scoped_ptr<GDataCacheEntry> cache_entry(
-      new GDataCacheEntry(iter->second));
+  const GDataCacheEntry& cache_entry = iter->second;
 
-  if (!CheckIfMd5Matches(md5, *cache_entry)) {
+  if (!CheckIfMd5Matches(md5, cache_entry)) {
     DVLOG(1) << "Non-matching md5: want=" << md5
              << ", found=[res_id=" << resource_id
-             << ", " << cache_entry->ToString()
+             << ", " << cache_entry.ToString()
              << "]";
-    return scoped_ptr<GDataCacheEntry>();
+    return false;
   }
 
   DVLOG(1) << "Found entry for res_id=" << resource_id
-           << ", " << cache_entry->ToString();
+           << ", " << cache_entry.ToString();
 
-  return cache_entry.Pass();
+  *entry = cache_entry;
+  return true;
 }
 
 void GDataCacheMetadataMap::RemoveTemporaryFiles() {
   AssertOnSequencedWorkerPool();
 
   CacheMap::iterator iter = cache_map_.begin();
   while (iter != cache_map_.end()) {
     if (!iter->second.IsPersistent()) {
       // Post-increment the iterator to avoid iterator invalidation.
       cache_map_.erase(iter++);
@@ skipping 144 matching lines @@
     // If the MD5 matching is not requested, don't check MD5.
     return true;
   } else if (md5 == cache_entry.md5()) {
     // Otherwise, compare the MD5.
     return true;
   }
   return false;
 }
 
 }  // namespace gdata