Move the responsibility to convert from JSON to FileResource into drive_api_operations.

chrome/browser/google_apis/drive_api_service.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/google_apis/drive_api_service.h"
 
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 70 matching lines @@
 
   PostTaskAndReplyWithResult(
       BrowserThread::GetBlockingPool(),
       FROM_HERE,
       base::Bind(&ParseResourceListOnBlockingPool,
                  base::Passed(&value), error),
       base::Bind(&DidParseResourceListOnBlockingPool,
                  callback, base::Owned(error)));
 }
 
-// Parses the JSON value to ResourceEntry runs |callback|.
+// Parses the FileResource value to ResourceEntry and runs |callback|.
 void ParseResourceEntryAndRun(
     const GetResourceEntryCallback& callback,
     GDataErrorCode error,
-    scoped_ptr<base::Value> value) {
+    scoped_ptr<FileResource> value) {
   DCHECK(!BrowserThread::CurrentlyOn(BrowserThread::UI));
 
   if (!value) {
     callback.Run(error, scoped_ptr<ResourceEntry>());
     return;
   }
 
-  // Parsing FileResource is cheap enough to do on UI thread.
-  scoped_ptr<FileResource> file_resource = FileResource::CreateFrom(*value);
-  if (!file_resource) {
-    callback.Run(GDATA_PARSE_ERROR, scoped_ptr<ResourceEntry>());
-    return;
-  }
-
   // Converting to ResourceEntry is cheap enough to do on UI thread.
   scoped_ptr<ResourceEntry> entry =
-      ResourceEntry::CreateFromFileResource(*file_resource);
+      ResourceEntry::CreateFromFileResource(*value);
   if (!entry) {
     callback.Run(GDATA_PARSE_ERROR, scoped_ptr<ResourceEntry>());
     return;
   }
 
   callback.Run(error, entry.Pass());
 }
 
 // Parses the AboutResource value to AccountMetadataFeed and runs |callback|
 // on the UI thread once parsing is done.
@@ skipping 441 matching lines @@
 }
 
 void DriveAPIService::OnProgressUpdate(
     const OperationProgressStatusList& list) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   FOR_EACH_OBSERVER(
       DriveServiceObserver, observers_, OnProgressUpdate(list));
 }
 
 }  // namespace google_apis