(In reply to Karl Tomlinson (needinfo?:karlt) from comment #41)
> >+WakeLockTopic::UninhibitScreensaver()
> >+{
> >+  if (!mShouldInhibit) {
> >+    // Screensaver isn't inhibited. Nothing to do here.
> >+    return NS_OK;
> 
> Could this instead assert that this path is not reached, as in
> InhibitScreensaver()?

No. There are two states "unlocked" and "locked-background" that we map
to Uninhibit, that can both be reached directly from "locked-
foreground".

Addressing other comments, new patch coming.

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to firefox in Ubuntu.
https://bugs.launchpad.net/bugs/434476

Title:
  screensaver starts while playing HTML5 videos

Status in The Mozilla Firefox Browser:
  Confirmed
Status in “firefox” package in Ubuntu:
  Confirmed
Status in “firefox-3.5” package in Ubuntu:
  Triaged

Bug description:
  Binary package hint: firefox-3.5

  karmic alpha6 + updates

  - click on a .ogv video link
  - after watching some time, screensaver starts.

  ProblemType: Bug
  Architecture: amd64
  Date: Tue Sep 22 08:45:53 2009
  DistroRelease: Ubuntu 9.10
  NonfreeKernelModules: nvidia
  Package: firefox 3.5.3+build1+nobinonly-0ubuntu2
  PackageArchitecture: all
  ProcEnviron:
   LANG=de_AT.UTF-8
   SHELL=/bin/bash
  ProcVersionSignature: Ubuntu 2.6.31-10.34-generic
  SourcePackage: firefox-3.5
  Uname: Linux 2.6.31-10-generic x86_64

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/434476/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to     : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to