Merge pull request #47 from Polymer/dynamic-ie

Fixes for IE

Author: Steve Orvell

build.json

@@ -1,4 +1,5 @@
 [
+  "../MutationObservers/build.json",
   "src/scope.js",
   "src/Loader.js",
   "src/Parser.js",

conf/karma.conf.js

@@ -13,7 +13,9 @@ module.exports = function(karma) {
       'HTMLImports/html-imports.js',
       {pattern: 'tools/**/*.js', included: false},
       {pattern: 'HTMLImports/src/*', included: false},
-      {pattern: 'HTMLImports/test/**/*', included: false}
+      {pattern: 'HTMLImports/test/**/*', included: false},
+      {pattern: 'MutationObservers/*.js', included: false},
+      {pattern: 'WeakMap/*.js', included: false}
     ],
   }));
 };

html-imports.js

@@ -9,6 +9,7 @@
 var thisFile = 'html-imports.js';
 var scopeName = 'HTMLImports';
 var modules = [
+  '../MutationObservers/mutation-observers.js',
   'src/scope.js',
   'src/Loader.js',
   'src/Parser.js',

src/HTMLImports.js

@@ -164,13 +164,23 @@ var currentScriptDescriptor = {
   get: function() {
     return HTMLImports.currentScript || document.currentScript;
   },
-  writeable: true,
   configurable: true
-}
+};
 
 Object.defineProperty(document, '_currentScript', currentScriptDescriptor);
 Object.defineProperty(mainDoc, '_currentScript', currentScriptDescriptor);
 
+// Polyfill document.baseURI for browsers without it.
+// ShadowDOM Polyfill wrapper should handle this automatically.
+if (!document.baseURI) {
+  Object.defineProperty(document, 'baseURI', {
+    get: function() {
+      return window.location.href;
+    },
+    configurable: true
+  });
+}
+
 // TODO(sorvell): multiple calls will install multiple event listeners
 // which may not be desireable; calls should resolve in the correct order,
 // however.

src/Observer.js

@@ -12,7 +12,8 @@ var importSelector = 'link[rel=' + IMPORT_LINK_TYPE + ']';
 var matches = HTMLElement.prototype.matches || 
     HTMLElement.prototype.matchesSelector || 
     HTMLElement.prototype.webkitMatchesSelector ||
-    HTMLElement.prototype.mozMatchesSelector;
+    HTMLElement.prototype.mozMatchesSelector ||
+    HTMLElement.prototype.msMatchesSelector;
 
 var importer = scope.importer;
 
@@ -22,7 +23,7 @@ function handler(mutations) {
       addedNodes(m.addedNodes);
     }
   }
-};
+}
 
 function addedNodes(nodes) {
   for (var i=0, l=nodes.length, n; (i<l) && (n=nodes[i]); i++) {

src/Parser.js

@@ -109,6 +109,32 @@ var importParser = {
     };
     elt.addEventListener('load', done);
     elt.addEventListener('error', done);
+
+    // NOTE: IE does not fire "load" event for styles that have already loaded
+    // This is in violation of the spec, so we try our hardest to work around it
+    if (isIe && elt.localName === 'style') {
+      var fakeLoad = false;
+      // If there's not @import in the textContent, assume it has loaded
+      if (elt.textContent.indexOf('@import') == -1) {
+        fakeLoad = true;
+      // if we have a sheet, we have been parsed
+      } else if (elt.sheet) {
+        fakeLoad = true;
+        var csr = elt.sheet.cssRules;
+        var len = csr ? csr.length : 0;
+        // search the rules for @import's
+        for (var i = 0, r; (i < len) && (r = csr[i]); i++) {
+          if (r.type === CSSRule.IMPORT_RULE) {
+            // if every @import has resolved, fake the load
+            fakeLoad = fakeLoad && Boolean(r.styleSheet);
+          }
+        }
+      }
+      // dispatch a fake load event and continue parsing
+      if (fakeLoad) {
+        elt.dispatchEvent(new CustomEvent('load', {bubbles: false}));
+      }
+    }
   },
   parseScript: function(scriptElt) {
     // acquire code to execute
@@ -216,4 +242,4 @@ scope.parser = importParser;
 scope.path = path;
 scope.isIE = isIe;
 
-})(HTMLImports);
+})(HTMLImports);