1530967 - GeckoView: Add support for taking screenshot of full page web content

Status: NEW

(GeckoView :: General, enhancement, P3)

Description

[Emily Toop (:fluffyemily)]

We would like to take screenshots of the whole page of web content to use them in our app.

Comment 1

[Chris Peterson [:cpeterson]]

Fenix will implement screenshots in Fenix M4, so GV should provide the GV API in M3.

https://github.com/mozilla-mobile/android-components/issues/1826

Comment 2

[Chris Peterson [:cpeterson]]

Vesta or Sebastian, what is the use case for full page shots? The GV API for viewport screenshots (bug 1462018) for tab thumbnails in the tab switcher landed last week. Do we need full page shots for Fenix MVP?

Comment 3

[Sebastian Kaspari (:sebastian; :pocmo)]

In this mock there's a screenshot functionality that - like desktop - allows you to screenshot the visible screen or the entire page:
https://mozilla.invisionapp.com/share/3EPQ15YUFC7#/screens/335575305

If and how that is planned for the MVP .. Vesta will know. :)

Comment 4

[Chris Peterson [:cpeterson]]

needinfo Andreas because he is talking with Barbara about screenshots.

Comment 5

[vzare]

The scope for MVP includes capturing the visible screen, the entire page, or a selected area.

Comment 6

[[ex-Mozilla] Agi Sferro | :agi]

Looks like the Desktop feature uses a canvas to take a screenshot of the entire page:

https://searchfox.org/mozilla-central/rev/7abb9117c8500ed20833746c9f8e800fce3a4688/browser/extensions/screenshots/selector/shooter.js#44-77

Comment 7

[Chris Peterson [:cpeterson]]

Moving full page shots from Fenix M3 to post-MVP as per Barbara.

Comment 9

[Chris Peterson [:cpeterson]]

I'm editing a bunch of GeckoView bugs. If you'd like to filter all this bugmail, search and destroy emails containing this UUID:

e88a5094-0fc0-4b7c-b7c5-aef00a11dbc9

Comment 10

[Petru-Mugurel Lingurar [:petru]]

Android 12 adds a new API for this - https://developer.android.com/reference/android/view/ScrollCaptureCallback
and seems that Chrome is planning to use it in the near feature - https://chromium-review.googlesource.com/c/chromium/src/+/3089805

Comment 11

[Chris Peterson [:cpeterson]]

Tasks and enhancements should have severity N/A.