Closed Bug 784732 Opened 12 years ago Closed 12 years ago

Pause Gecko screenshots when playing video

Categories

(Core :: Audio/Video, defect)

Product:
Core
Component:
Audio/Video
Platform:
ARM
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WORKSFORME

People

(Reporter: cpeterson, Unassigned)

References

(
URL
)

Details

STR: 1. Watch adb logcat 2. Visit http://videojs.com 3. Play video AR: Relentless log messages about Gecko taking screenshots: I/GeckoScreenshot(27254): rect: 361.000000, 466.149994, 512.849976, 407.700012 I/GeckoScreenshot(27254): rect: 361.000000, 466.149994, 512.849976, 407.700012 I/GeckoScreenshot(27254): rect: 361.000000, 466.149994, 512.849976, 407.700012 I/GeckoScreenshot(27254): rect: 361.000000, 466.149994, 512.849976, 407.700012 I/GeckoScreenshot(27254): rect: 361.000000, 466.149994, 512.849976, 407.700012 I/GeckoScreenshot(27254): rect: 361.000000, 466.149994, 512.849976, 407.700012 I/GeckoScreenshot(27254): rect: 361.000000, 466.149994, 512.849976, 407.700012 I/GeckoScreenshot(27254): rect: 361.000000, 466.149994, 512.849976, 407.700012 I/GeckoScreenshot(27254): rect: 361.000000, 466.149994, 512.849976, 407.700012 I/GeckoScreenshot(27254): rect: 361.000000, 466.149994, 512.849976, 407.700012 I/GeckoScreenshot(27254): rect: 361.000000, 466.149994, 512.849976, 407.700012 I/GeckoScreenshot(27254): rect: 361.000000, 466.149994, 512.849976, 407.700012 I/GeckoScreenshot(27254): rect: 361.000000, 466.149994, 512.849976, 407.700012 I/GeckoScreenshot(27254): rect: 361.000000, 466.149994, 512.849976, 407.700012
blassey suggested in IRC: we can cancel invalidations which should prevent the screenshots if they're caused by video or just pause screenshots. that sounds simpler kats suggested in IRC: we pause [screenshots] if the app is in the background we might actually want to disable [screenshots] on full screen as well
Blocks: 755364, 787226
Presumably we can use the ended, playing, and pause events listed at https://developer.mozilla.org/en-US/docs/DOM/Media_events to disable screenshots while video is playing. I think we'd want a more fine-grained solution for the long term (assuming of course we don't get rid of screenshotting altogether) so that we can disable screenshotting of just the video playback area rather than the whole page.
Note that we also get spammed with screenshots when CSS animations are running on the page (see http://nell-balloons.github.cscott.net/ for an example).
screenshots were disabled for everything in bug 795259 changeset: http://hg.mozilla.org/mozilla-central/rev/bd0161b9250f
Status: NEW → RESOLVED
Closed: 12 years ago
Depends on: 795259
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.