1498143 - Screenshot node menu item doesn't do anything

Status: VERIFIED FIXED

Description

[gk]

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:62.0) Gecko/20100101 Firefox/62.0

Steps to reproduce:

1. Go to 
2. Select div id="dvz-1842_ai2html_trojan-box"
3. Right-click and click on Screenshot node



Actual results:

Nothing


Expected results:

A screenshot is saved

Comment 1

[Matthias Versen [:Matti]]

>1. Go to 
go to ?

Comment 2

[gk]

https://www.bloomberg.com/news/features/2018-10-04/the-big-hack-how-china-used-a-tiny-chip-to-infiltrate-america-s-top-companies

Comment 3

[Matthias Versen [:Matti]]

>2. Select div id="dvz-1842_ai2html_trojan-box"
How do you select that div ?

>https://www.bloomberg.com/news/features/2018-10-04/the-big-hack-how-china-used-a-tiny-chip-to-infiltrate-america-s-top-companies
There is no div with that name in the source of that page.

Please provide a screenshot of your selection (use the windows screenshot function).

Comment 4

[gk]

Attachment: ai2html-node.PNG

Here is one.

Comment 5

[Punam Dahiya [:pdahiya]]

This looks like dev tools screenshot issue

Comment 6

[Patrick Brosset <:pbro> (off until July 22nd)]

Thanks for filing, I was able to reproduce the issue, and when it happened the following JS errors appeared in the Browser Console:

> Error while calling actor 'screenshot's method 'capture' node is null; can't access its "ownerDocument" property

> getRect@resource://devtools/shared/base-loader.js -> resource://devtools/shared/layout/utils.js:282:7
> createScreenshotDataURL@resource://devtools/shared/base-loader.js -> resource://devtools/shared/screenshot/capture.js:70:37
> captureScreenshot@resource://devtools/shared/base-loader.js -> resource://devtools/shared/screenshot/capture.js:41:10
> capture@resource://devtools/shared/base-loader.js -> resource://devtools/server/actors/screenshot.js:18:12
> handler@resource://devtools/shared/base-loader.js -> resource://devtools/shared/protocol.js:1198:21
> onPacket@resource://devtools/shared/base-loader.js -> resource://devtools/server/main.js:1313:15
> receiveMessage@resource://devtools/shared/base-loader.js -> resource://devtools/shared/transport/child-transport.js:66:5

> Protocol error (unknownError): node is null; can't access its "ownerDocument" property

Comment 7

[Patrick Brosset <:pbro> (off until July 22nd)]

I think this is related to capturing nodes within iframes. The code in devtools/shared/screenshot/capture.js:70 seems to always use the top level document, but then can't find the target node by its selector if that node is inside a sub-document (in an iframe).

Comment 8

[Patrick Brosset <:pbro> (off until July 22nd)]

Minimal test URL: data:text/html,<iframe srcdoc="<div id=test>screenshot me</div>">
Inspect the element inside of the iframe, right click on it in the inspector, try to use the "screenshot node" menu.

Comment 9

[Patrick Brosset <:pbro> (off until July 22nd)]

Attachment: WIP-node-screenshot-with-iframes.diff

This is an idea for how this might work.
This is creating an array of parent iframe selectors in the inspector, and then using this array to find the correct node.
I do not have time to work on this right now, but if this is the right approach, then we need to polish this up and potentially add:
- tests (there does not seem to be any tests today for the node screenshot feature)
- supports for shadow DOM (although this could be done in a follow-up too)

Comment 10

[Julian Descottes [:jdescottes]]

Attachment: Bug 1498143 - Pass nodeActorID to screenshot actor to enable feature in iframes and shadowroots;r=pbro,yulia

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9ee66620694f
Pass nodeActorID to screenshot actor to enable feature in iframes and shadowroots;r=pbro,yulia

Comment 12

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/9ee66620694f

Comment 13

[Maria Berlinger [:maria_berlinger], Release Desktop QA]

I have managed to reproduce this issue on an affected Firefox 62.0.3 build using Windows 10 x64  by following the STR from comment 0. 

This issue is verified fixed using Firefox 65.0b6 on the following OSes: Windows 10 x64, Ubuntu 18.04 x64, macOS 10.14.