Add mechanism for recording significant composites for creating recordings of things like pageload

RESOLVED FIXED in Firefox 68

Status

()

enhancement
RESOLVED FIXED
3 months ago
2 months ago

People

(Reporter: bas.schouten, Assigned: bas.schouten)

Tracking

unspecified
mozilla68
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox68 fixed)

Details

Attachments

(3 attachments)

Assignee

Description

3 months ago

Since screen capture software run locally can interfere significantly with pageload and browser operation. It would be good to have a way to efficiently be able to capture visual page load progress.

Comment 4

3 months ago
Pushed by bschouten@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8186704e6d03
Part 1: Add API for recording composition results with content. r=mstange
https://hg.mozilla.org/integration/mozilla-inbound/rev/ffb2e68a6445
Part 2: Add nsDOMWindowUtils API for enabling window recording. r=mstange
https://hg.mozilla.org/integration/mozilla-inbound/rev/fd1779ce4ec7
Part 3: Add ability to start and stop composition recordings from a keyboard combo. r=mconley

Comment 5

3 months ago
bugherder
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
Depends on: 1540521

Bas: this doesn't look very arch specific, but in a meeting today you suggested it needed additional work to function on Android. Can you confirm, and if that's so, open a follow-up and sketch out that work? Thanks!

Flags: needinfo?(bas)
Assignee

Comment 7

3 months ago

(In reply to Nick Alexander :nalexander [he/him] from comment #6)

Bas: this doesn't look very arch specific, but in a meeting today you suggested it needed additional work to function on Android. Can you confirm, and if that's so, open a follow-up and sketch out that work? Thanks!

Code needs to be added to grab a copy of the framebuffer in LayerManagerComposite/CompositorOGL. Brennie or Markus would probably know exactly what needs to be done. It's the bit in gfx/layers/mlgpu/LayerManagerMLGPU.cpp that's arch-specific.

Flags: needinfo?(bas)
Blocks: 1544572
You need to log in before you can comment on or make changes to this bug.