Files.app Test: check the cast icon in drive volume

ui/file_manager/video_player/js/cast/caster.js

 // Copyright 2014 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.
 
 'use strict';
 
 // This hack prevents a bug on the cast extension.
 // TODO(yoshiki): Remove this once the cast extension supports Chrome apps.
 // Although localStorage in Chrome app is not supported, but it's used in the
 // cast extension. This line prevents an exception on using localStorage.
 window.__defineGetter__('localStorage', function() { return {}; });
 
 // THIS IS A TEST APP.
 // TODO(yoshiki): Fix this before launch.
 var APPLICATION_ID = '214CC863';
 
 util.addPageLoadHandler(function() {
   initialize();
 }.wrap());
 
 /**
  * Starts initialization of cast-related feature.
  */
 function initialize() {
+  if (window.loadMockCastExtensionForTest) {
+    onLoadCastExtension(initializeApi);
+    return;
+  }
+
   CastExtensionDiscoverer.findInstalledExtension(function(foundId) {
     if (foundId)
       loadCastAPI(initializeApi);
     else
       console.info('No Google Cast extension is installed.');
   }.wrap());
 }
 
 /**
  * Executes the given callback after the cast extension is initialized.
@@ skipping 24 matching lines @@
                           chrome.runtime.lastError.message);
             return;
           }
 
           console.info('Google Cast API extension installed.');
           // Loads API again.
           setTimeout(loadCastAPI.bind(null, callback, true));
         }.wrap());
   }.wrap();
 
-  var onLoad = function() {
-    if(!chrome.cast || !chrome.cast.isAvailable) {
-      var checkTimer = setTimeout(function() {
-        console.error('Either "Google Cast API" or "Google Cast" extension ' +
-                      'seems not to be installed?');
-      }.wrap(), 5000);
-
-      window['__onGCastApiAvailable'] = function(loaded, errorInfo) {
-        clearTimeout(checkTimer);
-
-        if (loaded)
-          callback();
-        else
-          console.error('Google Cast extension load failed.', errorInfo);
-      }.wrap();
-    } else {
-      setTimeout(callback);  // Runs asynchronously.
-    }
-  }.wrap();
-
   script.src = '_modules/mafeflapfdfljijmlienjedomfjfmhpd/cast_sender.js';

[hirono, 2014/08/26 05:59:01]

How about overriding the path of chrome cat API for testing instead of checking
window.loadMockCastExtensionForTest?

[yoshiki, 2014/08/26 06:11:02]

That's not good because here this doesn't the load cast api extension directly
but the cast sdk loader which call the cast api extension indirectly.

[hirono, 2014/08/26 06:33:24]

Cannot we load the mock API instead of the SDK loader?
It looks both scripts have the same role: preparing the cast API and calling
window['__onGCastApiAvailable'].

   script.addEventListener('error', onError);
-  script.addEventListener('load', onLoad);
+  script.addEventListener('load', onLoadCastExtension.bind(null, callback));
   document.body.appendChild(script);
 }
 
 /**
+ * Loads the cast sdk extension.
+ * @param {function} callback Callback (executed asynchronously).

[hirono, 2014/08/26 05:59:01]

nit: Please add type annotation to the function.

[yoshiki, 2014/08/26 06:11:02]

Done.

+ */
+function onLoadCastExtension(callback) {
+  if(!chrome.cast || !chrome.cast.isAvailable) {
+    var checkTimer = setTimeout(function() {
+      console.error('Either "Google Cast API" or "Google Cast" extension ' +
+                    'seems not to be installed?');
+    }.wrap(), 5000);
+
+    window['__onGCastApiAvailable'] = function(loaded, errorInfo) {
+      clearTimeout(checkTimer);
+
+      if (loaded)
+        callback();
+      else
+        console.error('Google Cast extension load failed.', errorInfo);
+    }.wrap();
+  } else {
+    setTimeout(callback);  // Runs asynchronously.
+  }
+};
+
+/**
  * Initialize Cast API.
  */
 function initializeApi() {
   var onSession = function() {
     // TODO(yoshiki): Implement this.
   };
 
   var onInitSuccess = function() {
     // TODO(yoshiki): Implement this.
   };
@@ skipping 24 matching lines @@
     }
 
     player.setCastList(receivers);
   } else if (availability == chrome.cast.ReceiverAvailability.UNAVAILABLE) {
     player.setCastList([]);
   } else {
     console.error('Unexpected response in onReceiver.', arguments);
     player.setCastList([]);
   }
 }