-
Notifications
You must be signed in to change notification settings - Fork 162
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix The document.open function throws an error because document.defaultView returns null
(close #1272)
#1279
Conversation
…ultView returns null` (close DevExpress#1272)
✅ Tests for the commit 59a86aa have passed. See details: |
|
||
var iframeDocument = iframe.contentDocument; | ||
test('should not throw an error when document.defualtView is null', function () { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
test(should not throw an error when document.defualtView is null (GH-1272))
|
||
strictEqual(nativeMethods.querySelectorAll.call(document, '.' + SHADOW_UI_CLASSNAME.selfRemovingScript).length, 0); | ||
// NOTE: Without setTimeout the error does not reproduced |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// NOTE: Without wrapping in 'setTimeout' function the error is not reproduced
✅ Tests for the commit b8fade9 have passed. See details: |
ping @churkin |
…ultView returns null` (close DevExpress#1272) (DevExpress#1279) * fix `The document.open function throws an error because document.defaultView returns null` (close DevExpress#1272) * test refactoring * fix review's issues * fix review's issues
/cc @miherlosev @churkin