Be explicit about target type in platform_util::OpenItem()

chrome/browser/platform_util_chromeos.cc

Index: chrome/browser/platform_util_chromeos.cc
diff --git a/chrome/browser/platform_util_chromeos.cc b/chrome/browser/platform_util_chromeos.cc
index aad0b73602f9396699b2ae1b17ebdf69b738a162..a2c78160fa5a4963476c0106fc59fe9d62fb2d6c 100644
--- a/chrome/browser/platform_util_chromeos.cc
+++ b/chrome/browser/platform_util_chromeos.cc
@@ -25,9 +25,14 @@ void ShowItemInFolder(Profile* profile, const base::FilePath& full_path) {
   file_manager::util::ShowItemInFolder(profile, full_path);
 }
 
-void OpenItem(Profile* profile, const base::FilePath& full_path) {
+void OpenFile(Profile* profile, const base::FilePath& full_path) {

[James Hawkins, 2014/09/29 21:09:14]

The documentation says |full_path| must poing to an existing file; is that
asserted in file_manager::util::OpenFile?

[asanka, 2014/09/30 23:07:46]

I updated the documentation to not require callers to ensure the existence of
|full_path|.

   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  file_manager::util::OpenItem(profile, full_path);
+  file_manager::util::OpenFile(profile, full_path);
+}
+
+void OpenFolder(Profile* profile, const base::FilePath& full_path) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  file_manager::util::OpenFolder(profile, full_path);
 }
 
 void OpenExternal(Profile* profile, const GURL& url) {