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

Unified Diff: content/browser/renderer_host/pepper/pepper_file_system_browser_host_unittest.cc

Issue 26803004: PPAPI: Add PluginPrivateFileSystem (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 1 month 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
« no previous file with comments | « content/browser/renderer_host/pepper/pepper_file_system_browser_host.cc ('k') | content/content_tests.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/renderer_host/pepper/pepper_file_system_browser_host_unittest.cc
diff --git a/content/browser/renderer_host/pepper/pepper_file_system_browser_host_unittest.cc b/content/browser/renderer_host/pepper/pepper_file_system_browser_host_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..fa07e3938ab19e4c21c7960934ee6b5cd1b707e8
--- /dev/null
+++ b/content/browser/renderer_host/pepper/pepper_file_system_browser_host_unittest.cc
@@ -0,0 +1,80 @@
+// Copyright 2013 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 "content/browser/renderer_host/pepper/pepper_file_system_browser_host.h"
+
+#include <string>
+
+#include "base/basictypes.h"
+#include "base/memory/scoped_ptr.h"
+#include "content/browser/renderer_host/pepper/browser_ppapi_host_test.h"
+#include "ppapi/c/pp_instance.h"
+#include "ppapi/c/pp_resource.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace content {
+
+class PepperFileSystemBrowserHostTest
+ : public testing::Test,
+ public BrowserPpapiHostTest {
+ public:
+ PepperFileSystemBrowserHostTest() {}
+ virtual ~PepperFileSystemBrowserHostTest() {}
+
+ virtual void SetUp() OVERRIDE {
+ PP_Instance pp_instance = 12345;
+ PP_Resource pp_resource = 67890;
+ host_.reset(new PepperFileSystemBrowserHost(
+ GetBrowserPpapiHost(), pp_instance, pp_resource,
+ PP_FILESYSTEMTYPE_ISOLATED));
+ }
+
+ virtual void TearDown() OVERRIDE {
+ host_.reset();
+ }
+
+ protected:
+ std::string GeneratePluginId(const std::string& mime_type) {
+ return host_->GeneratePluginId(mime_type);
+ }
+
+ private:
+ scoped_ptr<PepperFileSystemBrowserHost> host_;
+
+ DISALLOW_COPY_AND_ASSIGN(PepperFileSystemBrowserHostTest);
+};
+
+TEST_F(PepperFileSystemBrowserHostTest, GeneratePluginId) {
+ // Should not contain wild card characters.
+ EXPECT_TRUE(GeneratePluginId("*").empty());
+ EXPECT_TRUE(GeneratePluginId("*/*").empty());
+
+ // Should contain only one slash.
+ EXPECT_TRUE(GeneratePluginId(".").empty());
+ EXPECT_TRUE(GeneratePluginId("..").empty());
+ EXPECT_TRUE(GeneratePluginId("application").empty());
+ EXPECT_TRUE(GeneratePluginId("application/mime/type").empty());
+
+ // Should start with "legal_top_level_types"
+ // (ex. "application", "audio", "x-"). See "mime_util.cc" for more details.
+ EXPECT_TRUE(GeneratePluginId("/mime").empty());
+ EXPECT_TRUE(GeneratePluginId("./mime").empty());
+ EXPECT_TRUE(GeneratePluginId("../mime").empty());
+ EXPECT_TRUE(GeneratePluginId("app/mime").empty());
+
+ // Should not contain characters other than alphanumeric or "._-".
+ EXPECT_TRUE(GeneratePluginId("application/mime+type").empty());
+ EXPECT_TRUE(GeneratePluginId("application/mime:type").empty());
+ EXPECT_TRUE(GeneratePluginId("application/mime;type").empty());
+
+ // Valid cases.
+ EXPECT_EQ("application_mime", GeneratePluginId("application/mime"));
+ EXPECT_EQ("x-app_mime", GeneratePluginId("x-app/mime"));
+ EXPECT_EQ("application_mime.type", GeneratePluginId("application/mime.type"));
+ EXPECT_EQ("application_mime_type", GeneratePluginId("application/mime_type"));
+ EXPECT_EQ("application_mime-type", GeneratePluginId("application/mime-type"));
+ EXPECT_EQ("application_..", GeneratePluginId("application/.."));
+}
+
+} // namespace content
« no previous file with comments | « content/browser/renderer_host/pepper/pepper_file_system_browser_host.cc ('k') | content/content_tests.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698