@applitools/dom-snapshot - npm Package Compare versions

Comparing version 2.0.1 to 2.0.2

dist/processPage.js

@@ -1,2 +0,2 @@
-/* @applitools/dom-snapshot@2.0.0 */
+/* @applitools/dom-snapshot@2.0.1 */
 
@@ -61,3 +61,5 @@ function __processPage() {
     return (
-      frame && frame.contentDocument && !/^https?:$/.test(frame.contentDocument.location.protocol)
+      frame &&
+      frame.contentDocument &&
+      (!/^https?:$/.test(frame.contentDocument.location.protocol) || frame.src === 'about:blank')
     );
@@ -396,4 +398,4 @@ }
         const urls = resourceUrls
+          .map(resourceUrl => absolutizeUrl_1(resourceUrl, url.replace(/^blob:/, '')))
           .map(toUnAnchoredUri_1)
-          .map(resourceUrl => absolutizeUrl_1(resourceUrl, url.replace(/^blob:/, '')))
           .filter(filterInlineUrl_1);
@@ -502,3 +504,3 @@ return getResourceUrlsAndBlobs(documents, baseUrl, urls).then(({resourceUrls, blobsObj}) => ({
       const win = doc.defaultView || (doc.ownerDocument && doc.ownerDocument.defaultView) || window;
-      return uniq_1(
+      const urls = uniq_1(
         Array.from(styleSheet.cssRules || []).reduce((acc, rule) => {
@@ -524,2 +526,3 @@ if (rule instanceof win.CSSImportRule) {
       );
+      return urls.filter(u => u[0] !== '#');
     };
@@ -587,5 +590,3 @@ }
     );
-    return iframes
-      .filter(f => isAccessibleFrame(f) && !isInlineFrame_1(f))
-      .map(f => f.contentDocument);
+    return iframes.filter(f => isAccessibleFrame(f) && !isInlineFrame_1(f)).map(f => f.contentDocument);
   }
@@ -667,5 +668,5 @@
       )
-        .map(toUnAnchoredUri_1)
         .map(toUriEncoding_1)
         .map(absolutizeThisUrl)
+        .map(toUnAnchoredUri_1)
         .filter(filterInlineUrlsIfExisting);
@@ -672,0 +673,0 @@

dist/processPageAndPoll.js

@@ -1,2 +0,2 @@
-/* @applitools/dom-snapshot@2.0.0 */
+/* @applitools/dom-snapshot@2.0.1 */
 
@@ -113,3 +113,5 @@ function __processPageAndPoll() {
     return (
-      frame && frame.contentDocument && !/^https?:$/.test(frame.contentDocument.location.protocol)
+      frame &&
+      frame.contentDocument &&
+      (!/^https?:$/.test(frame.contentDocument.location.protocol) || frame.src === 'about:blank')
     );
@@ -448,4 +450,4 @@ }
         const urls = resourceUrls
+          .map(resourceUrl => absolutizeUrl_1(resourceUrl, url.replace(/^blob:/, '')))
           .map(toUnAnchoredUri_1)
-          .map(resourceUrl => absolutizeUrl_1(resourceUrl, url.replace(/^blob:/, '')))
           .filter(filterInlineUrl_1);
@@ -554,3 +556,3 @@ return getResourceUrlsAndBlobs(documents, baseUrl, urls).then(({resourceUrls, blobsObj}) => ({
       const win = doc.defaultView || (doc.ownerDocument && doc.ownerDocument.defaultView) || window;
-      return uniq_1(
+      const urls = uniq_1(
         Array.from(styleSheet.cssRules || []).reduce((acc, rule) => {
@@ -576,2 +578,3 @@ if (rule instanceof win.CSSImportRule) {
       );
+      return urls.filter(u => u[0] !== '#');
     };
@@ -639,5 +642,3 @@ }
     );
-    return iframes
-      .filter(f => isAccessibleFrame(f) && !isInlineFrame_1(f))
-      .map(f => f.contentDocument);
+    return iframes.filter(f => isAccessibleFrame(f) && !isInlineFrame_1(f)).map(f => f.contentDocument);
   }
@@ -719,5 +720,5 @@
       )
-        .map(toUnAnchoredUri_1)
         .map(toUriEncoding_1)
         .map(absolutizeThisUrl)
+        .map(toUnAnchoredUri_1)
         .filter(filterInlineUrlsIfExisting);
@@ -724,0 +725,0 @@

dist/processPageAndSerialize.js

@@ -1,2 +0,2 @@
-/* @applitools/dom-snapshot@2.0.0 */
+/* @applitools/dom-snapshot@2.0.1 */
 
@@ -113,3 +113,5 @@ function __processPageAndSerialize() {
     return (
-      frame && frame.contentDocument && !/^https?:$/.test(frame.contentDocument.location.protocol)
+      frame &&
+      frame.contentDocument &&
+      (!/^https?:$/.test(frame.contentDocument.location.protocol) || frame.src === 'about:blank')
     );
@@ -448,4 +450,4 @@ }
         const urls = resourceUrls
+          .map(resourceUrl => absolutizeUrl_1(resourceUrl, url.replace(/^blob:/, '')))
           .map(toUnAnchoredUri_1)
-          .map(resourceUrl => absolutizeUrl_1(resourceUrl, url.replace(/^blob:/, '')))
           .filter(filterInlineUrl_1);
@@ -554,3 +556,3 @@ return getResourceUrlsAndBlobs(documents, baseUrl, urls).then(({resourceUrls, blobsObj}) => ({
       const win = doc.defaultView || (doc.ownerDocument && doc.ownerDocument.defaultView) || window;
-      return uniq_1(
+      const urls = uniq_1(
         Array.from(styleSheet.cssRules || []).reduce((acc, rule) => {
@@ -576,2 +578,3 @@ if (rule instanceof win.CSSImportRule) {
       );
+      return urls.filter(u => u[0] !== '#');
     };
@@ -639,5 +642,3 @@ }
     );
-    return iframes
-      .filter(f => isAccessibleFrame(f) && !isInlineFrame_1(f))
-      .map(f => f.contentDocument);
+    return iframes.filter(f => isAccessibleFrame(f) && !isInlineFrame_1(f)).map(f => f.contentDocument);
   }
@@ -719,5 +720,5 @@
       )
-        .map(toUnAnchoredUri_1)
         .map(toUriEncoding_1)
         .map(absolutizeThisUrl)
+        .map(toUnAnchoredUri_1)
         .filter(filterInlineUrlsIfExisting);
@@ -724,0 +725,0 @@

dist/processPageCjs.js

@@ -1,2 +0,2 @@
-/* @applitools/dom-snapshot@2.0.0 */
+/* @applitools/dom-snapshot@2.0.1 */
 'use strict';
@@ -58,3 +58,5 @@
   return (
-    frame && frame.contentDocument && !/^https?:$/.test(frame.contentDocument.location.protocol)
+    frame &&
+    frame.contentDocument &&
+    (!/^https?:$/.test(frame.contentDocument.location.protocol) || frame.src === 'about:blank')
   );
@@ -393,4 +395,4 @@ }
       const urls = resourceUrls
+        .map(resourceUrl => absolutizeUrl_1(resourceUrl, url.replace(/^blob:/, '')))
         .map(toUnAnchoredUri_1)
-        .map(resourceUrl => absolutizeUrl_1(resourceUrl, url.replace(/^blob:/, '')))
         .filter(filterInlineUrl_1);
@@ -499,3 +501,3 @@ return getResourceUrlsAndBlobs(documents, baseUrl, urls).then(({resourceUrls, blobsObj}) => ({
     const win = doc.defaultView || (doc.ownerDocument && doc.ownerDocument.defaultView) || window;
-    return uniq_1(
+    const urls = uniq_1(
       Array.from(styleSheet.cssRules || []).reduce((acc, rule) => {
@@ -521,2 +523,3 @@ if (rule instanceof win.CSSImportRule) {
     );
+    return urls.filter(u => u[0] !== '#');
   };
@@ -584,5 +587,3 @@ }
   );
-  return iframes
-    .filter(f => isAccessibleFrame(f) && !isInlineFrame_1(f))
-    .map(f => f.contentDocument);
+  return iframes.filter(f => isAccessibleFrame(f) && !isInlineFrame_1(f)).map(f => f.contentDocument);
 }
@@ -664,5 +665,5 @@
     )
-      .map(toUnAnchoredUri_1)
       .map(toUriEncoding_1)
       .map(absolutizeThisUrl)
+      .map(toUnAnchoredUri_1)
       .filter(filterInlineUrlsIfExisting);
@@ -669,0 +670,0 @@

package.json

 {
   "name": "@applitools/dom-snapshot",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "main": "index.js",
@@ -5,0 +5,0 @@ "license": "MIT",

src/browser/extractFrames.js

 'use strict';
 const flat = require('./flat');
-const isInlineDocument = require('./isInlineFrame');
+const isInlineFrame = require('./isInlineFrame');
 
@@ -9,5 +9,3 @@ function extractFrames(documents = [document]) {
   );
-  return iframes
-    .filter(f => isAccessibleFrame(f) && !isInlineDocument(f))
-    .map(f => f.contentDocument);
+  return iframes.filter(f => isAccessibleFrame(f) && !isInlineFrame(f)).map(f => f.contentDocument);
 }
@@ -14,0 +12,0 @@

src/browser/extractResourcesFromStyleSheet.js

@@ -8,3 +8,3 @@ 'use strict';
     const win = doc.defaultView || (doc.ownerDocument && doc.ownerDocument.defaultView) || window;
-    return uniq(
+    const urls = uniq(
       Array.from(styleSheet.cssRules || []).reduce((acc, rule) => {
@@ -30,2 +30,3 @@ if (rule instanceof win.CSSImportRule) {
     );
+    return urls.filter(u => u[0] !== '#');
   };
@@ -32,0 +33,0 @@ }

src/browser/isInlineFrame.js

@@ -5,3 +5,5 @@ 'use strict';
   return (
-    frame && frame.contentDocument && !/^https?:$/.test(frame.contentDocument.location.protocol)
+    frame &&
+    frame.contentDocument &&
+    (!/^https?:$/.test(frame.contentDocument.location.protocol) || frame.src === 'about:blank')
   );
@@ -8,0 +10,0 @@ }

src/browser/processPage.js

@@ -59,5 +59,5 @@ 'use strict';
     )
-      .map(toUnAnchoredUri)
       .map(toUriEncoding)
       .map(absolutizeThisUrl)
+      .map(toUnAnchoredUri)
       .filter(filterInlineUrlsIfExisting);
@@ -64,0 +64,0 @@

src/browser/processResource.js

@@ -66,4 +66,4 @@ 'use strict';
       const urls = resourceUrls
+        .map(resourceUrl => absolutizeUrl(resourceUrl, url.replace(/^blob:/, '')))
         .map(toUnAnchoredUri)
-        .map(resourceUrl => absolutizeUrl(resourceUrl, url.replace(/^blob:/, '')))
         .filter(filterInlineUrl);
@@ -70,0 +70,0 @@ return getResourceUrlsAndBlobs(documents, baseUrl, urls).then(({resourceUrls, blobsObj}) => ({

No alert changes

Improved metrics

Total package byte prevSize

587508

increased by0.09%

Lines of code

12485

increased by0.06%

No dependency changes