Do more strict checking when calling into MathJaX

chrome/browser/resources/chromeos/chromevox/chromevox/injected/api.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.
 
 /**
  * @fileoverview Public APIs to enable web applications to communicate
  * with ChromeVox.
  */
 if (typeof(goog) != 'undefined' && goog.provide) {
   goog.provide('cvox.Api');
@@ skipping 110 matching lines @@
     */
    function callSpeakAsync_(message, properties) {
      var callback = null;
      /* Use the user supplied callback as callAsync_'s callback. */
      if (properties && properties['endCallback']) {
        callback = properties['endCallback'];
      }
      callAsync_(message, callback);
    };
 
+   /**
+    * @param {string} path
+    * @return {*}
+    */
+   function getPath(path) {

[dmazzoni, 2017/03/20 18:16:15]

Could you name this something more clear and/or add
docstring? "Path" is an overloaded term. How about
resolveGlobalPath?

+     var pieces = path.split('.');
+     var resolved = window;
+     for (var i = 0; i < pieces.length; i++) {
+       resolved = resolved[pieces[i]];
+       if (!resolved)
+       return null;

[dmazzoni, 2017/03/20 18:16:15]

nit: indent

+     }
+     return resolved;
+   }
+
 
    /**
     * Maybe enable MathJaX support.
     */
-  function maybeEnableMathJaX() {
-     if (!MathJax || !MathJax.Hub || !MathJax.Hub.Register ||
-         !MathJax.Hub.Register.LoadHook)
+   function maybeEnableMathJaX() {
+     if (!getPath('MathJax.Hub.Register.LoadHook') ||
+         !getPath('MathJax.Ajax.Require')) {
        return;
+     }
 
-     MathJax.Hub.Register.LoadHook(
-         '[a11y]/explorer.js',
-         function() {
-           MathJax.Extension.explorer.Enable(true, true);
-         });
+     MathJax.Hub.Register.LoadHook('[a11y]/explorer.js', function() {
+       // |explorer| is an object instance, so we get it to call an instance
+       // |method.
+       var explorer = getPath('MathJax.Extension.explorer');
+       if (explorer.Enable) {
+         explorer.Enable(true, true);
+       }
+     });
      MathJax.Ajax.Require('[a11y]/explorer.js');
    }
 
 
    /*
     * Public API.
     */
 
    if (!window['cvox']) {
      window['cvox'] = {};
@@ skipping 78 matching lines @@
    /**
    * This method is kept to keep Docs from throwing an error.
    *
    */
    cvox.Api.stop = function() {
    };
 
    cvox.Api.internalEnable();
 
 })();