Windows changes for File IO, File dialog and getElapsedMilliseconds.

appshell/appshell_extensions_win.cpp

@@ -26,9 +26,13 @@
 #include <algorithm>
 #include <CommDlg.h>
 #include <ShlObj.h>
+#include <Shlwapi.h>
+#include <stdio.h>
+#include <sys/stat.h>
 
 // Forward declarations for functions at the bottom of this file
 void FixFilename(ExtensionString& filename);
+void EscapeJSONString(const std::wstring& str, std::wstring& finalResult);
 int ConvertErrnoCode(int errorCode, bool isReading = true);
 int ConvertWinErrorCode(int errorCode, bool isReading = true);
 
@@ -103,8 +107,6 @@ int32 ShowOpenDialog(bool allowMulitpleSelection,
         ofn.lpstrFile = szFile;
         ofn.nMaxFile = MAX_PATH;
 
-        allowMulitpleSelection = false;  // TODO: Raymond, please implement.
-
         // TODO (issue #65) - Use passed in file types. Note, when fileTypesStr is null, all files should be shown
         /* findAndReplaceString( fileTypesStr, std::string(" "), std::string(";*."));
         LPCWSTR allFilesFilter = L"All Files\0*.*\0\0";*/
@@ -119,23 +121,19 @@ int32 ShowOpenDialog(bool allowMulitpleSelection,
         if (GetOpenFileName(&ofn)) {
             if (allowMulitpleSelection) {
                 // Multiple selection encodes the files differently
-/*
+
                 // If multiple files are selected, the first null terminator
                 // signals end of directory that the files are all in
                 std::wstring dir(szFile);
 
                 // Check for two null terminators, which signal that only one file
                 // was selected
                 if (szFile[dir.length() + 1] == '\0') {
-                    // Escape the single file path and add it to the JSON array
-                    std::wstring escaped;
-                    EscapeJSONString(dir, escaped);
-                    results += L"\"" + escaped + L"\"";
+                    selectedFiles->SetString(0, ExtensionString(dir));
                 } else {
                     // Multiple files are selected
                     wchar_t fullPath[MAX_PATH];
-                    bool firstFile = true;
-                    for (int i = dir.length() + 1;;) {
+                    for (int i = (dir.length() + 1), fileIndex = 0; ; fileIndex++) {
                         // Get the next file name
                         std::wstring file(&szFile[i]);
 
@@ -146,24 +144,13 @@ int32 ShowOpenDialog(bool allowMulitpleSelection,
                         // The filename is relative to the directory that was specified as
                         // the first string
                         if (PathCombine(fullPath, dir.c_str(), file.c_str()) != NULL)
-                        {
-                            // Append a comma separator if it is not the first file in the list
-                            if (firstFile)
-                                firstFile = false;
-                            else
-                                results += L",";
-
-                            // Escape the path and add it to the list
-                            std::wstring escaped;
-                            EscapeJSONString(std::wstring(fullPath), escaped);
-                            results += L"\"" + escaped + L"\"";
-                        }
+                            selectedFiles->SetString(fileIndex, ExtensionString(fullPath));
 
                         // Go to the start of the next file name
                         i += file.length() + 1;
                     }
                 }
-*/
+
             } else {
                 // If multiple files are not allowed, add the single file
                 selectedFiles->SetString(0, szFile);
@@ -306,13 +293,34 @@ int32 WriteFile(ExtensionString filename, std::string contents, ExtensionString
 
 int32 SetPosixPermissions(ExtensionString filename, int32 mode)
 {
-    // TODO: Raymond, please implement
+    FixFilename(filename);
+
+    DWORD dwAttr = GetFileAttributes(filename.c_str());
+
+    if (dwAttr == INVALID_FILE_ATTRIBUTES)
+        return ERR_NOT_FOUND;
+
+    if ((dwAttr & FILE_ATTRIBUTE_DIRECTORY) != 0)
+        return NO_ERROR;
+
+    bool write = (mode & 0200) != 0; 
+    bool read = (mode & 0400) != 0;
+    int mask = (write ? _S_IWRITE : 0) | (read ? _S_IREAD : 0);
+
+    if (_wchmod(filename.c_str(), mask) == -1) {
+        return ConvertErrnoCode(errno); 
+    }
+
     return NO_ERROR;
 }
 
 int32 DeleteFileOrDirectory(ExtensionString filename)
 {
-    // TODO: Raymond, please implement
+    FixFilename(filename);
+
+    if (!DeleteFile(filename.c_str()))
+        return ConvertWinErrorCode(GetLastError());
+
     return NO_ERROR;
 }
 

appshell/cefclient_win.cpp

@@ -8,6 +8,7 @@
 #include <commdlg.h>
 #include <shellapi.h>
 #include <direct.h>
+#include <MMSystem.h>
 #include <sstream>
 #include <string>
 #include "include/cef_app.h"
@@ -28,6 +29,7 @@
 #endif // SHOW_TOOLBAR_UI
 
 // Global Variables:
+DWORD g_appStartupTime;
 HINSTANCE hInst;   // current instance
 TCHAR szTitle[MAX_LOADSTRING];  // The title bar text
 TCHAR szWindowClass[MAX_LOADSTRING];  // the main window class name
@@ -56,6 +58,8 @@ int APIENTRY wWinMain(HINSTANCE hInstance,
   UNREFERENCED_PARAMETER(hPrevInstance);
   UNREFERENCED_PARAMETER(lpCmdLine);
 
+  g_appStartupTime = timeGetTime();
+
   CefMainArgs main_args(hInstance);
   CefRefPtr<ClientApp> app(new ClientApp);
 

appshell/client_app_win.cpp

@@ -24,8 +24,11 @@
 #include "client_app.h"
 #include "resource.h"
 
+#include <MMSystem.h>
 #include <string>
 
+extern DWORD g_appStartupTime;
+
 std::string ClientApp::GetExtensionJSSource()
 {
     extern HINSTANCE hInst;
@@ -54,6 +57,6 @@ std::string ClientApp::GetExtensionJSSource()
 
 double ClientApp::GetElapsedMilliseconds()
 {
-    // TODO: Raymond, please implement
-    return 0;
+    return (timeGetTime() - g_appStartupTime);
 }
+