gdata: Remove use of FindEntryByPathAsync() from file_manager_util.cc

chrome/browser/chromeos/gdata/gdata_files.cc

Index: chrome/browser/chromeos/gdata/gdata_files.cc
diff --git a/chrome/browser/chromeos/gdata/gdata_files.cc b/chrome/browser/chromeos/gdata/gdata_files.cc
index 7774d328369fecdb213718f4493375ecf1d7f81e..3fdc4eadd912c8b6895d1c48f843957385bffa9b 100644
--- a/chrome/browser/chromeos/gdata/gdata_files.cc
+++ b/chrome/browser/chromeos/gdata/gdata_files.cc
@@ -643,6 +643,7 @@ void GDataEntry::FromProto(const GDataEntryProto& proto) {
   file_info_.creation_time = base::Time::FromInternalValue(
       proto.file_info().creation_time());
 
+  file_name_ = proto.file_name();

[achuithb, 2012/04/30 21:45:21]

It's fine to do this for consistency, but maybe add a comment saying that this
name will be changed by SetFileNameFromTitle()? Alternatively you can skip this
line altogether.

[satorux1, 2012/04/30 21:52:57]

Done.

   title_ = proto.title();
   resource_id_ = proto.resource_id();
   parent_resource_id_ = proto.parent_resource_id();
@@ -663,6 +664,7 @@ void GDataEntry::ToProto(GDataEntryProto* proto) const {
   proto_file_info->set_creation_time(
       file_info_.creation_time.ToInternalValue());
 
+  proto->set_file_name(file_name_);

[achuithb, 2012/04/30 21:45:21]

Maybe add a comment here saying that protos are also used by
GetFileInfoByPathAsync which needs this value?

[satorux1, 2012/04/30 21:52:57]

Done.

   proto->set_title(title_);
   proto->set_resource_id(resource_id_);
   proto->set_parent_resource_id(parent_resource_id_);