Fix scoping of raw_bytes in URLRequestFileDirJob::OnListFile

net/url_request/url_request_unittest.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 "build/build_config.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #include <shlobj.h>
 #endif
@@ skipping 926 matching lines @@
 
     d.set_cancel_in_received_data_pending(true);
 
     base::RunLoop().Run();
   }
 
   // Take out mock resource provider.
   NetModule::SetResourceProvider(NULL);
 }
 
+TEST_F(URLRequestTest, FileDirOutputSanity) {
+  // Verify the general sanity of the the output of the file:
+  // directory lister by checking for the output of a known existing
+  // file.
+  const char sentinel_name[] = "filedir-sentinel";
+
+  base::FilePath path;
+  PathService::Get(base::DIR_SOURCE_ROOT, &path);
+  path = path.Append(FILE_PATH_LITERAL("net"));
+  path = path.Append(FILE_PATH_LITERAL("data"));
+  path = path.Append(FILE_PATH_LITERAL("url_request_unittest"));
+
+  TestDelegate d;
+  URLRequest req(FilePathToFileURL(path), &d, &default_context_);
+  req.Start();
+  base::RunLoop().Run();
+
+  // Generate entry for the sentinel file.
+  base::FilePath sentinel_path = path.AppendASCII(sentinel_name);
+  base::PlatformFileInfo info;
+  EXPECT_TRUE(file_util::GetFileInfo(sentinel_path, &info));
+  EXPECT_GT(info.size, 0);
+  std::string sentinel_output = GetDirectoryListingEntry(
+      base::string16(sentinel_name, sentinel_name + strlen(sentinel_name)),
+      std::string(sentinel_name),
+      false /* is_dir */,
+      info.size,
+      info.last_modified);
+
+  ASSERT_LT(0, d.bytes_received());
+  ASSERT_FALSE(d.request_failed());
+  ASSERT_TRUE(req.status().is_success());
+  // Check for the entry generated for the "sentinel" file.
+  const std::string& data = d.data_received();
+  ASSERT_NE(data.find(sentinel_output), std::string::npos);
+}
+
 TEST_F(URLRequestTest, FileDirRedirectNoCrash) {
   // There is an implicit redirect when loading a file path that matches a
   // directory and does not end with a slash.  Ensure that following such
   // redirects does not crash.  See http://crbug.com/18686.
 
   base::FilePath path;
   PathService::Get(base::DIR_SOURCE_ROOT, &path);
   path = path.Append(FILE_PATH_LITERAL("net"));
   path = path.Append(FILE_PATH_LITERAL("data"));
   path = path.Append(FILE_PATH_LITERAL("url_request_unittest"));
@@ skipping 6009 matching lines @@
 
     EXPECT_FALSE(r.is_pending());
     EXPECT_EQ(1, d->response_started_count());
     EXPECT_FALSE(d->received_data_before_response());
     EXPECT_EQ(d->bytes_received(), static_cast<int>(file_size));
   }
 }
 #endif  // !defined(DISABLE_FTP_SUPPORT)
 
 }  // namespace net