fix Shouldn't process shadow ui elements (close #1570)

DevExpress · Apr 16, 2018 · bcebd87 · bcebd87
1 parent a301dcd
commit bcebd87
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/src/client/sandbox/node/index.js b/src/client/sandbox/node/index.js
@@ -46,7 +46,7 @@ export default class NodeSandbox extends SandboxBase {
     _processElement (el) {
         const processedContext = el[INTERNAL_PROPS.processedContext];
 
-        if (processedContext !== this.window) {
+        if (!domUtils.isShadowUIElement(el) && processedContext !== this.window) {
             let urlAttrName = null;
 
             if (processedContext) {

diff --git a/test/client/fixtures/sandbox/shadow-ui-test.js b/test/client/fixtures/sandbox/shadow-ui-test.js
@@ -1,5 +1,6 @@
 var SHADOW_UI_CLASSNAME = hammerhead.get('./../shadow-ui/class-name');
 var ShadowUI            = hammerhead.get('./sandbox/shadow-ui');
+var INTERNAL_PROPS      = hammerhead.get('../processing/dom/internal-properties');
 
 var shadowUI      = hammerhead.sandbox.shadowUI;
 var domUtils      = hammerhead.utils.dom;
@@ -838,4 +839,13 @@ test('processed nodeList should have non-enumerable "item" and "namedItem" prope
     strictEqual(collectionKeys.indexOf('item'), -1, 'item');
     strictEqual(collectionKeys.indexOf('namedItem'), -1, 'namedItem');
     strictEqual(collectionKeys.indexOf('length'), -1, 'length');
+
+    document.body.removeChild(div);
+});
+
+test('should not process shadow ui elements (GH-1570)', function () {
+    var shadowUIElements = hammerhead.nativeMethods.querySelectorAll.call(document, '[class$="-hammerhead-shadow-ui"]');
+
+    for (var i = 0; i < shadowUIElements.length; i++)
+        ok(!shadowUIElements[i][INTERNAL_PROPS.processedContext]);
 });