Unify HasGDocFileExtension and IsHostedDocumentByExtentoin.

chrome/browser/drive/drive_api_util_unittest.cc

Index: chrome/browser/drive/drive_api_util_unittest.cc
diff --git a/chrome/browser/drive/drive_api_util_unittest.cc b/chrome/browser/drive/drive_api_util_unittest.cc
index 7d32a4278a164491137ec2bedfb53540195fc4e5..201fba76ee4d69f2f1c4b5dd29b646ef3b594b68 100644
--- a/chrome/browser/drive/drive_api_util_unittest.cc
+++ b/chrome/browser/drive/drive_api_util_unittest.cc
@@ -164,5 +164,29 @@ TEST(DriveAPIUtilTest, GetMd5Digest) {
   EXPECT_EQ(base::MD5String(kTestData), GetMd5Digest(path));
 }
 
+TEST(DriveAPIUtilTest, HasHostedDocumentExtension) {
+  EXPECT_TRUE(
+      HasHostedDocumentExtension(base::FilePath::FromUTF8Unsafe("xx.gdoc")));
+  EXPECT_TRUE(
+      HasHostedDocumentExtension(base::FilePath::FromUTF8Unsafe("xx.gsheet")));
+  EXPECT_TRUE(
+      HasHostedDocumentExtension(base::FilePath::FromUTF8Unsafe("xx.gslides")));
+  EXPECT_TRUE(
+      HasHostedDocumentExtension(base::FilePath::FromUTF8Unsafe("xx.gdraw")));
+  EXPECT_TRUE(
+      HasHostedDocumentExtension(base::FilePath::FromUTF8Unsafe("xx.gtable")));
+  EXPECT_TRUE(
+      HasHostedDocumentExtension(base::FilePath::FromUTF8Unsafe("xx.gform")));
+  EXPECT_TRUE(
+      HasHostedDocumentExtension(base::FilePath::FromUTF8Unsafe("xx.glink")));

[hashimoto, 2014/07/25 06:19:59]

Seems this results in returning false?

[kinaba, 2014/07/25 06:24:38]

Removed the check. It will come back in the next CL.

+
+  EXPECT_FALSE(
+      HasHostedDocumentExtension(base::FilePath::FromUTF8Unsafe("xx.gdocs")));
+  EXPECT_FALSE(
+      HasHostedDocumentExtension(base::FilePath::FromUTF8Unsafe("xx.docx")));
+  EXPECT_FALSE(
+      HasHostedDocumentExtension(base::FilePath::FromUTF8Unsafe("xx.jpg")));
+}
+
 }  // namespace util
 }  // namespace drive