File manager drive API: handle case where no default task is associated.

chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc

Index: chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc
diff --git a/chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc b/chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc
index d0d410055e252e1a477a7e9c4782deaf62411908..8a62bb996bb087265b91a3af4119b52a850ccd50 100644
--- a/chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc
+++ b/chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc
@@ -142,10 +142,7 @@ void GetDriveEntryPropertiesFunction::OnGetFileInfo(
         file_specific_info.content_mime_type(),
         file_path_.Extension());
     file_tasks::TaskDescriptor default_task;
-    if (!file_tasks::ParseTaskID(default_task_id, &default_task)) {
-      LOG(WARNING) << "Invalid task ID: " << default_task_id;
-      return;
-    }
+    file_tasks::ParseTaskID(default_task_id, &default_task);

[tbarzic, 2013/08/12 21:09:49]

afaik, ParseTaskID should never fail for non-empty default_task_id, so it may be
useful to add something along the line of:

DCHECK(default_task_id.empty() || !default_task.app_id.empty());

[hshi1, 2013/08/12 21:13:41]

Done.

 
     ListValue* apps = new ListValue();
     properties_->Set("driveApps", apps);