gdata: Use member functions of CacheEntry where it can

chrome/browser/chromeos/gdata/gdata_cache.h

Index: chrome/browser/chromeos/gdata/gdata_cache.h
diff --git a/chrome/browser/chromeos/gdata/gdata_cache.h b/chrome/browser/chromeos/gdata/gdata_cache.h
index de99af4fff7d7ef8fc28342c1d2c4da2dee1e1f6..92ffddf277daa7ecf4596bb3411a07773cee8647 100644
--- a/chrome/browser/chromeos/gdata/gdata_cache.h
+++ b/chrome/browser/chromeos/gdata/gdata_cache.h
@@ -121,7 +121,7 @@ class GDataCache {
 
   // Structure to store information of an existing cache file.
   struct CacheEntry {
-    CacheEntry() : cache_state(0) {}
+    CacheEntry() : cache_state(CACHE_STATE_NONE) {}
 
     CacheEntry(const std::string& md5,
                int cache_state)
@@ -149,7 +149,8 @@ class GDataCache {
 
   // Callback for GetCacheEntryOnUIThread.
   // |success| indicates if the operation was successful.
-  // |cache_entry| is the obtained cache entry.
+  // |cache_entry| is the obtained cache entry. On failure, |cache_state| is
+  // |set to CACHE_STATE_NONE.

[hshi1, 2012/07/10 23:54:49]

Nit: remove extra character '|' before "set".

[satorux1, 2012/07/11 05:24:58]

Done.

   //
   // TODO(satorux): Unlike other callback types, this has to be defined
   // inside GDataCache as CacheEntry is inside GDataCache. We should get them