Added proper extensions for hosted documents.

chrome/browser/chromeos/gdata/gdata_parser.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_parser.h"
 
 #include "base/basictypes.h"
 #include "base/json/json_value_converter.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/string_number_conversions.h"
@@ skipping 18 matching lines @@
 const char kPdfTerm[] = "pdf";
 const char kDocumentTerm[] = "document";
 const char kSpreadSheetTerm[] = "spreadsheet";
 const char kPresentationTerm[] = "presentation";
 
 const char kSchemeLabels[] = "http://schemas.google.com/g/2005/labels";
 
 struct EntryKindMap {
   DocumentEntry::EntryKind kind;
   const char* entry;
+  const char* extension;
 };
 
 const EntryKindMap kEntryKindMap[] = {
-    { DocumentEntry::ITEM,         "item" },
-    { DocumentEntry::DOCUMENT,     "document"},
-    { DocumentEntry::SPREADSHEET,  "spreadsheet" },
-    { DocumentEntry::PRESENTATION, "presentation" },
-    { DocumentEntry::DRAWING,      "drawing"},
-    { DocumentEntry::TABLE,        "table"},
+    { DocumentEntry::ITEM,         "item",         NULL },
+    { DocumentEntry::DOCUMENT,     "document",     ".gdoc"},
+    { DocumentEntry::SPREADSHEET,  "spreadsheet",  ".gsheet"},
+    { DocumentEntry::PRESENTATION, "presentation", ".gslides" },
+    { DocumentEntry::DRAWING,      "drawing",      ".gdraw"},
+    { DocumentEntry::TABLE,        "table",        ".gtable"},
     { DocumentEntry::SITE,         "site"},
     { DocumentEntry::FOLDER,       "folder"},
     { DocumentEntry::FILE,         "file"},
     { DocumentEntry::PDF,          "pdf"},

[satorux1, 2012/03/02 22:13:01]

Add NULL to the last four types?

[zel, 2012/03/02 22:54:51]

Done already, forgot to upload it originally.

 };
 
 struct LinkTypeMap {
   Link::LinkType type;
   const char* rel;
 };
 
 const LinkTypeMap kLinkTypeMap[] = {
     { Link::SELF,
       "self" },
@@ skipping 277 matching lines @@
   // that's no problem because those feed must not have these fields
   // themselves, which does not report errors.
   converter->RegisterStringField(kFileNameField, &DocumentEntry::filename_);
   converter->RegisterStringField(kMD5Field, &DocumentEntry::file_md5_);
   converter->RegisterCustomField<int64>(
       kSizeField, &DocumentEntry::file_size_, &base::StringToInt64);
   converter->RegisterStringField(
       kSuggestedFileNameField, &DocumentEntry::suggested_filename_);
 }
 
-std::string DocumentEntry::GetEntryKindText() const {
-  return std::string(GetEntryKindDescription(kind_));
+std::string DocumentEntry::GetHostedDocumentExtension() const {
+  for (size_t i = 0; i < arraysize(kEntryKindMap); i++) {
+    if (kEntryKindMap[i].kind == kind_) {
+      if (kEntryKindMap[i].extension)
+        return std::string(kEntryKindMap[i].extension);
+      else
+        return std::string();
+    }
+  }
+  return std::string();
 }
 
 // static
-const char* DocumentEntry::GetEntryKindDescription(
-    DocumentEntry::EntryKind kind) {
-  for (size_t i = 0; i < arraysize(kEntryKindMap); i++) {
-    if (kEntryKindMap[i].kind == kind)
-      return kEntryKindMap[i].entry;
-  }
-  return "";
-}
-
-// static
 DocumentEntry::EntryKind DocumentEntry::GetEntryKindFromTerm(
     const std::string& term) {
   if (!StartsWithASCII(term, kTermPrefix, false)) {
     DVLOG(1) << "Unexpected term prefix term " << term;
     return DocumentEntry::UNKNOWN;
   }
 
   std::string type = term.substr(strlen(kTermPrefix));
   for (size_t i = 0; i < arraysize(kEntryKindMap); i++) {
     if (type == kEntryKindMap[i].entry)
@@ skipping 88 matching lines @@
   for (size_t i = 0; i < links_.size(); ++i) {
     if (links_[i]->type() == Link::NEXT) {
       *url = links_[i]->href();
       return true;
     }
   }
   return false;
 }
 
 }  // namespace gdata