Merge pull request #44 from Chris-Boyle/fix-hasAttribute-error

Conditionally check hasAttribute in the event the object isn't an element.

Author: mhemesath

CONTRIBUTORS.md

@@ -10,6 +10,7 @@ Cerner Corporation
 - Supriti Yeotikar [@supritiy]
 - Cody Price [@dev-cprice]
 - Roxanne Calderon [@foxannefoxanne]
+- Yusuf Ali [@yusufali2205]
 
 [@mhemesath]: https://github.com/mhemesath
 [@kafkahw]: https://github.com/kafkahw
@@ -21,3 +22,4 @@ Cerner Corporation
 [@supritiy]: https://github.com/supriticerner
 [@dev-cprice]: https://github.com/dev-cprice
 [@foxannefoxanne]: https://github.com/foxannefoxanne
+[@yusufali2205]: https://github.com/yusufali2205

package.json

@@ -17,6 +17,7 @@
     "clean": "rm -rf dist lib doc",
     "dev": "webpack-dev-server --config webpack.config.dev.js",
     "lint-js": "eslint 'src/**/*.js' --fix",
+    "prepare": "npm run build-es5 && npm run build-webpack",
     "test": "mocha --compilers js:babel-core/register --recursive"
   },
   "dependencies": {

src/provider/application.js

@@ -6,7 +6,6 @@ import logger from '../lib/logger';
 import { getOffsetHeightToBody, calculateHeight, calculateWidth } from '../lib/dimension';
 import MutationObserver from 'mutation-observer';
 
-
 /** Application class which represents an embedded application. */
 class Application extends EventEmitter {
   /**
@@ -273,7 +272,7 @@ class Application extends EventEmitter {
     const protocols = /^(tel|mailto|fax|sms|callto):/;
     const element = document.activeElement;
 
-    if (!element || !(element.hasAttribute('download') || protocols.test(element.href))) {
+    if (!element || !(element.hasAttribute && element.hasAttribute('download') || protocols.test(element.href))) {
       this.JSONRPC.notification('unload');
       this.trigger('xfc.unload');
     }

src/provider/provider.js

@@ -6,7 +6,7 @@ class Provider {
     const enforceSecurity = config.secret || config.acls.some((x) => x !== '*');
 
     // Set hidden attribute with script if not present and security is being enforced
-    if (enforceSecurity && window.self !== window.top && !document.documentElement.hasAttribute('hidden')) {
+    if (enforceSecurity && window.self !== window.top && !(document.documentElement.hasAttribute && document.documentElement.hasAttribute('hidden'))) {
       document.documentElement.setAttribute('hidden', null);
 
       // WARNING: Setting hidden attribute with script can be countered by