Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1593)

Unified Diff: chrome/browser/chromeos/arc/fileapi/arc_content_file_system_url_util_unittest.cc

Issue 2437693002: arc: Add URL converter functions (Closed)
Patch Set: Add test Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/arc/fileapi/arc_content_file_system_url_util_unittest.cc
diff --git a/chrome/browser/chromeos/arc/fileapi/arc_content_file_system_url_util_unittest.cc b/chrome/browser/chromeos/arc/fileapi/arc_content_file_system_url_util_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..a8b4de9ca9863569ae474644aec2ccb488e4935e
--- /dev/null
+++ b/chrome/browser/chromeos/arc/fileapi/arc_content_file_system_url_util_unittest.cc
@@ -0,0 +1,65 @@
+// Copyright 2016 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 "base/strings/utf_string_conversions.h"
+#include "chrome/browser/chromeos/arc/fileapi/arc_content_file_system_url_util.h"
+#include "content/public/common/url_constants.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "url/gurl.h"
+
+namespace arc {
+
+TEST(ArcContentFileSystemUrlUtilTest, EncodeAndDecode) {
+ {
+ GURL src("file://foo/bar/baz");
+ GURL dest = ArcUrlToExternalFileUrl(src);
+ EXPECT_TRUE(dest.is_valid());
+ EXPECT_EQ(content::kExternalFileScheme, dest.scheme());
+ GURL result = ExternalFileUrlToArcUrl(dest);
+ EXPECT_TRUE(result.is_valid());
+ EXPECT_EQ(src, result);
+ }
+ {
+ GURL src("content://org.chromium.foo/bar/baz");
+ GURL dest = ArcUrlToExternalFileUrl(src);
+ EXPECT_TRUE(dest.is_valid());
+ EXPECT_EQ(content::kExternalFileScheme, dest.scheme());
+ GURL result = ExternalFileUrlToArcUrl(dest);
+ EXPECT_TRUE(result.is_valid());
+ EXPECT_EQ(src, result);
+ }
+ {
+ GURL src("content://org.chromium.foo/bar/%19%20%21");
+ GURL dest = ArcUrlToExternalFileUrl(src);
+ EXPECT_TRUE(dest.is_valid());
+ EXPECT_EQ(content::kExternalFileScheme, dest.scheme());
+ GURL result = ExternalFileUrlToArcUrl(dest);
+ EXPECT_TRUE(result.is_valid());
+ EXPECT_EQ(src, result);
+ }
+ {
+ GURL src("content://org.chromium.foo/!@#$%^&*()_+|~-=\\`[]{};':\"<>?,./");
+ GURL dest = ArcUrlToExternalFileUrl(src);
+ EXPECT_TRUE(dest.is_valid());
+ EXPECT_EQ(content::kExternalFileScheme, dest.scheme());
+ GURL result = ExternalFileUrlToArcUrl(dest);
+ EXPECT_TRUE(result.is_valid());
+ EXPECT_EQ(src, result);
+ }
+ {
+ base::string16 utf16_string = {
+ 0x307b, // HIRAGANA_LETTER_HO
+ 0x3052, // HIRAGANA_LETTER_GE
+ };
+ GURL src("content://org.chromium.foo/" + base::UTF16ToUTF8(utf16_string));
+ GURL dest = ArcUrlToExternalFileUrl(src);
+ EXPECT_TRUE(dest.is_valid());
+ EXPECT_EQ(content::kExternalFileScheme, dest.scheme());
+ GURL result = ExternalFileUrlToArcUrl(dest);
+ EXPECT_TRUE(result.is_valid());
+ EXPECT_EQ(src, result);
+ }
+}
+
+} // namespace arc

Powered by Google App Engine
This is Rietveld 408576698