gdata: Apply correct mount label when mounting archives in GData

chrome/browser/resources/file_manager/js/file_manager.js

 // Copyright (c) 2012 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.
 
 /**
  * FileManager constructor.
  *
  * FileManager objects encapsulate the functionality of the file selector
  * dialogs, as well as the full screen file manager application (though the
  * latter is not yet implemented).
@@ skipping 676 matching lines @@
       this.unmountedPanel_.setAttribute('loading', true);
     }
 
     // If the user changed to another directory and then back to GData we
     // re-enter this method while the timer is still active. In this case
     // we only update the UI but do not request the mount again.
     if (this.gdataLoadingTimer_)
       return;
 
     metrics.startInterval('Load.GData');
-    chrome.fileBrowserPrivate.addMount('', 'gdata', {});
+    chrome.fileBrowserPrivate.addMount('', 'gdata', {},
+                                       function(sourcePath) {});
 
     // This timer could fire before the mount succeeds. We will silently
     // replace the error message with the correct directory contents.
     this.gdataLoadingTimer_ = setTimeout(function() {
           this.gdataLoadingTimer_ = null;
           this.onGDataUnreachable_('GData load timeout');
         }.bind(this),
         15 * 60 * 1000);
   };
 
@@ skipping 1915 matching lines @@
             // This device mount was the reason this File Manager instance was
             // created. Now the device is unmounted from another instance
             // or the user removed the device manually. Close this instance.
             // window.close() sometimes doesn't work.
             chrome.tabs.getCurrent(function(tab) {
               chrome.tabs.remove(tab.id);
             });
             return;
           }
           // Current directory just unmounted. Move to the 'Downloads'.
-          changeDirectoryTo = this.directoryModel_.getDefaultDirectory();
+          changeDirectoryTo = self.directoryModel_.getDefaultDirectory();
         }
       }
 
       // Even if something failed root list should be rescanned.
       // Failed mounts can "give" us new devices which might be formatted,
       // so we have to refresh root list then.
       self.directoryModel_.updateRoots(function() {
         if (changeDirectoryTo) {
           self.directoryModel_.changeDirectory(changeDirectoryTo);
         }
@@ skipping 15 matching lines @@
         position = urls.indexOf(selectedUrl);
       }
       chrome.mediaPlayerPrivate.play(urls, position);
     } else if (id == 'mount-archive') {
       var self = this;
       this.resolveSelectResults_(urls, function(urls) {
         for (var index = 0; index < urls.length; ++index) {
           // Url in MountCompleted event won't be escaped, so let's make sure
           // we don't use escaped one in mountRequests_.
           var unescapedUrl = unescape(urls[index]);
-          self.mountRequests_.push(unescapedUrl);
-          chrome.fileBrowserPrivate.addMount(unescapedUrl, 'file', {});
+          var callback = function(sourcePath) {
+            self.mountRequests_.push(self.isOnGData() ? sourcePath :
+                                                        unescapedUrl);
+          }
+          chrome.fileBrowserPrivate.addMount(unescapedUrl, 'file', {},

[tbarzic, 2012/04/27 05:44:21]

How about:
chrome.fileBrowserPrivate.addMount(unescapedUrl, 'file', {}.
     function(sourcePath) {
       self.mountRequests_.push(sourcePath);
     });

[hshi, 2012/04/27 18:48:56]

Done.

+                                             callback);
         }
       });
     } else if (id == 'format-device') {
       this.confirm.show(str('FORMATTING_WARNING'), function() {
         chrome.fileBrowserPrivate.formatDevice(urls[0]);
       });
     } else if (id == 'gallery') {
       this.openGallery_(urls);
     } else if (id == 'view-pdf' || id == 'view-in-browser' ||
         id == 'install-crx' || id == 'open-hosted') {
@@ skipping 1690 matching lines @@
 
     this.directoryModel_.addEventListener('scan-completed', maybeShowBanner);
     this.directoryModel_.addEventListener('rescan-completed', maybeShowBanner);
 
     var style = this.document_.createElement('link');
     style.rel = 'stylesheet';
     style.href = 'css/gdrive_welcome.css';
     this.document_.head.appendChild(style);
   };
 })();