Formatting feature initial commit for ChromeOS Tree

chrome/browser/extensions/extension_file_browser_private_api.cc

Index: chrome/browser/extensions/extension_file_browser_private_api.cc
diff --git a/chrome/browser/extensions/extension_file_browser_private_api.cc b/chrome/browser/extensions/extension_file_browser_private_api.cc
index 7ef2d1b653618394cc901cbbb2af732c22ac72cd..99e6ab86cb604e683171e8dbf7f3385e7be473bd 100644
--- a/chrome/browser/extensions/extension_file_browser_private_api.cc
+++ b/chrome/browser/extensions/extension_file_browser_private_api.cc
@@ -1224,6 +1224,7 @@ bool RemoveMountFunction::RunImpl() {
   return true;
 }
 
+
 GetMountPointsFunction::GetMountPointsFunction() {
 }
 
@@ -1237,7 +1238,7 @@ bool GetMountPointsFunction::RunImpl() {
   base::ListValue *mounts = new base::ListValue();
   result_.reset(mounts);
 
-  #ifdef OS_CHROMEOS
+#ifdef OS_CHROMEOS
   chromeos::MountLibrary *mount_lib =
       chromeos::CrosLibrary::Get()->GetMountLibrary();
   chromeos::MountLibrary::MountPointMap mount_points =
@@ -1250,6 +1251,32 @@ bool GetMountPointsFunction::RunImpl() {
     mounts->Append(MountPointToValue(profile_, it->second));
   }
 #endif
+
+  SendResponse(true);
+  return true;
+}
+
+FormatDeviceFunction::FormatDeviceFunction() {
+}
+
+FormatDeviceFunction::~FormatDeviceFunction() {
+}
+
+bool FormatDeviceFunction::RunImpl() {
+  if (args_->GetSize() != 1) {
+    return false;
+  }
+
+  std::string volume_mount_path;
+  if (!args_->GetString(0, &volume_mount_path)) {

[asargent_no_longer_on_chrome, 2011/07/29 23:14:19]

nit: you probably want to either CHECK(false) here and crash the whole browser,
or return early from the function and not go on to pass an empty string to
FormatMountedDevice below (since NOTREACHED only does something in Debug builds
and is elided in Release builds).

[sidor.dev, 2011/07/29 23:28:27]

Done.

+    NOTREACHED();
+  }
+
+#ifdef OS_CHROMEOS
+  chromeos::CrosLibrary::Get()->GetMountLibrary()->FormatMountedDevice(
+      volume_mount_path.c_str());
+#endif
+
   SendResponse(true);
   return true;
 }
@@ -1405,6 +1432,8 @@ bool FileDialogStringsFunction::RunImpl() {
   SET_STRING(IDS_FILE_BROWSER, CONFIRM_DELETE_ONE);
   SET_STRING(IDS_FILE_BROWSER, CONFIRM_DELETE_SOME);
 
+  SET_STRING(IDS_FILE_BROWSER, FORMATTING_WARNING);
+
   SET_STRING(IDS_FILE_BROWSER, SELECT_FOLDER_TITLE);
   SET_STRING(IDS_FILE_BROWSER, SELECT_OPEN_FILE_TITLE);
   SET_STRING(IDS_FILE_BROWSER, SELECT_OPEN_MULTI_FILE_TITLE);