Fix file handler executer setting wrong cache path permissions on drive.

chrome/browser/chromeos/gdata/gdata_util.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_util.h"
 
 #include <string>
 #include <vector>
 #include <utility>
 
@@ skipping 222 matching lines @@
     Profile* profile,
     const FilePath& gdata_path,
     std::vector<std::pair<FilePath, int> >* cache_paths ) {
   DCHECK(cache_paths);
 
   GDataFileSystem* file_system = GetGDataFileSystem(profile);
   if (!file_system)
     return;
 
   GDataFileProperties file_properties;
-  file_system->GetFileInfoByPath(gdata_path, &file_properties);
+  if (!file_system->GetFileInfoByPath(gdata_path, &file_properties))
+   return;

[hshi1, 2012/06/08 01:24:08]

Nit: need 2-spaces indentation before "return;"

 
   std::string resource_id = file_properties.resource_id;
   std::string file_md5 = file_properties.file_md5;
 
   // We check permissions for raw cache file paths only for read-only
   // operations (when fileEntry.file() is called), so read only permissions
   // should be sufficient for all cache paths. For the rest of supported
   // operations the file access check is done for drive/ paths.
   cache_paths->push_back(std::make_pair(
       file_system->GetCacheFilePath(resource_id, file_md5,
@@ skipping 96 matching lines @@
   // Assign the extracted extensions to md5 and extra_extension.
   int extension_count = extensions.size();
   *md5 = (extension_count > 0) ? extensions[extension_count - 1] :
                                  std::string();
   *extra_extension = (extension_count > 1) ? extensions[extension_count - 2] :
                                              std::string();
 }
 
 }  // namespace util
 }  // namespace gdata