Alternative GDrive promo for the File Manager

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 63 matching lines @@
   var DOWNLOADS_FAQ_URL =
       'http://support.google.com/chromeos/bin/answer.py?answer=1061547';
 
   /**
    * Location of the FAQ about Google Drive.
    */
   var GOOGLE_DRIVE_FAQ_URL =
       'https://support.google.com/chromeos/?p=filemanager_drive';
 
   /**
+   * Location of the Chromebook information page.
+   */
+  var CHROMEBOOK_INFO_URL = 'http://google.com/chromebook';
+
+  /**
    * Location of the page to buy more storage for Google Drive.
    */
   var GOOGLE_DRIVE_BUY_STORAGE =
       'https://www.google.com/settings/storage';
 
   /**
    * Location of Google Drive specific help.
    */
   var GOOGLE_DRIVE_HELP =
       'https://support.google.com/chromeos/?p=filemanager_drivehelp';
@@ skipping 3930 matching lines @@
   };
 
   FileManager.prototype.showGDataWelcome_ = function(type) {
     if (this.dialogContainer_.getAttribute('gdrive-welcome') != type) {
       this.dialogContainer_.setAttribute('gdrive-welcome', type);
       this.requestResize_(200);  // Resize only after the animation is done.
     }
   };
 
   FileManager.prototype.createGDataWelcomeHandler_ = function() {
+    var board = str('CHROMEOS_RELEASE_BOARD');
+    var new_welcome = board.match(/^(stumpy|lumpy)/i);
+
     var WELCOME_HEADER_COUNTER_KEY = 'gdataWelcomeHeaderCounter';
     var WELCOME_HEADER_COUNTER_LIMIT = 5;
 
     function getHeaderCounter() {
       return parseInt(localStorage[WELCOME_HEADER_COUNTER_KEY] || '0');
     }
 
     if (getHeaderCounter() >= WELCOME_HEADER_COUNTER_LIMIT)
       return null;
 
@@ skipping 20 matching lines @@
       self.showGDataWelcome_(type);
 
       var container = self.dialogDom_.querySelector('.gdrive-welcome.' + type);
       if (container.firstElementChild)
         return;  // Do not re-create.
 
       var wrapper = createDiv('gdrive-welcome-wrapper', container);
       createDiv('gdrive-welcome-icon', wrapper);
 
       var close = createDiv('cr-dialog-close', wrapper);
-      close.addEventListener('click', closeBanner);
+      close.addEventListener('click', dismissBanner);
 
       var message = createDiv('gdrive-welcome-message', wrapper);
 
       var title = createDiv('gdrive-welcome-title', message);
-      title.textContent = str('GDATA_WELCOME_TITLE');
 
       var text = createDiv('gdrive-welcome-text', message);
       text.innerHTML = str(messageId);
 
       var links = createDiv('gdrive-welcome-links', message);
 
-      var more = createDiv('gdrive-welcome-more plain-link', links);
-      more.textContent = str('GDATA_LEARN_MORE');
-      more.addEventListener('click',
-          self.onExternalLinkClick_.bind(self, GOOGLE_DRIVE_FAQ_URL));
+      var more;
+      if (new_welcome) {
+        title.textContent = str('GDATA_WELCOME_TITLE_ALTERNATIVE');
+        more = links.ownerDocument.createElement('a');

[Dmitry Zvorygin, 2012/07/25 15:15:46]

Why not to use this.document(or self.document, although I prefer bind() over
self-closures)? Isn't it clearer than links.ownerDocument ?

[Vladislav Kaznacheev, 2012/07/25 16:45:13]

Changed to self.document_
On 2012/07/25 15:15:46, Dmitry Zvorygin wrote:

+        more.className = 'gdata-welcome-button gdata-welcome-start';
+        more.textContent = str('GDATA_WELCOME_GET_STARTED');
+        more.addEventListener('click',
+            self.onExternalLinkClick_.bind(self, CHROMEBOOK_INFO_URL));
+        more.addEventListener('click', hideBanner);
+      } else {
+        title.textContent = str('GDATA_WELCOME_TITLE');
+        more = links.ownerDocument.createElement('div');
+        more.className = 'plain-link';
+        more.textContent = str('GDATA_LEARN_MORE');
+        more.addEventListener('click',
+            self.onExternalLinkClick_.bind(self, GOOGLE_DRIVE_FAQ_URL));
+      }
+      links.appendChild(more);
 
-      var dismiss = createDiv('gdrive-welcome-dismiss plain-link', links);
+      var dismiss;
+      if (new_welcome) {
+        dismiss = links.ownerDocument.createElement('a');
+        dismiss.className = 'gdata-welcome-button';
+      } else {
+        dismiss = links.ownerDocument.createElement('div');
+        dismiss.className = 'plain-link';
+      }
+      dismiss.classList.add('gdrive-welcome-dismiss');
       dismiss.textContent = str('GDATA_WELCOME_DISMISS');
-      dismiss.addEventListener('click', closeBanner);
+      dismiss.addEventListener('click', dismissBanner);
+      links.appendChild(dismiss);
     }
 
     var previousDirWasOnGData = false;
 
     function maybeShowBanner() {
       if (!self.isOnGData()) {
         self.showGDataWelcome_('none');
         previousDirWasOnGData = false;
         return;
       }
@@ skipping 11 matching lines @@
             showBanner('page', 'GDATA_WELCOME_TEXT_LONG');
         }, 2000);
       } else if (counter < WELCOME_HEADER_COUNTER_LIMIT) {
         // We do not want to increment the counter when the user navigates
         // between different directories on GDrive, but we increment the counter
         // once anyway to prevent the full page banner from showing.
         if (!previousDirWasOnGData || counter == 0)
           localStorage[WELCOME_HEADER_COUNTER_KEY] = ++counter;
         showBanner('header', 'GDATA_WELCOME_TEXT_SHORT');
       } else {
-        closeBanner();
+        dismissBanner();
       }
       previousDirWasOnGData = true;
     }
 
-    function closeBanner() {
+    function hideBanner() {
       self.cleanupGDataWelcome_();
-      // Stop showing the welcome banner.
+    }
+
+    // Hide the welcome banner and prevent it from showing again.
+    function dismissBanner() {
+      hideBanner();
       localStorage[WELCOME_HEADER_COUNTER_KEY] = WELCOME_HEADER_COUNTER_LIMIT;
     }
 
-    return maybeShowBanner;
+    function checkSpaceAndShowBanner() {
+      if (new_welcome && self.isOnGData())
+        chrome.fileBrowserPrivate.getSizeStats(self.getCurrentDirectoryURL(),
+           function(result) {
+             if (result.totalSizeKB < 100 * 1024 * 1024)

[Dmitry Zvorygin, 2012/07/25 15:15:46]

Consider using util.SCALE.

[Vladislav Kaznacheev, 2012/07/25 16:45:13]

I tried and it looks even less readable. Added a comment instead.
On 2012/07/25 15:15:46, Dmitry Zvorygin wrote:

+               maybeShowBanner();
+             else
+               dismissBanner();
+           });
+      else
+        maybeShowBanner();
+    }
+
+    return checkSpaceAndShowBanner;
   };
 
   /**
    * Listener invoked on gdata menu show event, to update gdata free/total
    * space info in opened menu.
    * @private
    */
   FileManager.prototype.onGDataMenuShow_ = function() {
     this.updateCommands_();
     this.gdataSpaceInfoBar_.setAttribute('pending', '');
@@ skipping 31 matching lines @@
       this.defaultActionMenuItem_.taskId = taskItem.taskId;
     }
 
     var defaultActionSeparator =
         this.dialogDom_.querySelector('#default-action-separator');
 
     this.defaultActionMenuItem_.hidden = !taskItem;
     defaultActionSeparator.hidden = !taskItem;
   }
 })();