Fixed documentation inside source files

Author: nethip

appshell/appshell_extensions.cpp

@@ -134,9 +134,8 @@ class ProcessMessageDelegate : public ClientHandler::ProcessMessageDelegate {
                                browser,
                                response);
 
-                // On MAC, beginModalSheet is going
-                // to inform us asynchronusly. So here return from here
-                // and delay sending the response until later.
+                // Skip standard callback handling. ShowOpenDialog fires the
+                // callback asynchronously.
 
                 return true;
 #else
@@ -172,10 +171,8 @@ class ProcessMessageDelegate : public ClientHandler::ProcessMessageDelegate {
                 ExtensionString proposedNewFilename = argList->GetString(3);
 
 #ifdef OS_MACOSX
-                // ShowSaveDialog itself is going to
-                // take of communicating with the
-                // render process.
-                
+                // Skip standard callback handling. ShowSaveDialog fires the
+                // callback asynchronously.
                 ShowSaveDialog(title,
                                initialPath,
                                proposedNewFilename,

appshell/appshell_extensions_mac.mm

@@ -409,8 +409,8 @@ void ShowOpenDialog(bool allowMulitpleSelection,
 
     [openPanel setAllowedFileTypes:allowedFileTypes];
 
-    // cache the browser details, so has to access
-    // the same inside the completionHandler block.
+    // cache the browser and response variables, so that these
+    // can be accessed from within the completionHandler block.
     CefRefPtr<CefBrowser>        _browser  = browser;
     CefRefPtr<CefProcessMessage> _response = response;
 
@@ -451,8 +451,8 @@ void ShowSaveDialog(ExtensionString title,
 
     [savePanel setNameFieldStringValue:[NSString stringWithUTF8String:proposedNewFilename.c_str()]];
 
-    // cache the browser details, so has to access
-    // the same inside the completionHandler block.
+    // cache the browser and response variables, so that these
+    // can be accessed from within the completionHandler block.
     CefRefPtr<CefBrowser>        _browser  = browser;
     CefRefPtr<CefProcessMessage> _response = response;
 

appshell/client_handler_mac.mm

@@ -61,11 +61,13 @@
   if ([window styleMask] & NSFullScreenWindowMask) {
       [window setTitle:str];
   }
-    
-  // This is required as we are now managing the window title
-  // on our own, because of which the window title entires 
-  // are not getting added to 'Windows' menu. This is an
-  // an official API.
+    // This is required as we are now managing the window's title
+    // on our own, using CustomTitlbeBarView. As we are nuking the
+    // window's title, currently open windows like new brackets
+    // windows/dev tools are not getting listed under the
+    // 'Window' menu. So we are now explicitly telling OS to make
+    // sure we have a menu entry under 'Window' menu. This is a
+    // safe call and is officially supported.
   [NSApp changeWindowsItem:window title:str filename:NO];
 
   [delegate performSelectorOnMainThread:@selector(windowTitleDidChange:) withObject:str waitUntilDone:NO];