Fix SD Polyfill + document.baseURI

I assumed (incorrectly) that SD Polyfill provided wrappers for document.baseURI
googlearchive · Feb 4, 2014 · cde4a8a · cde4a8a
1 parent f0dc21f
commit cde4a8a
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/HTMLImports.js b/src/HTMLImports.js
@@ -171,14 +171,16 @@ 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', {
+  var baseURIDescriptor = {
     get: function() {
       return window.location.href;
     },
     configurable: true
-  });
+  };
+
+  Object.defineProperty(document, 'baseURI', baseURIDescriptor);
+  Object.defineProperty(mainDoc, 'baseURI', baseURIDescriptor);
 }
 
 // TODO(sorvell): multiple calls will install multiple event listeners