Move __activateDir into check instead of replace

Remove unused imports in testing
Polymer · Nov 7, 2018 · ec00d26 · ec00d26
1 parent 08bc1ff
commit ec00d26
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/lib/mixins/dir-mixin.js b/lib/mixins/dir-mixin.js
@@ -115,6 +115,7 @@ export const DirMixin = dedupingMixin((base) => {
       cssText = super._processStyleText(cssText, baseURI);
       if (!SHIM_SHADOW && DIR_CHECK.test(cssText)) {
         cssText = this._replaceDirInCssText(cssText);
+        this.__activateDir = true;
       }
       return cssText;
     }
@@ -129,9 +130,6 @@ export const DirMixin = dedupingMixin((base) => {
       let replacedText = text;
       replacedText = replacedText.replace(HOST_DIR, HOST_DIR_REPLACMENT);
       replacedText = replacedText.replace(EL_DIR, EL_DIR_REPLACMENT);
-      if (text !== replacedText) {
-        this.__activateDir = true;
-      }
       return replacedText;
     }
 

diff --git a/test/unit/dir.html b/test/unit/dir.html
@@ -13,8 +13,6 @@
   <script src="../../node_modules/@webcomponents/webcomponentsjs/webcomponents-bundle.js"></script>
   <script src="wct-browser-config.js"></script>
   <script src="../../node_modules/wct-browser-legacy/browser.js"></script>
-  <script type="module" src="../../polymer-legacy.js"></script>
-  <script type="module" src="../../lib/mixins/dir-mixin.js"></script>
 </head>
 <body>
   <!--