Add a unit test that filenames aren't unintentionally converted to URLs.

ui/base/dragdrop/os_exchange_data_unittest.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 "base/file_util.h"
 #include "base/message_loop/message_loop.h"
 #include "base/pickle.h"
 #include "base/strings/utf_string_conversions.h"
+#include "net/base/filename_util.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "testing/platform_test.h"
 #include "ui/base/dragdrop/os_exchange_data.h"
 #include "ui/events/platform/platform_event_source.h"
 #include "url/gurl.h"
 
 namespace ui {
 
 class OSExchangeDataTest : public PlatformTest {
  public:
@@ skipping 13 matching lines @@
   EXPECT_TRUE(data.HasString());
 
   OSExchangeData data2(data.provider().Clone());
   base::string16 output;
   EXPECT_TRUE(data2.HasString());
   EXPECT_TRUE(data2.GetString(&output));
   EXPECT_EQ(input, output);
   std::string url_spec = "http://www.goats.com/";
   GURL url(url_spec);
   base::string16 title;
-  EXPECT_FALSE(
-      data2.GetURLAndTitle(OSExchangeData::CONVERT_FILENAMES, &url, &title));
+  EXPECT_FALSE(data2.GetURLAndTitle(
+      OSExchangeData::DO_NOT_CONVERT_FILENAMES, &url, &title));
   // No URLs in |data|, so url should be untouched.
   EXPECT_EQ(url_spec, url.spec());
 }
 
 TEST_F(OSExchangeDataTest, TestURLExchangeFormats) {
   OSExchangeData data;
   std::string url_spec = "http://www.google.com/";
   GURL url(url_spec);
   base::string16 url_title = base::ASCIIToUTF16("www.google.com");
   EXPECT_FALSE(data.HasURL(OSExchangeData::DO_NOT_CONVERT_FILENAMES));
@@ skipping 27 matching lines @@
   base::string16 url_title = base::ASCIIToUTF16("www.google.com");
   data.SetURL(url, url_title);
 
   base::string16 output_string;
   EXPECT_TRUE(data.GetString(&output_string));
   EXPECT_EQ(string, output_string);
 
   GURL output_url;
   base::string16 output_title;
   EXPECT_TRUE(data.GetURLAndTitle(
-      OSExchangeData::CONVERT_FILENAMES, &output_url, &output_title));
+      OSExchangeData::DO_NOT_CONVERT_FILENAMES, &output_url, &output_title));
   EXPECT_EQ(url_spec, output_url.spec());
   EXPECT_EQ(url_title, output_title);
 }
 
+TEST_F(OSExchangeDataTest, TestFileToURLConversion) {
+  OSExchangeData data;
+  EXPECT_FALSE(data.HasURL(OSExchangeData::DO_NOT_CONVERT_FILENAMES));
+  EXPECT_FALSE(data.HasURL(OSExchangeData::CONVERT_FILENAMES));
+  EXPECT_FALSE(data.HasFile());
+
+  base::FilePath current_directory;
+  ASSERT_TRUE(base::GetCurrentDirectory(&current_directory));
+
+  data.SetFilename(current_directory);
+  {
+    EXPECT_FALSE(data.HasURL(OSExchangeData::DO_NOT_CONVERT_FILENAMES));
+    GURL actual_url;
+    base::string16 actual_title;
+    EXPECT_FALSE(data.GetURLAndTitle(
+        OSExchangeData::DO_NOT_CONVERT_FILENAMES, &actual_url, &actual_title));
+    EXPECT_EQ(GURL(), actual_url);
+    EXPECT_EQ(base::string16(), actual_title);
+  }
+  {
+// Filename to URL conversion is not implemented on ChromeOS.
+#if !defined(OS_CHROMEOS)
+    const bool expected_success = true;
+    const GURL expected_url(net::FilePathToFileURL(current_directory));
+#else
+    const bool expected_success = false;
+    const GURL expected_url;
+#endif
+    EXPECT_EQ(expected_success, data.HasURL(OSExchangeData::CONVERT_FILENAMES));
+    GURL actual_url;
+    base::string16 actual_title;
+    EXPECT_EQ(
+        expected_success,
+        data.GetURLAndTitle(
+            OSExchangeData::CONVERT_FILENAMES, &actual_url, &actual_title));
+    EXPECT_EQ(expected_url, actual_url);
+    EXPECT_EQ(base::string16(), actual_title);
+  }
+  EXPECT_TRUE(data.HasFile());
+  base::FilePath actual_path;
+  EXPECT_TRUE(data.GetFilename(&actual_path));
+  EXPECT_EQ(current_directory, actual_path);
+}
+
 TEST_F(OSExchangeDataTest, TestPickledData) {
   const OSExchangeData::CustomFormat kTestFormat =
       ui::Clipboard::GetFormatType("application/vnd.chromium.test");
 
   Pickle saved_pickle;
   saved_pickle.WriteInt(1);
   saved_pickle.WriteInt(2);
   OSExchangeData data;
   data.SetPickledData(kTestFormat, saved_pickle);
 
@@ skipping 21 matching lines @@
   data.SetHtml(html, url);
 
   OSExchangeData copy(data.provider().Clone());
   base::string16 read_html;
   EXPECT_TRUE(copy.GetHtml(&read_html, &url));
   EXPECT_EQ(html, read_html);
 }
 #endif
 
 }  // namespace ui