[NaCl SDK] nacl_io: Always create directory node in http filesystem root.

native_client_sdk/src/tests/nacl_io_test/http_fs_test.cc

 // 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 <fcntl.h>
 #include <gmock/gmock.h>
 #include <ppapi/c/ppb_file_io.h>
 #include <ppapi/c/pp_errors.h>
 #include <ppapi/c/pp_instance.h>
 #include <sys/stat.h>
@@ skipping 160 matching lines @@
 
 TEST_P(HttpFsTest, GetStat) {
   const char contents[] = "contents";
   ASSERT_TRUE(ppapi_.server_template()->AddEntity("file", contents, NULL));
 
   ScopedNode node;
   ASSERT_EQ(0, fs_.Open(Path("/file"), O_RDONLY, &node));
 
   struct stat statbuf;
   EXPECT_EQ(0, node->GetStat(&statbuf));
-  EXPECT_EQ(S_IFREG | S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH | S_IWOTH,
-            statbuf.st_mode);
+  EXPECT_EQ(S_IFREG | S_IRUSR | S_IRGRP | S_IROTH, statbuf.st_mode);
   EXPECT_EQ(strlen(contents), statbuf.st_size);
   // These are not currently set.
   EXPECT_EQ(0, statbuf.st_atime);
   EXPECT_EQ(0, statbuf.st_ctime);
   EXPECT_EQ(0, statbuf.st_mtime);
 }
 
 TEST_P(HttpFsTest, FTruncate) {
   const char contents[] = "contents";
   ASSERT_TRUE(ppapi_.server_template()->AddEntity("file", contents, NULL));
@@ skipping 53 matching lines @@
   // TODO(binji): If the server doesn't send the content length, this operation
   // will be incredibly slow; it will attempt to read all of the data from the
   // server to find the file length. Can we do anything smarter?
   ppapi_.server_template()->set_send_content_length(true);
 
   ScopedNode node;
   ASSERT_EQ(0, fs_.Open(Path("/file"), O_RDONLY, &node));
 
   struct stat statbuf;
   EXPECT_EQ(0, node->GetStat(&statbuf));
-  EXPECT_EQ(S_IFREG | S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH | S_IWOTH,
-            statbuf.st_mode);
+  EXPECT_EQ(S_IFREG | S_IRUSR | S_IRGRP | S_IROTH, statbuf.st_mode);
   EXPECT_EQ(size, statbuf.st_size);
   // These are not currently set.
   EXPECT_EQ(0, statbuf.st_atime);
   EXPECT_EQ(0, statbuf.st_ctime);
   EXPECT_EQ(0, statbuf.st_mtime);
 }
 
 // Instantiate the large file tests, only when cache content is off.
 // TODO(binji): make cache content smarter, so it doesn't try to cache enormous
 // files. See http://crbug.com/369279.
 INSTANTIATE_TEST_CASE_P(Default,
                         HttpFsLargeFileTest,
                         ::testing::Values((uint32_t)kStringMapParamCacheNone,
                                           (uint32_t)kStringMapParamCacheStat));
 
+TEST(HttpFsDirTest, Root) {
+  StringMap_t args;
+  HttpFsForTesting fs(args, NULL);
+
+  // Check root node is directory
+  ScopedNode node;
+  ASSERT_EQ(0, fs.Open(Path("/"), O_RDONLY, &node));
+  ASSERT_TRUE(node->IsaDir());
+
+  // We have to r+w access to the root node
+  ASSERT_EQ(0, fs.Access(Path("/"), R_OK));
+  ASSERT_EQ(0, fs.Access(Path("/"), X_OK));
+  ASSERT_EQ(EACCES, fs.Access(Path("/"), W_OK));
+}
+
 TEST(HttpFsDirTest, Mkdir) {
   StringMap_t args;
   HttpFsForTesting fs(args, NULL);
   char manifest[] = "-r-- 123 /mydir/foo\n-rw- 234 /thatdir/bar\n";
   ASSERT_EQ(0, fs.ParseManifest(manifest));
   // mkdir of existing directories should give "File exists".
   EXPECT_EQ(EEXIST, fs.Mkdir(Path("/"), 0));
   EXPECT_EQ(EEXIST, fs.Mkdir(Path("/mydir"), 0));
   // mkdir of non-existent directories should give "Permission denied".
   EXPECT_EQ(EACCES, fs.Mkdir(Path("/non_existent"), 0));
@@ skipping 79 matching lines @@
   EXPECT_FALSE(bar->GetStat(&sbar));
 
   EXPECT_EQ(123, sfoo.st_size);
   EXPECT_EQ(S_IFREG | S_IRALL, sfoo.st_mode);
 
   EXPECT_EQ(234, sbar.st_size);
   EXPECT_EQ(S_IFREG | S_IRALL | S_IWALL, sbar.st_mode);
 }
 
 TEST(HttpFsBlobUrlTest, Basic) {
-  const char* kUrl =
-      "blob:http%3A//example.com/6b87a5a6-713e-46a4-9f0c-78066406455d";
+  const char* kUrl = "blob:http%3A//example.com/6b87a5a6-713e";
+  const char* kContent = "hello";
   FakePepperInterfaceURLLoader ppapi;
-  ASSERT_TRUE(ppapi.server_template()->SetBlobEntity(kUrl, "", NULL));
+  ASSERT_TRUE(ppapi.server_template()->SetBlobEntity(kUrl, kContent, NULL));
 
   StringMap_t args;
   args["SOURCE"] = kUrl;
 
   HttpFsForTesting fs(args, &ppapi);
 
-  // We have to read from the mount root to read a Blob URL.
+  // Check access to root folder
   ASSERT_EQ(0, fs.Access(Path("/"), R_OK));
   ASSERT_EQ(EACCES, fs.Access(Path("/"), W_OK));
   ASSERT_EQ(EACCES, fs.Access(Path("/"), X_OK));
 
   // Any other path will fail.
   ScopedNode foo;
-  ASSERT_EQ(ENOENT, fs.Access(Path(""), R_OK));
-  ASSERT_EQ(ENOENT, fs.Access(Path("."), R_OK));
-  ASSERT_EQ(ENOENT, fs.Access(Path("blah"), R_OK));
+  ASSERT_EQ(ENOENT, fs.Access(Path("/blah"), R_OK));
+
+  // Verify file size
+  ScopedNode node;
+  struct stat statbuf;
+  ASSERT_EQ(0, fs.Open(Path("/"), O_RDONLY, &node));
+  ASSERT_EQ(0, node->GetStat(&statbuf));
+  ASSERT_EQ(0, node->GetStat(&statbuf));
+  ASSERT_EQ(strlen(kContent), statbuf.st_size);
 }